diff options
| author | Bastien Montagne <montagne29@wanadoo.fr> | 2014-05-22 00:35:08 +0400 |
|---|---|---|
| committer | Bastien Montagne <montagne29@wanadoo.fr> | 2014-05-22 00:37:50 +0400 |
| commit | a217db0d6373a4050658049a9ded84f34b33f952 (patch) | |
| tree | 18a2c2befaaf38f0d4ab40e1a8b96a45e71bf067 /source/blender/bmesh/intern/bmesh_edgeloop.c | |
| parent | 8d297394ba51246f2db0d503df7f4cae19ca6e92 (diff) | |
Fix T40297: Crash while ripping an edge when autosmooth is activated.
Turned out there was still quite a few cases were indices were set dirty,
but elem_index_dirty was not tagged accordingly (mostly for BM_LOOP,
but a few others as well). So probably this crash was not the only one
hidden here.
Hopefully all possible cases were catched this time!
Diffstat (limited to 'source/blender/bmesh/intern/bmesh_edgeloop.c')
| -rw-r--r-- | source/blender/bmesh/intern/bmesh_edgeloop.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source/blender/bmesh/intern/bmesh_edgeloop.c b/source/blender/bmesh/intern/bmesh_edgeloop.c index bbf4b661a1b..c8a954ab6a3 100644 --- a/source/blender/bmesh/intern/bmesh_edgeloop.c +++ b/source/blender/bmesh/intern/bmesh_edgeloop.c @@ -189,7 +189,7 @@ static void vs_add(BLI_mempool *vs_pool, ListBase *lb, struct VertStep *vs_new = BLI_mempool_alloc(vs_pool); vs_new->v = v; - BM_elem_index_set(v, iter_tot); + BM_elem_index_set(v, iter_tot); /* set_dirty */ /* This edge stores a direct path back to the original vertex so we can * backtrack without having to store an array of previous verts. */ @@ -248,6 +248,7 @@ static bool bm_loop_path_build_step(BLI_mempool *vs_pool, ListBase *lb, const in BLI_mempool_free(vs_pool, vs); } + /* bm->elem_index_dirty |= BM_VERT; */ /* Commented because used in a loop, and this flag has already been set. */ /* lb is now full of free'd items, overwrite */ *lb = lb_tmp; @@ -303,6 +304,7 @@ bool BM_mesh_edgeloops_find_path(BMesh *bm, ListBase *r_eloops, /* edge args are dummy */ vs_add(vs_pool, &lb_src, v_src, v_src->e, 1); vs_add(vs_pool, &lb_dst, v_dst, v_dst->e, -1); + bm->elem_index_dirty |= BM_VERT; do { if ((bm_loop_path_build_step(vs_pool, &lb_src, 1, v_match) == false) || v_match[0]) { |