phynexis-deps/cgal
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

check face_index map is initialized

Browse Source 
this is a minimal check, that checks that the 2 first faces of the range
do not share the same index
This test is not complete and the complete check should test that no 2 vertices
share the same index, but that's too costly to be worth it here
This commit is contained in:
Jane Tournois
2016-04-22 16:25:17 +02:00
parent 638b0ba9ab
commit ccf66afe26
1 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/border.h
+13 -1
View File
@@ -159,14 +159,26 @@ namespace Polygon_mesh_processing {
typedef typename boost::property_map<typename internal::Dummy_PM,
CGAL::face_index_t>::type Unset_FIMap;
if (boost::is_same<FIMap, Unset_FIMap>::value)
if (boost::is_same<FIMap, Unset_FIMap>::value || boost::size(faces) == 1)
{
//face index map is not given in named parameters, nor as an internal property map
return internal::border_halfedges_impl(faces, out, pmesh);
}
//face index map given as a named parameter, or as an internal property map
FIMap fim = choose_param(get_param(np, face_index),
get(CGAL::face_index, pmesh));
//make a minimal check that it's properly initialized :
//if the 2 first faces have the same id, we know the property map is not initialized
typename boost::range_iterator<const FaceRange>::type it = boost::const_begin(faces);
if (get(fim, *it++) == get(fim, *it))
{
std::cerr << "WARNING : the internal property map for CGAL::face_index_t" << std::endl
<< " is not properly initialized." << std::endl
<< " Initialize it before calling border_halfedges()" << std::endl;
}
return internal::border_halfedges_impl(faces, fim, out, pmesh);
}