From 6191f7de3c2520135ee7f411f5e084b0b397718f Mon Sep 17 00:00:00 2001
From: Jane Tournois <janetournois@users.noreply.github.com>
Date: Tue, 13 Feb 2024 10:31:26 +0100
Subject: [PATCH] use split_face() instead of triangulate_faces()

and remove dependency on T2
---
 .../include/CGAL/Polygon_mesh_processing/bbox.h     | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/bbox.h b/Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/bbox.h
index 813c6a3ef8e..aa80a4b4f0c 100644
--- a/Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/bbox.h
+++ b/Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/bbox.h
@@ -19,7 +19,7 @@
 
 #include <CGAL/boost/graph/generators.h>
 #include <CGAL/boost/graph/copy_face_graph.h>
-#include <CGAL/Polygon_mesh_processing/triangulate_faces.h>
+#include <CGAL/boost/graph/Euler_operations.h>
 
 #include <boost/graph/graph_traits.hpp>
 #include <CGAL/Named_function_parameters.h>
@@ -325,8 +325,15 @@ namespace CGAL {
                             bbox_mesh);
 
       if(!dont_triangulate)
-        CGAL::Polygon_mesh_processing::triangulate_faces(bbox_mesh);
-
+      {
+        for (auto h : halfedges(bbox_mesh))
+        {
+          if (is_quad(h, bbox_mesh))
+            CGAL::Euler::split_face(h, next(next(h, bbox_mesh), bbox_mesh), bbox_mesh);
+          else
+            CGAL_assertion(is_triangle(h, bbox_mesh));
+        }
+      }
       CGAL::copy_face_graph(bbox_mesh, pmesh,
                             parameters::default_values(),
                             np);