...
 
Commits (2)
...@@ -363,7 +363,8 @@ private: // topological modifiers ...@@ -363,7 +363,8 @@ private: // topological modifiers
_m.set_face_handle( new_heh, _m.face_handle(heh) ); _m.set_face_handle( new_heh, _m.face_handle(heh) );
_m.set_halfedge_handle( vh, new_heh); _m.set_halfedge_handle( vh, new_heh);
_m.set_halfedge_handle( _m.face_handle(heh), heh ); if ( !_m.is_boundary(heh) )
_m.set_halfedge_handle( _m.face_handle(heh), heh );
_m.set_halfedge_handle( vh1, opp_new_heh ); _m.set_halfedge_handle( vh1, opp_new_heh );
// Never forget this, when playing with the topology // Never forget this, when playing with the topology
......
...@@ -353,8 +353,8 @@ TEST_F(OpenMeshSubdividerUniform_Triangle, Subdivider_Loop_delete_vertex) { ...@@ -353,8 +353,8 @@ TEST_F(OpenMeshSubdividerUniform_Triangle, Subdivider_Loop_delete_vertex) {
OpenMesh::Subdivider::Uniform::LoopT<Mesh> loop; OpenMesh::Subdivider::Uniform::LoopT<Mesh> loop;
// Check setup // Check setup
EXPECT_EQ(9u, mesh_.n_vertices() ) << "Wrong number of vertices"; EXPECT_EQ(8u, mesh_.n_vertices() ) << "Wrong number of vertices";
EXPECT_EQ(8u, mesh_.n_faces() ) << "Wrong number of faces"; EXPECT_EQ(6u, mesh_.n_faces() ) << "Wrong number of faces";
// Execute 3 subdivision steps // Execute 3 subdivision steps
loop.attach(mesh_); loop.attach(mesh_);
...@@ -362,8 +362,8 @@ TEST_F(OpenMeshSubdividerUniform_Triangle, Subdivider_Loop_delete_vertex) { ...@@ -362,8 +362,8 @@ TEST_F(OpenMeshSubdividerUniform_Triangle, Subdivider_Loop_delete_vertex) {
loop.detach(); loop.detach();
// Check setup // Check setup
EXPECT_EQ(121u, mesh_.n_vertices() ) << "Wrong number of vertices after subdivision with loop"; EXPECT_EQ(225u, mesh_.n_vertices() ) << "Wrong number of vertices after subdivision with loop";
EXPECT_EQ(216u, mesh_.n_faces() ) << "Wrong number of faces after subdivision with loop"; EXPECT_EQ(384u, mesh_.n_faces() ) << "Wrong number of faces after subdivision with loop";
} }
......