Remove redundant function

Author: hannobraun

experiments/2024-12-09/src/operations/connect.rs

@@ -27,8 +27,8 @@ pub trait ConnectExt {
 impl ConnectExt for Handle<Face> {
     fn connect(self, other: Self) -> Solid {
         assert_eq!(
-            self.half_edges().count(),
-            other.half_edges().count(),
+            self.half_edges.len(),
+            other.half_edges.len(),
             "Can only connect faces that have the same number of vertices.",
         );
 

experiments/2024-12-09/src/operations/flip.rs

@@ -6,6 +6,6 @@ pub trait FlipExt {
 
 impl FlipExt for &Face {
     fn flip(self) -> Face {
-        Face::new(self.surface.flip(), self.half_edges().cloned())
+        Face::new(self.surface.flip(), self.half_edges.clone())
     }
 }

experiments/2024-12-09/src/operations/translate.rs

@@ -14,7 +14,8 @@ impl TranslateExt for Face {
 
         Face::new(
             self.surface.translate(offset),
-            self.half_edges()
+            self.half_edges
+                .iter()
                 .map(|half_edge| Handle::new(half_edge.translate(offset))),
         )
     }

experiments/2024-12-09/src/topology/face.rs

@@ -28,10 +28,6 @@ impl Face {
         }
     }
 
-    pub fn half_edges(&self) -> impl Iterator<Item = &Handle<HalfEdge>> {
-        self.half_edges.iter()
-    }
-
     pub fn start_and_end_vertices(
         &self,
     ) -> impl Iterator<Item = [&Handle<Vertex>; 2]> {
@@ -52,6 +48,9 @@ impl Object for Face {
     }
 
     fn children(&self) -> Vec<HandleAny> {
-        self.half_edges().map(|vertex| vertex.to_any()).collect()
+        self.half_edges
+            .iter()
+            .map(|vertex| vertex.to_any())
+            .collect()
     }
 }