diff --git a/crates/fj-core/src/operations/sweep/face.rs b/crates/fj-core/src/operations/sweep/face.rs index bef57f79f..ac33f810b 100644 --- a/crates/fj-core/src/operations/sweep/face.rs +++ b/crates/fj-core/src/operations/sweep/face.rs @@ -15,7 +15,7 @@ use super::{SweepCache, SweepRegion}; /// /// [module documentation]: super pub trait SweepFace { - /// # Sweep the [`Face`] + /// # Sweep the [`Face`] into a [`Shell`] fn sweep_face( &self, path: impl Into>, @@ -51,12 +51,12 @@ impl SweepFace for Handle { let bottom_face = self.clone(); faces.push(bottom_face.clone()); - let side_faces = bottom_face + let other_faces = bottom_face .region() .sweep_region(bottom_face.surface(), path, cache, core) .all_faces() .map(|side_face| side_face.insert(&mut core.services)); - faces.extend(side_faces); + faces.extend(other_faces); Shell::new(faces) } diff --git a/crates/fj-core/src/operations/sweep/region.rs b/crates/fj-core/src/operations/sweep/region.rs index 700d9deb1..bc54c7148 100644 --- a/crates/fj-core/src/operations/sweep/region.rs +++ b/crates/fj-core/src/operations/sweep/region.rs @@ -60,21 +60,21 @@ impl SweepRegion for Region { core, ); - let mut top_interiors = Vec::new(); - - for bottom_cycle in self.interiors() { - let top_cycle = sweep_cycle( - bottom_cycle, - surface, - self.color(), - &mut faces, - path, - cache, - core, - ); - - top_interiors.push(top_cycle); - } + let top_interiors = self + .interiors() + .iter() + .map(|bottom_cycle| { + sweep_cycle( + bottom_cycle, + surface, + self.color(), + &mut faces, + path, + cache, + core, + ) + }) + .collect::>(); let top_face = { let top_surface = surface