Remove redundant calls to insert/derive_from

Those are already called by `update_exterior`.
hannobraun · Feb 22, 2024 · 0c70cb5 · 0c70cb5
1 parent 2a055df
commit 0c70cb5
Showing 1 changed file with 28 additions and 34 deletions.
diff --git a/crates/fj-core/src/operations/split/face.rs b/crates/fj-core/src/operations/split/face.rs
@@ -127,23 +127,20 @@ impl SplitFace for Shell {
         let split_face_a = updated_face_after_split_edges
             .update_region(
                 |region, core| {
-                    region
-                        .update_exterior(
-                            |_, core| {
-                                Cycle::empty()
-                                    .add_half_edges(
-                                        half_edges_b_to_c_inclusive,
-                                        core,
-                                    )
-                                    .add_half_edges(
-                                        [dividing_half_edge_c_to_b],
-                                        core,
-                                    )
-                            },
-                            core,
-                        )
-                        .insert(core)
-                        .derive_from(region, core)
+                    region.update_exterior(
+                        |_, core| {
+                            Cycle::empty()
+                                .add_half_edges(
+                                    half_edges_b_to_c_inclusive,
+                                    core,
+                                )
+                                .add_half_edges(
+                                    [dividing_half_edge_c_to_b],
+                                    core,
+                                )
+                        },
+                        core,
+                    )
                 },
                 core,
             )
@@ -158,23 +155,20 @@ impl SplitFace for Shell {
         let split_face_b = updated_face_after_split_edges
             .update_region(
                 |region, core| {
-                    region
-                        .update_exterior(
-                            |_, core| {
-                                Cycle::empty()
-                                    .add_half_edges(
-                                        half_edges_d_to_a_inclusive,
-                                        core,
-                                    )
-                                    .add_half_edges(
-                                        [dividing_half_edge_a_to_d],
-                                        core,
-                                    )
-                            },
-                            core,
-                        )
-                        .insert(core)
-                        .derive_from(region, core)
+                    region.update_exterior(
+                        |_, core| {
+                            Cycle::empty()
+                                .add_half_edges(
+                                    half_edges_d_to_a_inclusive,
+                                    core,
+                                )
+                                .add_half_edges(
+                                    [dividing_half_edge_a_to_d],
+                                    core,
+                                )
+                        },
+                        core,
+                    )
                 },
                 core,
             )