Merge pull request #4792 from lrineau/Mesh_3-reduce_size_of_debuginfo-GF

Reduce size of symbols (and debug infos)

Mesh_3/include/CGAL/Compact_mesh_cell_base_3.h

@@ -98,12 +98,9 @@ namespace CGAL {
 // Base for Compact_mesh_cell_base_3, with specializations
 // for different values of Concurrency_tag
 // Sequential
-template <typename GT, typename Concurrency_tag>
+template <typename Point_3, typename Concurrency_tag>
 class Compact_mesh_cell_base_3_base
 {
-  typedef typename GT::Point_3                Point_3;
-  typedef typename GT::Weighted_point_3       Point;
-
 protected:
   Compact_mesh_cell_base_3_base()
     : bits_(0)
@@ -176,11 +173,9 @@ protected:
 #ifdef CGAL_LINKED_WITH_TBB
 // Class Compact_mesh_cell_base_3_base
 // Specialization for parallel
-template <typename GT>
+template <typename Point_3>
-class Compact_mesh_cell_base_3_base<GT, Parallel_tag>
+class Compact_mesh_cell_base_3_base<Point_3, Parallel_tag>
 {
-  typedef typename GT::Point_3  Point_3;
-
 protected:
   Compact_mesh_cell_base_3_base()
   {
@@ -260,14 +255,16 @@ protected:
 // Class Compact_mesh_cell_base_3
 // Cell base class used in 3D meshing process.
 // Adds information to Cb about the cell of the input complex containing it
-template< class GT,
+template< class Point_3,
-          class MD,
+          class Weighted_point_3,
-          class TDS = void >
+          class Subdomain_index_,
-class Compact_mesh_cell_base_3
+          class Surface_patch_index_,
-  : public Compact_mesh_cell_base_3_base<GT, typename TDS::Concurrency_tag>
+          class Index_,
+          class TDS>
+class Compact_mesh_cell_3
+  : public Compact_mesh_cell_base_3_base<Point_3, typename TDS::Concurrency_tag>
 {
-  typedef typename GT::FT FT;
+  typedef Compact_mesh_cell_base_3_base<Point_3,typename TDS::Concurrency_tag> Base;
-  typedef Compact_mesh_cell_base_3_base<GT,typename TDS::Concurrency_tag> Base;
   using Base::weighted_circumcenter_;
 
 public:
@@ -279,20 +276,12 @@ public:
   typedef typename TDS::Cell_data      TDS_data;
 
 
-  template <typename TDS2>
-  struct Rebind_TDS {
-    typedef Compact_mesh_cell_base_3<GT, MD, TDS2> Other;
-  };
-
-
   // Index Type
-  typedef typename MD::Subdomain_index      Subdomain_index;
+  typedef Subdomain_index_      Subdomain_index;
-  typedef typename MD::Surface_patch_index  Surface_patch_index;
+  typedef Surface_patch_index_  Surface_patch_index;
-  typedef typename MD::Index                Index;
+  typedef Index_                Index;
 
-  typedef GT                                Geom_traits;
+  typedef Weighted_point_3      Point;
-  typedef typename GT::Point_3              Point_3;
-  typedef typename GT::Weighted_point_3     Point;
 
 
   typedef Point*                            Point_container;
@@ -310,20 +299,10 @@ public:
 
 public:
   // Constructors
-  Compact_mesh_cell_base_3()
+  Compact_mesh_cell_3()
-    : surface_index_table_()
-    , surface_center_table_()
-#ifdef CGAL_INTRUSIVE_LIST
-    , next_intrusive_()
-    , previous_intrusive_()
-#endif
-    , surface_center_index_table_()
-    , sliver_value_(FT(0.))
-    , subdomain_index_()
-    , sliver_cache_validity_(false)
   {}
 
-  Compact_mesh_cell_base_3(const Compact_mesh_cell_base_3& rhs)
+  Compact_mesh_cell_3(const Compact_mesh_cell_3& rhs)
     : N(rhs.N)
     , V(rhs.V)
 #ifdef CGAL_INTRUSIVE_LIST
@@ -342,26 +321,16 @@ public:
     }
   }
 
-  Compact_mesh_cell_base_3 (Vertex_handle v0,
+  Compact_mesh_cell_3 (Vertex_handle v0,
                        Vertex_handle v1,
                        Vertex_handle v2,
                        Vertex_handle v3)
-    : surface_index_table_()
+    : V(CGAL::make_array(v0, v1, v2, v3))
-    , surface_center_table_()
-    , V(CGAL::make_array(v0, v1, v2, v3))
-#ifdef CGAL_INTRUSIVE_LIST
-    , next_intrusive_()
-    , previous_intrusive_()
-#endif
-    , surface_center_index_table_()
-    , sliver_value_(FT(0.))
-    , subdomain_index_()
-    , sliver_cache_validity_(false)
   {
   }
 
 
-  Compact_mesh_cell_base_3 (Vertex_handle v0,
+  Compact_mesh_cell_3 (Vertex_handle v0,
                        Vertex_handle v1,
                        Vertex_handle v2,
                        Vertex_handle v3,
@@ -369,22 +338,12 @@ public:
                        Cell_handle n1,
                        Cell_handle n2,
                        Cell_handle n3)
-    : surface_index_table_()
+    : N(CGAL::make_array(n0, n1, n2, n3))
-    , surface_center_table_()
-    , N(CGAL::make_array(n0, n1, n2, n3))
     , V(CGAL::make_array(v0, v1, v2, v3))
-#ifdef CGAL_INTRUSIVE_LIST
-    , next_intrusive_()
-    , previous_intrusive_()
-#endif
-    , surface_center_index_table_()
-    , sliver_value_(FT(0.))
-    , subdomain_index_()
-    , sliver_cache_validity_(false)
   {
   }
 
-  ~Compact_mesh_cell_base_3()
+  ~Compact_mesh_cell_3()
   {
     if(!internal_tbb::is_null(weighted_circumcenter_)){
       internal_tbb::delete_circumcenter(weighted_circumcenter_);
@@ -550,11 +509,6 @@ public:
     return *weighted_circumcenter_;
   }
 
-  const Point_3& weighted_circumcenter() const
-  {
-    return weighted_circumcenter(Geom_traits());
-  }
-
   // Returns the index of the cell of the input complex that contains the cell
   Subdomain_index subdomain_index() const { return subdomain_index_; }
 
@@ -562,13 +516,13 @@ public:
   void set_subdomain_index(const Subdomain_index& index)
   { subdomain_index_ = index; }
 
-  void set_sliver_value(const FT& value)
+  void set_sliver_value(double value)
   {
     sliver_cache_validity_ = true;
     sliver_value_ = value;
   }
 
-  const FT& sliver_value() const
+  double sliver_value() const
   {
     CGAL_assertion(is_cache_valid());
     return sliver_value_;
@@ -646,6 +600,7 @@ public:
   static
   std::string io_signature()
   {
+    using Geom_traits = typename Kernel_traits<Point>::type;
     return
       Get_io_signature<Subdomain_index>()() + "+" +
       Get_io_signature<Regular_triangulation_cell_base_3<Geom_traits> >()()
@@ -683,41 +638,37 @@ private:
 
 
   /// Stores surface_index for each facet of the cell
-  std::array<Surface_patch_index, 4> surface_index_table_;
+  std::array<Surface_patch_index, 4> surface_index_table_ = {};
   /// Stores surface center of each facet of the cell
-  std::array<Point_3, 4> surface_center_table_;
+  std::array<Point_3, 4> surface_center_table_ = {};
   /// Stores surface center index of each facet of the cell
 
   std::array<Cell_handle, 4> N;
   std::array<Vertex_handle, 4> V;
 
 #ifdef CGAL_INTRUSIVE_LIST
-  Cell_handle next_intrusive_, previous_intrusive_;
+  Cell_handle next_intrusive_ = {}, previous_intrusive_ = {};
 #endif
   std::size_t time_stamp_;
 
-  std::array<Index, 4> surface_center_index_table_;
+  std::array<Index, 4> surface_center_index_table_ = {};
   /// Stores visited facets (4 first bits)
 
   //  Point_container _hidden;
 
-  FT sliver_value_;
+  double sliver_value_ = 0.;
 
   // The index of the cell of the input complex that contains me
-  Subdomain_index subdomain_index_;
+  Subdomain_index subdomain_index_ = {};
 
   TDS_data      _tds_data;
-  mutable bool sliver_cache_validity_;
+  mutable bool sliver_cache_validity_ = false;
 
+public:
 
-};  // end class Compact_mesh_cell_base_3
+  friend std::istream& operator>>(std::istream &is, Compact_mesh_cell_3 &c)
-
-template < class GT, class MT, class Cb >
-std::istream&
-operator>>(std::istream &is,
-           Compact_mesh_cell_base_3<GT, MT, Cb> &c)
   {
-  typename Compact_mesh_cell_base_3<GT, MT, Cb>::Subdomain_index index;
+    Subdomain_index index;
     if(is_ascii(is))
       is >> index;
     else
@@ -726,7 +677,7 @@ operator>>(std::istream &is,
       c.set_subdomain_index(index);
       for(int i = 0; i < 4; ++i)
         {
-      typename Compact_mesh_cell_base_3<GT, MT, Cb>::Surface_patch_index i2;
+          Surface_patch_index i2;
           if(is_ascii(is))
             is >> iformat(i2);
           else
@@ -739,10 +690,8 @@ operator>>(std::istream &is,
     return is;
   }
 
-template < class GT, class MT, class Cb >
+  friend
-std::ostream&
+  std::ostream& operator<<(std::ostream &os, const Compact_mesh_cell_3 &c)
-operator<<(std::ostream &os,
-           const Compact_mesh_cell_base_3<GT, MT, Cb> &c)
   {
     if(is_ascii(os))
       os << c.subdomain_index();
@@ -758,6 +707,12 @@ operator<<(std::ostream &os,
     return os;
   }
 
+};  // end class Compact_mesh_cell_3
+
+template< class GT,
+          class MD,
+          class TDS = void >
+class Compact_mesh_cell_base_3;
 
 // Specialization for void.
 template <typename GT, typename MD>
@@ -768,7 +723,35 @@ public:
   typedef Triangulation_data_structure::Vertex_handle   Vertex_handle;
   typedef Triangulation_data_structure::Cell_handle     Cell_handle;
   template <typename TDS2>
-  struct Rebind_TDS { typedef Compact_mesh_cell_base_3<GT, MD, TDS2> Other; };
+  struct Rebind_TDS {
+    typedef Compact_mesh_cell_3<typename GT::Point_3,
+                                typename GT::Weighted_point_3,
+                                typename MD::Subdomain_index,
+                                typename MD::Surface_patch_index,
+                                typename MD::Index,
+                                TDS2> Other;
+  };
+};
+
+template <typename GT,
+  typename Subdomain_index,
+  typename Surface_patch_index,
+  typename Index>
+class Compact_mesh_cell_generator_3
+{
+public:
+  typedef internal::Dummy_tds_3                         Triangulation_data_structure;
+  typedef Triangulation_data_structure::Vertex_handle   Vertex_handle;
+  typedef Triangulation_data_structure::Cell_handle     Cell_handle;
+  template <typename TDS2>
+  struct Rebind_TDS {
+    typedef Compact_mesh_cell_3<typename GT::Point_3,
+                                typename GT::Weighted_point_3,
+                                Subdomain_index,
+                                Surface_patch_index,
+                                Index,
+                                TDS2> Other;
+  };
 };
 
 }  // end namespace CGAL

Mesh_3/include/CGAL/Mesh_3/io_signature.h

@@ -219,12 +219,10 @@ struct Get_io_signature<Weighted_point_3<K> >
 };
 
 #ifdef CGAL_TRIANGULATION_3_H
-template <class Gt, class Vb, class Cb, class C_tag>
+template <class Gt, class Tds>
 struct
-Get_io_signature<Triangulation_3<Gt, Triangulation_data_structure_3<Vb, Cb, C_tag> > >
+Get_io_signature<Triangulation_3<Gt, Tds > >
 {
-  typedef Triangulation_data_structure_3<Vb, Cb, C_tag> Tds;
-
   std::string operator()() {
     return std::string("Triangulation_3(") +
       Get_io_signature<typename Tds::Vertex::Point>()() +

Mesh_3/include/CGAL/Mesh_triangulation_3.h

@@ -120,65 +120,40 @@ public:
 //
 template<class MD,
          class K_ = Default,
-         class Concurrency_tag = Sequential_tag,
+         class Concurrency_tag_ = Sequential_tag,
          class Vertex_base_ = Default,
          class Cell_base_   = Default>
-struct Mesh_triangulation_3;
-
-// Sequential version (default)
-template<class MD, class K_, class Concurrency_tag,
-         class Vertex_base_, class Cell_base_>
 struct Mesh_triangulation_3
 {
 private:
-  typedef typename Default::Lazy_get<K_, Kernel_traits<MD> >::type K;
+  using K = typename Default::Lazy_get<K_, Kernel_traits<MD> >::type;
 
-  typedef typename details::Mesh_geom_traits_generator<K>::type    Geom_traits;
+  using Geom_traits = typename details::Mesh_geom_traits_generator<K>::type;
 
-  typedef typename Default::Get<
+  using Indices_tuple = Mesh_3::internal::Indices_tuple_t<MD>;
+  using Vertex_base = typename Default::Get<
     Vertex_base_,
-    Mesh_vertex_base_3<Geom_traits, MD> >::type                    Vertex_base;
+    Mesh_vertex_generator_3<Geom_traits,
-  typedef typename Default::Get<
+                            Indices_tuple,
+                            typename MD::Index> >::type;
+  using Cell_base = typename Default::Get<
     Cell_base_,
-    Compact_mesh_cell_base_3<Geom_traits, MD> >::type              Cell_base;
+    Compact_mesh_cell_generator_3<Geom_traits,
-
+                                  typename MD::Subdomain_index,
-  typedef Triangulation_data_structure_3<Vertex_base,Cell_base>    Tds;
+                                  typename MD::Surface_patch_index,
-  typedef Mesh_3_regular_triangulation_3_wrapper<Geom_traits, Tds> Triangulation;
+                                  typename MD::Index> >::type;
+  using Concurrency_tag =
+      typename Default::Get<Concurrency_tag_, Sequential_tag>::type;
+  struct Tds : public Triangulation_data_structure_3<Vertex_base, Cell_base,
+                                                     Concurrency_tag> {};
+  using Triangulation =
+      Mesh_3_regular_triangulation_3_wrapper<Geom_traits, Tds>;
+  ;
 
 public:
-  typedef Triangulation type;
+  using type = Triangulation;
-  typedef type Type;
+  using Type = type;
 };  // end struct Mesh_triangulation_3
-
-#ifdef CGAL_LINKED_WITH_TBB
-// Parallel version (specialization)
-//
-template<class MD, class K_,
-         class Vertex_base_, class Cell_base_>
-struct Mesh_triangulation_3<MD, K_, Parallel_tag, Vertex_base_, Cell_base_>
-{
-private:
-  typedef typename Default::Get<K_, typename Kernel_traits<MD>::Kernel>::type K;
-
-  typedef typename details::Mesh_geom_traits_generator<K>::type       Geom_traits;
-
-  typedef typename Default::Get<
-    Vertex_base_,
-    Mesh_vertex_base_3<Geom_traits, MD> >::type                       Vertex_base;
-  typedef typename Default::Get<
-    Cell_base_,
-    Compact_mesh_cell_base_3<Geom_traits, MD> >::type                 Cell_base;
-
-  typedef Triangulation_data_structure_3<
-    Vertex_base, Cell_base, Parallel_tag>                             Tds;
-  typedef Mesh_3_regular_triangulation_3_wrapper<Geom_traits, Tds>    Triangulation;
-
-public:
-  typedef Triangulation type;
-  typedef type Type;
-};  // end struct Mesh_triangulation_3
-#endif // CGAL_LINKED_WITH_TBB
-
 }  // end namespace CGAL
 
 #include <CGAL/enable_warnings.h>

Mesh_3/include/CGAL/Mesh_vertex_base_3.h

@@ -24,7 +24,7 @@
 
 
 #include <CGAL/Regular_triangulation_vertex_base_3.h>
-#include <CGAL/internal/Mesh_3/get_index.h>
+#include <CGAL/internal/Mesh_3/indices_management.h>
 #include <CGAL/Mesh_3/io_signature.h>
 #include <CGAL/Has_timestamp.h>
 #include <CGAL/tags.h>
@@ -92,9 +92,10 @@ protected:
 // Adds information to Vb about the localization of the vertex in regards
 // to the 3D input complex.
 template<class GT,
-         class MD,
+         class Indices_tuple,
-         class Vb = Regular_triangulation_vertex_base_3<GT> >
+         class Index_,
-class Mesh_vertex_base_3
+         class Vb>
+class Mesh_vertex_3
 : public Vb,
   public Mesh_vertex_base_3_base<
     typename Vb::Triangulation_data_structure::Concurrency_tag>
@@ -103,19 +104,12 @@ public:
   typedef Vb Cmvb3_base;
   typedef typename Vb::Vertex_handle  Vertex_handle;
 
-  // To get correct vertex type in TDS
-  template < class TDS3 >
-  struct Rebind_TDS {
-    typedef typename Vb::template Rebind_TDS<TDS3>::Other Vb3;
-    typedef Mesh_vertex_base_3 <GT, MD, Vb3> Other;
-  };
-
   // Types
-  typedef typename MD::Index                      Index;
+  typedef Index_                      Index;
   typedef typename GT::FT             FT;
 
   // Constructor
-  Mesh_vertex_base_3()
+  Mesh_vertex_3()
     : Vb()
     , number_of_incident_facets_(0)
     , number_of_components_(0)
@@ -248,18 +242,10 @@ private:
   Vertex_handle previous_intrusive_;
 #endif
   std::size_t time_stamp_;
+public:
 
-};  // end class Mesh_vertex_base_3
+  friend std::istream& operator>>(std::istream &is, Mesh_vertex_3& v)
-
-template<class GT,
-         class MD,
-         class Vb>
-inline
-std::istream&
-operator>>(std::istream &is, Mesh_vertex_base_3<GT,MD,Vb>& v)
   {
-  typedef Mesh_vertex_base_3<GT,MD,Vb> Vertex;
-  typedef typename Vertex::Cmvb3_base Cmvb3_base;
     is >> static_cast<Cmvb3_base&>(v);
     int dimension;
     if(is_ascii(is)) {
@@ -271,21 +257,15 @@ operator>>(std::istream &is, Mesh_vertex_base_3<GT,MD,Vb>& v)
     v.set_dimension(dimension);
     CGAL_assertion(v.in_dimension() >= -1);
     CGAL_assertion(v.in_dimension() < 4);
-  typename Vertex::Index index =
+    Index index =
-    Mesh_3::internal::Read_mesh_domain_index<MD>()(v.in_dimension(), is);
+      Mesh_3::internal::Read_write_index<Indices_tuple,
+                                         Index>()(is, v.in_dimension());
     v.set_index(index);
     return is;
   }
 
-template<class GT,
+  friend std::ostream& operator<<(std::ostream &os, const Mesh_vertex_3& v)
-         class MD,
-         class Vb>
-inline
-std::ostream&
-operator<<(std::ostream &os, const Mesh_vertex_base_3<GT,MD,Vb>& v)
   {
-  typedef Mesh_vertex_base_3<GT,MD,Vb> Vertex;
-  typedef typename Vertex::Cmvb3_base Cmvb3_base;
     os << static_cast<const Cmvb3_base&>(v);
     if(is_ascii(os)) {
       os << " " << v.in_dimension()
@@ -293,11 +273,46 @@ operator<<(std::ostream &os, const Mesh_vertex_base_3<GT,MD,Vb>& v)
     } else {
       CGAL::write(os, v.in_dimension());
     }
-  Mesh_3::internal::Write_mesh_domain_index<MD>()(os,
+    Mesh_3::internal::Read_write_index<Indices_tuple,
+                                       Index>()(os,
                                                 v.in_dimension(),
                                                 v.index());
     return os;
   }
+};  // end class Mesh_vertex_3
+
+template<class GT,
+         class MD,
+         class Vb = Regular_triangulation_vertex_base_3<GT> >
+struct Mesh_vertex_base_3 {
+  using Triangulation_data_structure = internal::Dummy_tds_3;
+  using Vertex_handle = typename Triangulation_data_structure::Vertex_handle;
+  using Cell_handle = typename Triangulation_data_structure::Cell_handle;
+
+  template < class TDS3 >
+  struct Rebind_TDS {
+    using Vb3 = typename Vb::template Rebind_TDS<TDS3>::Other;
+    using Other = Mesh_vertex_3 <GT,
+                                 Mesh_3::internal::Indices_tuple_t<MD>,
+                                 typename MD::Index, Vb3>;
+  };
+};
+
+template<class GT,
+         class Indices_tuple,
+         class Index,
+         class Vb = Regular_triangulation_vertex_base_3<GT> >
+struct Mesh_vertex_generator_3 {
+  using Triangulation_data_structure = internal::Dummy_tds_3;
+  using Vertex_handle = typename Triangulation_data_structure::Vertex_handle;
+  using Cell_handle = typename Triangulation_data_structure::Cell_handle;
+
+  template < class TDS3 >
+  struct Rebind_TDS {
+    typedef typename Vb::template Rebind_TDS<TDS3>::Other Vb3;
+    typedef Mesh_vertex_3 <GT, Indices_tuple, Index, Vb3> Other;
+  };
+};
 
 }  // end namespace CGAL
 

Mesh_3/include/CGAL/internal/Mesh_3/indices_management.h

@@ -23,10 +23,14 @@
 
 #include <boost/type_traits/is_same.hpp>
 #include <boost/utility/enable_if.hpp>
-#include <boost/variant.hpp>
+#include <boost/variant/variant.hpp>
+#include <boost/variant/get.hpp>
+#include <boost/variant/apply_visitor.hpp>
 #include <CGAL/Mesh_3/Has_features.h>
 #include <CGAL/IO/io.h>
 
+#include <tuple>
+
 namespace CGAL {
 namespace Mesh_3 {
 namespace internal {
@@ -49,6 +53,26 @@ struct Index_generator<T, T>
   typedef Index   type;
 };
 
+template <typename MD, bool has_feature = Has_features<MD>::value>
+struct Indices_tuple_generator
+{
+  using type = std::tuple<typename MD::Subdomain_index,
+                          typename MD::Surface_patch_index,
+                          typename MD::Curve_index,
+                          typename MD::Corner_index
+                          >;
+};
+
+template <typename MD>
+struct Indices_tuple_generator<MD, false>
+{
+  using type = std::tuple<typename MD::Subdomain_index,
+                          typename MD::Surface_patch_index>;
+};
+
+template <typename MD>
+using Indices_tuple_t = typename Indices_tuple_generator<MD>::type;
+
 // Nasty meta-programming to get a boost::variant of four types that
 // may not be all different.
 template <typename T0> struct seq1 {
@@ -245,6 +269,65 @@ struct Write_mesh_domain_index<Mesh_domain, false> {
 }; // end template partial specialization
    // Write_mesh_domain_index<Mesh_domain, false>
 
+template <typename, typename Index>
+struct Read_write_index {
+  void operator()(std::ostream& os, int, Index index) const {
+    if(is_ascii(os)) os << oformat(index);
+    else CGAL::write(os, index);
+  }
+  Index operator()(std::istream& is, int) const {
+    Index index;
+    if(is_ascii(is)) is >> iformat(index);
+    else CGAL::read(is, index);
+    return index;
+  }
+};
+
+struct Variant_write_visitor {
+  std::ostream& os;
+  template <typename T>
+  void operator()(T v) const {
+    if(is_ascii(os)) os << CGAL::oformat(v);
+    else CGAL::write(os, v);
+  }
+};
+
+template <typename Index>
+struct Variant_read_visitor {
+  std::istream& is;
+  Index& variant;
+  template <typename T>
+  void operator()(T) const {
+    T v;
+    if(is_ascii(is)) is >> CGAL::iformat(v);
+    else CGAL::read(is, v);
+    variant = v;
+  }
+};
+
+template <typename Indices_types, typename... Args>
+struct Read_write_index<Indices_types, boost::variant<Args...>> {
+  using Index = boost::variant<Args...>;
+  using index_seq = std::make_index_sequence<std::tuple_size<Indices_types>::value>;
+
+  template <std::size_t... Is>
+  Index get_index(int dimension, std::index_sequence<Is...>) const{
+    static const Index variants[] = { std::tuple_element_t<Is, Indices_types>{}... };
+    return variants[dimension < 0 ? 0 : 3-dimension];
+  }
+
+  void operator()(std::ostream& os, int, Index index) const {
+    Variant_write_visitor visitor{os};
+    apply_visitor(visitor, index);
+  }
+  Index operator()(std::istream& is, int dimension) const {
+    Index index = get_index(dimension, index_seq{});
+    Variant_read_visitor<Index> visitor{is, index};
+    apply_visitor(visitor, index);
+    return index;
+  }
+};
+
 } // end namespace internal
 } // end namespace Mesh_3
 } // end namespace CGAL

Mesh_3/test/Mesh_3/CMakeLists.txt

@@ -118,6 +118,12 @@ if ( CGAL_FOUND )
     endforeach()
   endif()
 
+  if(BUILD_TESTING)
+    set_tests_properties(
+      execution___of__test_meshing_polyhedron_with_features
+      execution___of__test_meshing_verbose
+      PROPERTIES RESOURCE_LOCK Mesh_3_Tests_IO)
+  endif()
 else()
 
     message(STATUS "This program requires the CGAL library, and will not be compiled.")

Polyhedron/demo/Polyhedron/Plugins/Mesh_3/C3t3_rib_exporter_plugin.cpp

@@ -732,8 +732,8 @@ void
 C3t3_rib_exporter_plugin::
 write_surface_cells(const C3t3& c3t3, const Plane& /* plane */, std::ofstream& out)
 {
-  Geom_traits::Construct_point_3 wp2p
+  const Geom_traits& gt = c3t3.triangulation().geom_traits();
-    = c3t3.triangulation().geom_traits().construct_point_3_object();
+  Geom_traits::Construct_point_3 wp2p = gt.construct_point_3_object();
 
   for ( C3t3::Cells_in_complex_iterator it_cell = c3t3.cells_in_complex_begin(),
        end = c3t3.cells_in_complex_end() ; it_cell != end ; ++it_cell )
@@ -755,7 +755,7 @@ write_surface_cells(const C3t3& c3t3, const Plane& /* plane */, std::ofstream& o
 
     //const int TRANSPARENCY_ALPHA_VALUE = 100;
     CGAL::Bbox_3 bbox = c3t3.bbox();
-    float relPos = static_cast<float>((c->weighted_circumcenter().x() - bbox.xmin())
+    float relPos = static_cast<float>((c->weighted_circumcenter(gt).x() - bbox.xmin())
                                       / (bbox.xmax() - bbox.xmin()));
     float TRANSPARENCY_ALPHA_VALUE =
       1.f -