diff options
author | Campbell Barton <ideasman42@gmail.com> | 2012-04-21 10:42:21 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2012-04-21 10:42:21 +0400 |
commit | f910abaddaec82dde3bd27ae3fa720b37f2359fc (patch) | |
tree | a422d3be2bbe278fc35fd06d5cd24356a5386fee /source/blender/editors/mesh | |
parent | 15eb3452ecd96e4d79d438e8c954918639dc283d (diff) |
fix error in recent rip refactor, also add comment.
Diffstat (limited to 'source/blender/editors/mesh')
-rw-r--r-- | source/blender/editors/mesh/editmesh_rip.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/source/blender/editors/mesh/editmesh_rip.c b/source/blender/editors/mesh/editmesh_rip.c index 37b12803962..c68d2a91460 100644 --- a/source/blender/editors/mesh/editmesh_rip.c +++ b/source/blender/editors/mesh/editmesh_rip.c @@ -658,13 +658,19 @@ static int edbm_rip_invoke__edge(bContext *C, wmOperator *op, wmEvent *event) } } + if (!edbm_rip_call_edgesplit(em, op)) { + return OPERATOR_CANCELLED; + } + + /* note: the output of the bmesh operator is ignored, since we built + * the contiguous loop pairs to split already, its possibe that some + * edge did not split even though it was tagged which would not work + * as expected (but not crash), however there are checks to ensure + * tagged edges will split. So far its not been an issue. */ edbm_ripsel_deselect_helper(bm, eloop_pairs, ar, projectMat, fmval); MEM_freeN(eloop_pairs); - if (!edbm_rip_call_edgesplit(em, op)) { - return OPERATOR_CANCELLED; - } return OPERATOR_FINISHED; } |