From d63f88c1fdf2efc5681cd43102450fcc81a4b23a Mon Sep 17 00:00:00 2001
From: Hanno Braun <hello@hannobraun.com>
Date: Wed, 22 May 2024 20:07:38 +0200
Subject: [PATCH] Refactor to prepare for follow-on change

---
 crates/fj-core/src/validate/shell.rs | 32 ++++++++++++++--------------
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/crates/fj-core/src/validate/shell.rs b/crates/fj-core/src/validate/shell.rs
index 508edcc0e..1fd0a801f 100644
--- a/crates/fj-core/src/validate/shell.rs
+++ b/crates/fj-core/src/validate/shell.rs
@@ -32,9 +32,12 @@ impl Validate for Shell {
         errors.extend(
             HalfEdgeHasNoSibling::check(self, geometry, config).map(Into::into),
         );
-        errors.extend(ShellValidationError::check_half_edge_coincidence(
-            self, geometry, config,
-        ));
+        errors.extend(
+            ShellValidationError::check_half_edge_coincidence(
+                self, geometry, config,
+            )
+            .map(Into::into),
+        );
     }
 }
 
@@ -52,7 +55,7 @@ impl ShellValidationError {
         shell: &Shell,
         geometry: &Geometry,
         config: &ValidationConfig,
-    ) -> impl Iterator<Item = ValidationError> {
+    ) -> impl Iterator<Item = ShellValidationError> {
         let mut errors = Vec::new();
 
         let edges_and_surfaces =
@@ -101,18 +104,15 @@ impl ShellValidationError {
                                 )
                         });
 
-                    errors.push(
-                        Self::CoincidentHalfEdgesAreNotSiblings(
-                            CoincidentHalfEdgesAreNotSiblings {
-                                boundaries,
-                                curves,
-                                vertices,
-                                half_edge_a: half_edge_a.clone(),
-                                half_edge_b: half_edge_b.clone(),
-                            },
-                        )
-                        .into(),
-                    )
+                    errors.push(Self::CoincidentHalfEdgesAreNotSiblings(
+                        CoincidentHalfEdgesAreNotSiblings {
+                            boundaries,
+                            curves,
+                            vertices,
+                            half_edge_a: half_edge_a.clone(),
+                            half_edge_b: half_edge_b.clone(),
+                        },
+                    ))
                 }
             }
         }