Set curve geometry in BuildShell method

hannobraun · Apr 30, 2024 · b29d731 · b29d731
1 parent e032ab1
commit b29d731
Showing 1 changed file with 16 additions and 10 deletions.
diff --git a/crates/fj-core/src/operations/build/shell.rs b/crates/fj-core/src/operations/build/shell.rs
@@ -4,10 +4,10 @@ use fj_interop::ext::ArrayExt;
 use fj_math::Point;
 
 use crate::{
-    geometry::CurveBoundary,
+    geometry::{CurveBoundary, HalfEdgeGeom},
     operations::{
         build::{BuildFace, BuildHalfEdge, BuildSurface, Polygon},
-        geometry::UpdateHalfEdgeGeometry,
+        geometry::{UpdateCurveGeometry, UpdateHalfEdgeGeometry},
         insert::{Insert, IsInserted, IsInsertedNo, IsInsertedYes},
         join::JoinCycle,
         reverse::ReverseCurveCoordinateSystems,
@@ -90,21 +90,27 @@ pub trait BuildShell {
                         .map(|((vertex, positions), (curve, boundary))| {
                             let boundary = boundary.reverse();
 
-                            let half_edge = HalfEdge::line_segment(
+                            let curve = curve.make_line_on_surface(
                                 positions,
                                 Some(boundary),
                                 surface.clone(),
-                                core,
+                                &mut core.layers.geometry,
                             );
-                            half_edge
+
+                            HalfEdge::unjoined(core)
                                 .update_start_vertex(|_, _| vertex, core)
-                                .update_curve(|_, _| curve, core)
+                                .update_curve(|_, _| curve.clone(), core)
                                 .insert(core)
                                 .set_geometry(
-                                    *core
-                                        .layers
-                                        .geometry
-                                        .of_half_edge(&half_edge),
+                                    HalfEdgeGeom {
+                                        path: core
+                                            .layers
+                                            .geometry
+                                            .of_curve(&curve)
+                                            .local_on(&surface)
+                                            .path,
+                                        boundary,
+                                    },
                                     &mut core.layers.geometry,
                                 )
                         })