cgal icon indicating copy to clipboard operation
cgal copied to clipboard

static thread_local variables in remove/move functions of `CGAL::Periodic_Delaunay_triangulation_2` (same as #9058)

Open lrineau opened this issue 4 months ago • 1 comments

Same as issue #9058, but for CGAL::Periodic_Delaunay_triangulation_2

lrineau avatar Sep 08 '25 16:09 lrineau

@MaelRL Do you think a patch similar to https://github.com/CGAL/cgal/pull/9060 could be used for PDt2?

lrineau avatar Sep 22 '25 10:09 lrineau

@MaelRL Do you think a patch similar to #9060 could be used for PDt2?

In P2DT2, we actually use the ranges for values > 7 so the same strategy cannot be re-used.

Given that there is no actual need to get rid of the static thread local variables, I'd suggest to close the issue.

MaelRL avatar Dec 11 '25 15:12 MaelRL

I see that the code is quite different, indeed. But I see ways to implement something similar to https://github.com/CGAL/cgal/issues/9058.

lrineau avatar Dec 12 '25 17:12 lrineau

I'm sure it could be implemented, but given that it was not requested and that it is P2T2 and not T2, which is not the same priority, I don't really intend to spend time on this in the future. Thus, I would close the issue.

MaelRL avatar Dec 12 '25 17:12 MaelRL