I clarified the documentation, by putting closer to the top some
details about ch_graham_andrew_scan(), especially the precondition.
I also updated the ch_graham_anderson() example in the documentation.
TODO: Investigate more:
In my system, the following line in the example:
*result = *(V.rbegin()); ++result;
is rendered without the first `*' in the HTML documentation.
The previous version did not output the last point of the convex
hull to the result iterator. As a result, ch_graham_anderson.cpp
did not output a correct convex hull in some cases.
Bugfix: if neither Eigen nor Lapack is present, including header files from
the point set processing package should not lead to a compilation error.
Tested in CGAL-4.4-Ic-100
All Polygon_2::erase() overloads now return an iterator or a circulator,
to make them more convenient (and STL-like).
That patch is a non-significant contribution (as regard the owernship of
code) from a CGAL-discuss user.
Tested in CGAL-4.4.-Ic-91.
Enhance test/Mesh_3/test_meshing_polylines_only.cpp
- It was far too verbose.
- It was not really testing anything but the absence of assertions.
Tested in CGAL-4.4-Ic-91.
The section 'Non const access' of the documentation of Triangulation_2
was including const methods. I move the const method in the section
'Access Functions'.
That function set_shared_data is never tested in CGAL, happened not to
be callable using its documented API. I do not understand why there is
an additional template parameter 'PrimtiveType'. That commit removes it.
(cherry picked from commit 99790a0d39864c3d2e72d18876f41603ab94a5a5)
Introduces the Triangulated Surface Mesh Segmentation Package
See also wiki/Features/Surface_mesh_segmentation
Tested in 4.4-Ic-84
Approved by the release manager
Conflicts:
Documentation/biblio/how_to_cite.html
Documentation/biblio/manual-4.3.bib
Manual/doc_tex/Manual/geom.bib
Small feature:
cgal.org/wiki/Features/Small_Features/Boolean_tag_is_a_Boost_Integral_Constant
Now CGAL::Boolean_tag<true> and CGAL::Boolean_tag<true> are model of the
Boost concept "Integral Constant".
Tested in CGAL-4.4-Ic-84.
The Intel Compiler 14.0, with -O2 or -O3, triggers a bug in the old
version of the code. After the construction of the filtered circulator,
`test(*this)` was not guaranteed. I am not sure if the bug is in the
code or in the compiler, but the constructor was an ugly piece of
code! (*this) was used in the body of the constructor in non obvious
ways. An object is reputed constructed once the internalizers are
evaluated, as far as I know, so probably the previous version of the
code was valid. However, that sound strange to copy an object inside the
body of the constructor of that object. Maybe that confused the
compiler.
Anyway, I have modified the implementation of that constructor in a
cleaner way, and I have added an assertion (that was failing with the
old implementation).
Sébastien Loriot
Panagiotis Cheilaris
Michael Hemmer
Laurent Rineau
Andreas Fabri