daniel543x/designcraft
1
0 Fork 0
mirror of https://github.com/storytold/designcraft.git synced 2026-11-14 14:22:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix Clippy range loop in path orientation

Browse Source
This commit is contained in:
Dylan M. Taylor committed 2026-11-13 02:05:13 -05:00
1 parent 32c239cacc
commit 3eaba67f58
1 file changed
+7 -5
Download Patch File Download Diff File Expand all files Collapse all files
crates/engine/src/cmd/paths.rs
+7 -5
View File
@@ -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<BezPath> = 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();