From 5734f4fae51527b238340a4360ae098e7ed24e9d Mon Sep 17 00:00:00 2001
From: Ivan Paden <i.paden@tudelft.nl>
Date: Fri, 11 Oct 2024 12:10:42 +0200
Subject: [PATCH] Update roofer with cgal60 changes

---
 .../src/reconstruction/ArrangementBase.hpp    | 22 ++++++++++++-------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/thirdparty/roofer/src/reconstruction/ArrangementBase.hpp b/thirdparty/roofer/src/reconstruction/ArrangementBase.hpp
index 42ba12b..ecaf916 100644
--- a/thirdparty/roofer/src/reconstruction/ArrangementBase.hpp
+++ b/thirdparty/roofer/src/reconstruction/ArrangementBase.hpp
@@ -82,7 +82,7 @@ typedef CGAL::Arr_face_overlay_traits<Arrangement_2,
 
 // An arrangement observer, used to receive notifications of face splits and
 // to update the indices of the newly created faces.
-class Face_index_observer : public CGAL::Arr_observer<roofer::Arrangement_2>
+class Face_index_observer : public CGAL::Arr_observer<Arrangement_2>
 {
 private:
   int     n_faces;          // The current number of faces.
@@ -91,12 +91,15 @@ class Face_index_observer : public CGAL::Arr_observer<roofer::Arrangement_2>
   float   elevation=0;
   Plane   plane;
 public:
-  Face_index_observer (roofer::Arrangement_2& arr, bool is_footprint, size_t pid, float elevation, Plane plane) :
-    CGAL::Arr_observer<roofer::Arrangement_2> (arr),
+  Face_index_observer (Arrangement_2& arr, bool is_footprint, size_t pid, float elevation, Plane plane) :
+    CGAL::Arr_observer<Arrangement_2> (arr),
     n_faces (0), in_footprint(is_footprint), plane_id(pid), elevation(elevation), plane(plane)
   {
     CGAL_precondition (arr.is_empty());
-    arr.unbounded_face()->data().is_finite=false;
+    for (auto uf = arr.unbounded_faces_begin();
+         uf != arr.unbounded_faces_end(); ++uf) {
+        uf->data().is_finite = false;
+    }
     n_faces++;
   };
   virtual void after_split_face (Face_handle old_face,
@@ -111,18 +114,21 @@ class Face_index_observer : public CGAL::Arr_observer<roofer::Arrangement_2>
     n_faces++;
   }
 };
-class Face_split_observer : public CGAL::Arr_observer<roofer::Arrangement_2>
+class Face_split_observer : public CGAL::Arr_observer<Arrangement_2>
 {
 private:
   int   n_faces;          // The current number of faces.
   bool  hole_mode=false;
 public:
-  Face_split_observer (roofer::Arrangement_2& arr) :
-    CGAL::Arr_observer<roofer::Arrangement_2> (arr),
+  Face_split_observer (Arrangement_2& arr) :
+    CGAL::Arr_observer<Arrangement_2> (arr),
     n_faces (0)
   {
     CGAL_precondition (arr.is_empty());
-    arr.unbounded_face()->data().in_footprint=false;
+    for (auto uf = arr.unbounded_faces_begin();
+         uf != arr.unbounded_faces_end(); ++uf) {
+        uf->data().is_finite = false;
+    }
     n_faces++;
   }
   virtual void after_split_face (Face_handle old_face,