fix(Error Message): add component name in error message

Author: francoisbonneau

src/geode/mesh/core/solid_mesh.cpp

@@ -193,7 +193,8 @@ namespace
         } while( facet.polyhedron_id != first_polyhedron
                  && safety_count < MAX_SAFETY_COUNT );
         OPENGEODE_EXCEPTION( safety_count < MAX_SAFETY_COUNT,
-            "[SolidMesh::propagate_around_edge] Too many polyhedra "
+            "[SolidMesh::propagate_around_edge] Solid: ", solid.name(),
+            " - Too many polyhedra "
             "around edge ",
             edge_vertices[0], " ", edge_vertices[1], " (",
             solid.point( edge_vertices[0] ).string(), " ",
@@ -298,7 +299,9 @@ namespace
             }
         }
         OPENGEODE_EXCEPTION( safety_count < MAX_SAFETY_COUNT,
-            "[SolidMesh::compute_polyhedra_around_vertex] Too many polyhedra "
+            "[SolidMesh::compute_polyhedra_around_vertex] Solid: ",
+            solid.name(),
+            " Too many polyhedra "
             "around vertex ",
             vertex_id, " (", solid.point( vertex_id ).string(),
             "). This is probably related to a bug in the polyhedra "
@@ -424,12 +427,14 @@ namespace geode
 
     public:
         explicit Impl( SolidMesh& solid )
-            : polyhedron_around_vertex_( solid.vertex_attribute_manager()
+            : polyhedron_around_vertex_(
+                  solid.vertex_attribute_manager()
                       .template find_or_create_attribute< VariableAttribute,
                           PolyhedronVertex >( "polyhedron_around_vertex",
                           PolyhedronVertex{},
                           { false, false, false } ) ),
-              polyhedra_around_vertex_( solid.vertex_attribute_manager()
+              polyhedra_around_vertex_(
+                  solid.vertex_attribute_manager()
                       .template find_or_create_attribute< VariableAttribute,
                           CachedPolyhedra >( POLYHEDRA_AROUND_VERTEX_NAME,
                           CachedPolyhedra{},
@@ -1335,7 +1340,7 @@ namespace geode
             {
                 facet_vertices.push_back(
                     vertices[polyhedron_facet_vertex_id( { facet, v } )
-                            .vertex_id] );
+                                 .vertex_id] );
             }
         }
         return facets_vertices;

src/geode/mesh/core/surface_mesh.cpp

@@ -181,7 +181,8 @@ namespace
             }
         }
         OPENGEODE_EXCEPTION( safety_count < MAX_SAFETY_COUNT,
-            "[SurfaceMesh::polygons_around_vertex] Too many polygons "
+            "[SurfaceMesh::polygons_around_vertex] Surface: ", mesh.name(),
+            " Too many polygons "
             "around vertex ",
             vertex_id, " (", mesh.point( vertex_id ).string(),
             "). This is probably related to a bug in the polygon "
@@ -249,12 +250,14 @@ namespace geode
 
     public:
         Impl( SurfaceMesh& surface )
-            : polygon_around_vertex_( surface.vertex_attribute_manager()
+            : polygon_around_vertex_(
+                  surface.vertex_attribute_manager()
                       .template find_or_create_attribute< VariableAttribute,
                           PolygonVertex >( "polygon_around_vertex",
                           PolygonVertex{},
                           { false, false, false } ) ),
-              polygons_around_vertex_( surface.vertex_attribute_manager()
+              polygons_around_vertex_(
+                  surface.vertex_attribute_manager()
                       .template find_or_create_attribute< VariableAttribute,
                           CachedPolygons >( POLYGONS_AROUND_VERTEX_NAME,
                           CachedPolygons{},

src/geode/mesh/helpers/repair_polygon_orientations.cpp

@@ -51,16 +51,29 @@ namespace
 
         absl::FixedArray< geode::index_t > compute_bad_oriented_polygons()
         {
-            absl::FixedArray< bool > visited( mesh_.nb_polygons(), false );
-            for( const auto p : geode::Range{ mesh_.nb_polygons() } )
+            try
             {
-                if( visited[p] )
+                absl::FixedArray< bool > visited( mesh_.nb_polygons(), false );
+                for( const auto p : geode::Range{ mesh_.nb_polygons() } )
                 {
-                    continue;
+                    if( visited[p] )
+                    {
+                        continue;
+                    }
+                    queue_.emplace( p );
+                    visited[p] = true;
+                    process_polygon_queue( visited );
                 }
-                queue_.emplace( p );
-                visited[p] = true;
-                process_polygon_queue( visited );
+            }
+            catch( geode::OpenGeodeException& e )
+            {
+                const auto msg =
+                    absl::StrCat( "Surface ", mesh_.name(), " - ", e.what() );
+                throw( geode::OpenGeodeException( msg ) );
+            }
+            catch( ... )
+            {
+                throw;
             }
             return get_bad_oriented_polygons();
         }