[Bf-blender-cvs] [46b5cda] master: BMesh: remove redundant link-list manipulation

Campbell Barton noreply at git.blender.org
Tue Nov 15 02:25:56 CET 2016


Commit: 46b5cdaa4d1b9d78bbbd078e98f64de1e6c288a8
Author: Campbell Barton
Date:   Tue Nov 15 12:39:33 2016 +1100
Branches: master
https://developer.blender.org/rB46b5cdaa4d1b9d78bbbd078e98f64de1e6c288a8

BMesh: remove redundant link-list manipulation

No need to track previous edge in vert-separate cleanup

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

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

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

diff --git a/source/blender/bmesh/intern/bmesh_core.c b/source/blender/bmesh/intern/bmesh_core.c
index c71afc7..0cd9110 100644
--- a/source/blender/bmesh/intern/bmesh_core.c
+++ b/source/blender/bmesh/intern/bmesh_core.c
@@ -2405,18 +2405,13 @@ static void bmesh_vert_separate__cleanup(BMesh *bm, LinkNode *edges_separate)
 		do {
 			BMEdge *e_orig = n_orig->link;
 			LinkNode *n_step = n_orig->next;
-			LinkNode *n_prev = n_orig;
 			do {
 				BMEdge *e = n_step->link;
 				BLI_assert(e != e_orig);
 				if ((e->v1 == e_orig->v1) && (e->v2 == e_orig->v2)) {
 					BM_edge_splice(bm, e_orig, e);
-					n_prev->next = n_step->next;
-					n_step = n_prev;
 				}
-			} while ((void)
-			         (n_prev = n_step),
-			         (n_step = n_step->next));
+			} while ((n_step = n_step->next));
 
 		} while ((n_orig = n_orig->next) && n_orig->next);
 	} while ((edges_separate = edges_separate->next));




More information about the Bf-blender-cvs mailing list