Merge pull request #407 from afabri/BGL-bug_fix_make_triangle-GF

bug fix:  set_next() was not called for the opposite edges

Author: lrineau

BGL/include/CGAL/boost/graph/helpers.h

@@ -457,6 +457,9 @@ make_triangle(const P& p0, const P& p1, const P& p2, Graph& g)
   h0 = opposite(h0,g);
   h1 = opposite(h1,g);
   h2 = opposite(h2,g);
+  set_next(h0, h2, g);
+  set_next(h2, h1, g);
+  set_next(h1, h0, g);
   set_target(h0, v0, g);
   set_target(h1, v1, g);
   set_target(h2, v2, g);
@@ -517,6 +520,10 @@ make_quad(const P& p0, const P& p1, const P& p2, const P& p3, Graph& g)
   h1 = opposite(h1,g);
   h2 = opposite(h2,g);
   h3 = opposite(h3,g);
+  set_next(h0, h3, g);
+  set_next(h3, h2, g);
+  set_next(h2, h1, g);
+  set_next(h1, h0, g);
   set_target(h0, v0, g);
   set_target(h1, v1, g);
   set_target(h2, v2, g);

BGL/test/BGL/test_helpers.cpp

@@ -42,19 +42,23 @@ int main()
   Mesh m;
   halfedge_descriptor hd;
   hd = CGAL::make_triangle(a,b,c,m);
-  assert(CGAL::is_triangle(hd,m));
+  assert(CGAL::is_isolated_triangle(hd,m));
+  assert(CGAL::is_valid(m));
   m.clear();
   hd = CGAL::make_quad(a,b,c,d,m);
-  assert(CGAL::is_quad(hd,m));
+  assert(CGAL::is_isolated_quad(hd,m));
+  assert(CGAL::is_valid(m));
   assert(CGAL::is_quad_mesh(m));
   m.clear();
   hd = CGAL::make_tetrahedron(a,b,c,d,m);
   assert(CGAL::is_tetrahedron(hd,m));
   assert(CGAL::is_triangle_mesh(m));
+  assert(CGAL::is_valid(m));
   m.clear();
   hd = CGAL::make_hexahedron(a,b,c,d,aa,bb,cc,dd,m);
   assert(CGAL::is_hexahedron(hd,m));
   assert(CGAL::is_quad_mesh(m));
+  assert(CGAL::is_valid(m));
 
   std::cerr << "done" << std::endl; 
   return 0;