Using Side_of_triangle_mesh in Mesh_3

CGAL / cgal

The public CGAL repository, see the README below

https://github.com/CGAL/cgal#readme

Other

4.87k stars 1.38k forks source link

Using Side_of_triangle_mesh in Mesh_3 #2084

Closed sloriot closed 7 years ago

sloriot commented 7 years ago

Mesh_3 Polyhedral_mesh_domain_3 should be using Side_of_triangle_mesh as an implementation detail. TriangleAccessor_3 concept should also be replaced by FaceListGraph. If we want to avoid a dependency between packages, we could leave Polyhedral_mesh_domain_3 as his, (deprecated it?) and introduced a new class working on models of FaceListGraph. This new class can be put in PMP.

lrineau commented 7 years ago

I agree with @sloriot. I wonder if that would conflict with the work done in #1983, that makes Mesh_3 compatible with CGAL::Surface_mesh. Cc @afabri

janetournois commented 7 years ago

I'm working on it