[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [56785] trunk/blender/source/blender/bmesh /intern/bmesh_walkers_impl.c: tweak to r56784, no functional change, utility function worked confusingly.

Campbell Barton ideasman42 at gmail.com
Tue May 14 08:10:26 CEST 2013


Revision: 56785
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=56785
Author:   campbellbarton
Date:     2013-05-14 06:10:26 +0000 (Tue, 14 May 2013)
Log Message:
-----------
tweak to r56784, no functional change, utility function worked confusingly.

Revision Links:
--------------
    http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=56784

Modified Paths:
--------------
    trunk/blender/source/blender/bmesh/intern/bmesh_walkers_impl.c

Modified: trunk/blender/source/blender/bmesh/intern/bmesh_walkers_impl.c
===================================================================
--- trunk/blender/source/blender/bmesh/intern/bmesh_walkers_impl.c	2013-05-14 04:55:21 UTC (rev 56784)
+++ trunk/blender/source/blender/bmesh/intern/bmesh_walkers_impl.c	2013-05-14 06:10:26 UTC (rev 56785)
@@ -429,12 +429,12 @@
  * Starts at a tool-flagged edge and walks over the edge loop
  */
 
-/* utility function */
-static bool bm_loop_is_single(BMLoop *l)
+/* utility function to see if an edge is apart of an ngon boundary */
+static bool bm_edge_is_single(BMEdge *e)
 {
-	return ((BM_edge_is_boundary(l->e)) &&
-	        (l->f->len != 4) &&
-	        (BM_edge_is_boundary(l->next->e) || BM_edge_is_boundary(l->prev->e)));
+	return ((BM_edge_is_boundary(e)) &&
+	        (e->l->f->len != 4) &&
+	        (BM_edge_is_boundary(e->l->next->e) || BM_edge_is_boundary(e->l->prev->e)));
 }
 
 static void bmw_LoopWalker_begin(BMWalker *walker, void *data)
@@ -453,7 +453,7 @@
 	lwalk->cur = lwalk->start = e;
 	lwalk->lastv = lwalk->startv = v;
 	lwalk->is_boundary = BM_edge_is_boundary(e);
-	lwalk->is_single = (lwalk->is_boundary && bm_loop_is_single(e->l));
+	lwalk->is_single = (lwalk->is_boundary && bm_edge_is_single(e));
 
 	/* could also check that vertex*/
 	if ((lwalk->is_boundary == false) &&
@@ -648,7 +648,7 @@
 			} while (true);
 		}
 
-		if (owalk.is_single == false && bm_loop_is_single(l)) {
+		if (owalk.is_single == false && bm_edge_is_single(l->e)) {
 			l = NULL;
 		}
 




More information about the Bf-blender-cvs mailing list