From 3eaba67f5853a2f268fef77b72f2e901bdc43fda Mon Sep 17 00:00:00 2001 From: "Dylan M. Taylor" Date: Fri, 13 Nov 2026 02:05:13 -0500 Subject: [PATCH] Fix Clippy range loop in path orientation --- crates/engine/src/cmd/paths.rs | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/crates/engine/src/cmd/paths.rs b/crates/engine/src/cmd/paths.rs index 7800b3189..b3f8d3300 100644 --- a/crates/engine/src/cmd/paths.rs +++ b/crates/engine/src/cmd/paths.rs @@ -79,11 +79,13 @@ fn spread_xf(d: &Document, id: ItemId) -> Option<(SpreadRef, Affine)> { /// non-zero rule they become holes. fn orient_holes(path: &mut PathData) { let beziers: Vec = path.subpaths.iter().map(|s| PathData::single(s.clone()).to_bezpath()).collect(); - let n = path.subpaths.len(); - for i in 0..n { - let Some(p) = path.subpaths[i].anchors.first().map(|a| a.p) else { continue }; - let depth = (0..n).filter(|&j| j != i && designcraft_geom::hit::fill_contains(&beziers[j], designcraft_geom::FillRule::NonZero, p)).count(); - let sp = &mut path.subpaths[i]; + for (i, sp) in path.subpaths.iter_mut().enumerate() { + let Some(p) = sp.anchors.first().map(|a| a.p) else { continue }; + let depth = beziers + .iter() + .enumerate() + .filter(|(j, bezier)| *j != i && designcraft_geom::hit::fill_contains(bezier, designcraft_geom::FillRule::NonZero, p)) + .count(); let cw = sp.area() > 0.0; if cw == (depth % 2 == 1) { sp.reverse();