From a52afd88003de8424642eca8d3790bd97bb3376f Mon Sep 17 00:00:00 2001 From: Jane Tournois Date: Mon, 14 Apr 2025 11:30:44 +0200 Subject: [PATCH] setting status of edges that will disappear is cheaper than checking status --- .../internal/Isotropic_remeshing/remesh_impl.h | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/internal/Isotropic_remeshing/remesh_impl.h b/Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/internal/Isotropic_remeshing/remesh_impl.h index 5ec16e98fa7..776f5654879 100644 --- a/Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/internal/Isotropic_remeshing/remesh_impl.h +++ b/Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/internal/Isotropic_remeshing/remesh_impl.h @@ -1699,10 +1699,9 @@ private: { set_status(hf, MESH_BORDER); //only 1 or 2 of the listed halfedges //will survive face removal, but status will be correct - - halfedge_descriptor hfo = opposite(hf, mesh_); - if(status(hfo) != MESH_BORDER) - set_status(hfo, PATCH_BORDER); + set_status(opposite(hf, mesh_), PATCH_BORDER); //idem + //some of them will not survive but setting status + //is cheaper then checking which should be set } CGAL::Euler::remove_face(h, mesh_); }