Andreas Fabri

Results 479 comments of Andreas Fabri

I made it of size 2 * number of elements to store in case that each leaf node stores only one element, in order to be sure that no reallocation...

> To ensure that each leaf contains at least two vertices. So changing this back or changing the `2 *` might be worth investigation? I missed that. You are right...

I can reproduce the problem and investigate now.

This one "CGAL/Arr_geometry_traits/Conic_x_monotone_arc_2.h:205:42: warning: bitwise operation between different enumeration types" is in Qt, not in CGAL.

So let me apply a `static_cast` before or-ing.

What is missing: A line in the Reference Manual [index page](https://github.com/CGAL/cgal/blob/master/Polygon_mesh_processing/doc/Polygon_mesh_processing/PackageDescription.txt#L222) A line in the [change log](https://github.com/CGAL/cgal/blob/master/Installation/CHANGES.md) A test program.

But what would be the better choice? Also not only that the test thinks it, but it is only one platform where this triggers an assertion.

You must first triangulate the quad mesh.

Sorry, but I do not understand. I had suggested to call [`triangulate_faces()`](https://doc.cgal.org/latest/Polygon_mesh_processing/group__PMP__meshing__grp.html#ga1e9c8778755b2d1741f47736ba6e4b45).

When we change the concept we have to rename or scope it. Is it possible to have a concept in a namespace?