forked from bartvdbraak/blender
Fix T44618: Rip Fill on a single vert would only generate one of the two expected faces.
Was tagging (for filling) the wrong edge for one of the two involved loops...
This commit is contained in:
parent
51f33a2e55
commit
4487358da7
@ -784,11 +784,20 @@ static int edbm_rip_invoke__vert(bContext *C, wmOperator *op, const wmEvent *eve
|
|||||||
else {
|
else {
|
||||||
if (BM_edge_is_manifold(e_best)) {
|
if (BM_edge_is_manifold(e_best)) {
|
||||||
BMLoop *l_iter, *l_first;
|
BMLoop *l_iter, *l_first;
|
||||||
|
|
||||||
l_iter = l_first = e_best->l;
|
l_iter = l_first = e_best->l;
|
||||||
do {
|
do {
|
||||||
larr[larr_len] = BM_edge_vert_share_loop(l_iter, v);
|
larr[larr_len] = BM_edge_vert_share_loop(l_iter, v);
|
||||||
|
|
||||||
|
if (do_fill) {
|
||||||
|
/* Only needed when filling...
|
||||||
|
* Also, we never want to tag best edge, that one won't change during split. See T44618. */
|
||||||
|
if (larr[larr_len]->e == e_best) {
|
||||||
|
BM_elem_flag_enable(larr[larr_len]->prev->e, BM_ELEM_TAG);
|
||||||
|
}
|
||||||
|
else {
|
||||||
BM_elem_flag_enable(larr[larr_len]->e, BM_ELEM_TAG);
|
BM_elem_flag_enable(larr[larr_len]->e, BM_ELEM_TAG);
|
||||||
|
}
|
||||||
|
}
|
||||||
larr_len++;
|
larr_len++;
|
||||||
} while ((l_iter = l_iter->radial_next) != l_first);
|
} while ((l_iter = l_iter->radial_next) != l_first);
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user