Skip to content

fix(core): satisfy clippy::manual_flatten in scan stat pass - #3

Merged
kingchenc merged 2 commits into
mainfrom
fix/clippy-manual-flatten
Jul 10, 2026
Merged

kingchenc merged 2 commits into
mainfrom
fix/clippy-manual-flatten

Conversation

@kingchenc

Copy link
Copy Markdown
Owner

The 0.5.0 perf change (#2) parallelised the scan stat pass with

for res in children.iter_mut() { if let Ok(e) = res { ... } }

which clippy::manual_flatten denies under -D warnings, turning main CI red on all three OS. Iterate the Ok entries directly via .flatten() — no behaviour change.

Verified locally: cargo clippy --workspace --all-targets -- -D warnings and cargo fmt --all --check both clean.

The parallelized stat pass used `for res in children.iter_mut() { if let Ok(e) = res }`,
which clippy::manual_flatten denies under -D warnings. Iterate the Ok entries directly
via .flatten(); no behaviour change.
A new advisory (RUSTSEC-2026-0204: invalid pointer dereference in the
fmt::Pointer impl for Atomic/Shared) was published against crossbeam-epoch
0.9.18, pulled transitively via jwalk and rayon — turning cargo-deny red
on its own. Lockfile-only bump to the patched 0.9.20.
@kingchenc
kingchenc merged commit c6ff647 into main Jul 10, 2026
7 checks passed
@kingchenc
kingchenc deleted the fix/clippy-manual-flatten branch July 10, 2026 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant