no need for is_valid as there is no foldover like on the boundary

sloriot · sloriot · commit d7c4b64762b0 · 2026-03-25T16:10:56.000+01:00
diff --git a/PMP_Remeshing/include/CGAL/Polygon_mesh_processing/triangulate_faces.h b/PMP_Remeshing/include/CGAL/Polygon_mesh_processing/triangulate_faces.h
@@ -235,7 +235,7 @@ class Triangulate_polygon_mesh_modifier
     if(border_vertices.size() <= 3)
       return true;
 
-    if(border_vertices.size() == 4) // TODO: also use is_valid
+    if(border_vertices.size() == 4)
     {
       std::array<halfedge_descriptor,4> hverts;
       hverts[0] = halfedge(f, pmesh);

Original file line number	Diff line number	Diff line change
`@@ -235,7 +235,7 @@ class Triangulate_polygon_mesh_modifier`
`235`	`235`	`if(border_vertices.size() <= 3)`
`236`	`236`	`return true;`
`237`	`237`
`238`		`- if(border_vertices.size() == 4) // TODO: also use is_valid`
	`238`	`+ if(border_vertices.size() == 4)`
`239`	`239`	`{`
`240`	`240`	`std::array<halfedge_descriptor,4> hverts;`
`241`	`241`	`hverts[0] = halfedge(f, pmesh);`