[Bf-blender-cvs] [de031b7] master: BMesh: replace radial count with simple checks

Campbell Barton noreply at git.blender.org
Sat May 2 20:47:16 CEST 2015


Commit: de031b7c89cc5c5dd079d407ebd3a690b1c4b567
Author: Campbell Barton
Date:   Sat May 2 17:12:32 2015 +1000
Branches: master
https://developer.blender.org/rBde031b7c89cc5c5dd079d407ebd3a690b1c4b567

BMesh: replace radial count with simple checks

===================================================================

M	source/blender/bmesh/intern/bmesh_core.c
M	source/blender/bmesh/intern/bmesh_queries.c

===================================================================

diff --git a/source/blender/bmesh/intern/bmesh_core.c b/source/blender/bmesh/intern/bmesh_core.c
index 7083642..06bd622 100644
--- a/source/blender/bmesh/intern/bmesh_core.c
+++ b/source/blender/bmesh/intern/bmesh_core.c
@@ -1053,8 +1053,9 @@ static bool disk_is_flagged(BMVert *v, const char api_flag)
 			return false;
 		}
 		
-		if (bmesh_radial_length(l) == 1)
+		if (BM_edge_is_boundary(l->e)) {
 			return false;
+		}
 		
 		do {
 			if (!BM_ELEM_API_FLAG_TEST(l->f, api_flag))
diff --git a/source/blender/bmesh/intern/bmesh_queries.c b/source/blender/bmesh/intern/bmesh_queries.c
index 61328a5..296bde9 100644
--- a/source/blender/bmesh/intern/bmesh_queries.c
+++ b/source/blender/bmesh/intern/bmesh_queries.c
@@ -877,7 +877,7 @@ bool BM_vert_is_manifold(const BMVert *v)
 		/* loose edge or edge shared by more than two faces,
 		 * edges with 1 face user are OK, otherwise we could
 		 * use BM_edge_is_manifold() here */
-		if (e->l == NULL || bmesh_radial_length(e->l) > 2) {
+		if (e->l == NULL || (e->l != e->l->radial_next->radial_next)) {
 			return false;
 		}
 		len++;




More information about the Bf-blender-cvs mailing list