diff options
author | Campbell Barton <ideasman42@gmail.com> | 2013-08-01 22:33:35 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2013-08-01 22:33:35 +0400 |
commit | 78820edafcad730493e95d0dfdeccba02445c1f3 (patch) | |
tree | 69c99dc8102f03a90f719fa760e51f149a55b99e /source/blender/bmesh/operators/bmo_dupe.c | |
parent | e03b1668e0106019eed6097678cdaf7df3885638 (diff) |
use bmesh for loop macros for looping over mesh verts/edges/faces
Diffstat (limited to 'source/blender/bmesh/operators/bmo_dupe.c')
-rw-r--r-- | source/blender/bmesh/operators/bmo_dupe.c | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/source/blender/bmesh/operators/bmo_dupe.c b/source/blender/bmesh/operators/bmo_dupe.c index f9ff308b706..1dc7b0a414d 100644 --- a/source/blender/bmesh/operators/bmo_dupe.c +++ b/source/blender/bmesh/operators/bmo_dupe.c @@ -385,33 +385,30 @@ void bmo_split_exec(BMesh *bm, BMOperator *op) BMEdge *e; BMFace *f; BMIter iter, iter2; - int found; /* make sure to remove edges and verts we don't need */ - for (e = BM_iter_new(&iter, bm, BM_EDGES_OF_MESH, NULL); e; e = BM_iter_step(&iter)) { - found = 0; - f = BM_iter_new(&iter2, bm, BM_FACES_OF_EDGE, e); - for ( ; f; f = BM_iter_step(&iter2)) { + BM_ITER_MESH (e, &iter, bm, BM_EDGES_OF_MESH) { + bool found = false; + BM_ITER_ELEM (f, &iter2, e, BM_FACES_OF_EDGE) { if (!BMO_elem_flag_test(bm, f, SPLIT_INPUT)) { - found = 1; + found = true; break; } } - if (!found) { + if (found == false) { BMO_elem_flag_enable(bm, e, SPLIT_INPUT); } } - for (v = BM_iter_new(&iter, bm, BM_VERTS_OF_MESH, NULL); v; v = BM_iter_step(&iter)) { - found = 0; - e = BM_iter_new(&iter2, bm, BM_EDGES_OF_VERT, v); - for ( ; e; e = BM_iter_step(&iter2)) { + BM_ITER_MESH (v, &iter, bm, BM_VERTS_OF_MESH) { + bool found = false; + BM_ITER_ELEM (e, &iter2, v, BM_EDGES_OF_VERT) { if (!BMO_elem_flag_test(bm, e, SPLIT_INPUT)) { - found = 1; + found = true; break; } } - if (!found) { + if (found == false) { BMO_elem_flag_enable(bm, v, SPLIT_INPUT); } } |