wip apply Laurent's reviews

Mesh_3/doc/Mesh_3/CGAL/Mesh_3/parameters.h

@@ -491,7 +491,8 @@ unspecified_type initial_points_generator(const InitialPointsGenerator& generato
  * \ingroup PkgMesh3Parameters
  *
  * The function `parameters::initial_points()` enables the user to specify a container, model of `Range`, that contains
- * initial points to be used in the `make_mesh_3()` function for mesh generation. Items in the container are
+ * initial points constructed on surfaces,
+ * to be used in the `make_mesh_3()` function for mesh generation. Items in the container are
  * tuple-like objects containing a `Weighted_point_3`, an `int`, and a `MeshDomain::Index`,
  * where `Weighted_point_3` represents the position and the weight of the point,
  * `int` the dimension of the minimal subcomplex on which the point lies,

Mesh_3/examples/Mesh_3/mesh_3D_image_with_image_initialization.cpp

@@ -11,7 +11,7 @@
 #include <CGAL/make_mesh_3.h>
 #include <CGAL/Image_3.h>
 
-#include <CGAL/SMDS_3/Dump_c3t3.h>
+#include <CGAL/IO/File_medit.h>
 
 // Domain
 typedef CGAL::Exact_predicates_inexact_constructions_kernel K;
@@ -56,7 +56,9 @@ int main()
   /// [Meshing]
 
   // Output
-  CGAL::dump_c3t3(c3t3, "out");
+  std::ofstream medit_file("out.mesh");
+  CGAL::IO::write_MEDIT(medit_file, c3t3);
+  medit_file.close();
 
   return 0;
 }

Mesh_3/include/CGAL/Mesh_3/Construct_initial_points_gray_image.h

@@ -51,6 +51,9 @@ struct Construct_initial_points_gray_image
   const typename MeshDomain::R::FT iso_value_;
   Functor image_values_to_subdomain_indices_;
 
+  /*!
+  * @todo
+  */
   Construct_initial_points_gray_image(const CGAL::Image_3 & image,
          const MeshDomain& domain,
          const double iso_value,

Mesh_3/include/CGAL/Mesh_3/Construct_initial_points_labeled_image.h

@@ -100,6 +100,9 @@ struct Construct_initial_points_labeled_image
   const CGAL::Image_3& image_;
   const MeshDomain& domain_;
 
+  /*!
+  * @todo
+  */
   Construct_initial_points_labeled_image(const CGAL::Image_3& image,
                                          const MeshDomain& domain)
     : image_(image)
@@ -122,7 +125,7 @@ struct Construct_initial_points_labeled_image
     return pts;
   }
 
-  /*!
+  /* //internal undocumented
    * \brief Same as above, but a `TransformOperator` that transforms values of the image is used.
    *
    * @tparam OutputIterator model of `OutputIterator` for
@@ -306,9 +309,6 @@ struct Construct_initial_points_labeled_image
             continue;
 
           // insert point in temporary triangulation
-
-          /// The following lines show how to insert initial points in the
-          /// `c3t3` object. [insert initial points]
           Vertex_handle v = tr.insert(pi);
 
           // `v` could be null if `pi` is hidden by other vertices of `tr`.
@@ -316,7 +316,6 @@ struct Construct_initial_points_labeled_image
 
           c3t3.set_dimension(v, 2); // by construction, points are on surface
           c3t3.set_index(v, intersect_index);
-          /// [insert initial points]
 
           *pts++ = std::make_pair(pi, intersect_index); // dimension 2 by construction, points are on surface
         }