From dd4722caa75ab21d8cb15f753fb53149b8d1d0ed Mon Sep 17 00:00:00 2001
From: Jane Tournois <janetournois@users.noreply.github.com>
Date: Mon, 23 Jan 2023 21:49:17 +0100
Subject: [PATCH] mesh domain is now const

and cleaning (add_input_features has become useless)
---
 Mesh_3/include/CGAL/Labeled_mesh_domain_3.h | 30 +++------------------
 1 file changed, 3 insertions(+), 27 deletions(-)

diff --git a/Mesh_3/include/CGAL/Labeled_mesh_domain_3.h b/Mesh_3/include/CGAL/Labeled_mesh_domain_3.h
index 4e8200f5186..8d66e559c3a 100644
--- a/Mesh_3/include/CGAL/Labeled_mesh_domain_3.h
+++ b/Mesh_3/include/CGAL/Labeled_mesh_domain_3.h
@@ -128,7 +128,7 @@ namespace internal {
   struct Detect_features_in_domain {
     std::vector<std::vector<typename MeshDomain::Point_3>>
     operator()(const CGAL::Image_3& image,
-               MeshDomain& domain,
+               const MeshDomain& domain,
                DetectFunctor functor) const {
       return functor(image, domain);
     }
@@ -138,7 +138,7 @@ namespace internal {
   struct Detect_features_in_domain<MeshDomain, Null_functor> {
     std::vector<std::vector<typename MeshDomain::Point_3>>
     operator()(const CGAL::Image_3&,
-               MeshDomain&,
+               const MeshDomain&,
                Null_functor) const {
       return std::vector<std::vector<typename MeshDomain::Point_3>>();
     }
@@ -147,37 +147,13 @@ namespace internal {
   template<typename MeshDomain, typename DetectFunctor>
   std::vector<std::vector<typename MeshDomain::Point_3>>
   detect_features(const CGAL::Image_3& image,
-                  MeshDomain& domain,
+                  const MeshDomain& domain,
                   DetectFunctor functor)
   {
     return Detect_features_in_domain<MeshDomain, DetectFunctor>()
             (image, domain, functor);
   }
 
-  // Add_input_features
-  template<typename MeshDomain, typename DetectFunctor>
-  struct Add_input_features_in_domain {
-    std::vector<std::vector<typename MeshDomain::Point_3>>
-    operator()(MeshDomain& domain, DetectFunctor functor) const {
-       return functor(domain);
-    }
-  };
-  // specialization for `Null_functor`: create the default functor
-  template<typename MeshDomain>
-  struct Add_input_features_in_domain<MeshDomain, Null_functor> {
-    std::vector<std::vector<typename MeshDomain::Point_3>>
-    operator()(MeshDomain&, Null_functor) const {
-      return std::vector<std::vector<typename MeshDomain::Point_3>>();
-    }
-  };
-
-  template<typename MeshDomain, typename DetectFunctor>
-  std::vector<std::vector<typename MeshDomain::Point_3>>
-  add_input_features(MeshDomain& domain, DetectFunctor functor)
-  {
-    return Add_input_features_in_domain<MeshDomain, DetectFunctor>()(domain, functor);
-  }
-
   template<bool WithFeatures>
   struct Add_features_in_domain {
     template<typename MeshDomain, typename InputFeatureRange, typename DetectFunctor>