revert own commit r44684 it broke edges with one vertex on a boundry

This commit is contained in:
Campbell Barton
2012-03-07 03:46:30 +00:00
parent 04beeea9c9
commit 4fe4cfdb88
2 changed files with 16 additions and 25 deletions

View File

@@ -223,7 +223,7 @@ void bmo_edgesplit_exec(BMesh *bm, BMOperator *op)
BMFace *f, *f2;
BMLoop *l, *l2, *l3;
BMLoop *l_next, *l_prev;
BMEdge *e;
BMEdge *e, *e2;
BMVert *v, *v2;
/* face/vert aligned vert array */
@@ -236,32 +236,23 @@ void bmo_edgesplit_exec(BMesh *bm, BMOperator *op)
BMO_slot_buffer_flag_enable(bm, op, "edges", EDGE_SEAM, BM_EDGE);
/* untag edges not connected to other tagged edges */
{
unsigned char *vtouch;
BM_mesh_elem_index_ensure(bm, BM_VERT);
vtouch = MEM_callocN(sizeof(char) * bm->totvert, __func__);
/* single marked edges unconnected to any other marked edges
* are illegal, go through and unmark them */
BMO_ITER(e, &siter, bm, op, "edges", BM_EDGE) {
/* lame, but we dont want the count to exceed 255,
* so just count to 2, its all we need */
unsigned char *c;
c = &vtouch[BM_elem_index_get(e->v1)]; if (*c < 2) (*c)++;
c = &vtouch[BM_elem_index_get(e->v2)]; if (*c < 2) (*c)++;
}
BMO_ITER(e, &siter, bm, op, "edges", BM_EDGE) {
if (vtouch[BM_elem_index_get(e->v1)] == 1 &&
vtouch[BM_elem_index_get(e->v2)] == 1)
{
BMO_elem_flag_disable(bm, e, EDGE_SEAM);
/* single marked edges unconnected to any other marked edges
* are illegal, go through and unmark them */
BMO_ITER(e, &siter, bm, op, "edges", BM_EDGE) {
for (i = 0; i < 2; i++) {
BM_ITER(e2, &iter, bm, BM_EDGES_OF_VERT, i ? e->v2 : e->v1) {
if (e != e2 && BMO_elem_flag_test(bm, e2, EDGE_SEAM)) {
break;
}
}
if (e2) {
break;
}
}
MEM_freeN(vtouch);
if (!e2) {
BMO_elem_flag_disable(bm, e, EDGE_SEAM);
}
}
etags = MEM_callocN(sizeof(EdgeTag) * bm->totedge, "EdgeTag");

View File

@@ -2522,7 +2522,7 @@ static int mesh_rip_invoke(bContext *C, wmOperator *op, wmEvent *event)
dist = FLT_MAX;
BM_ITER(v, &iter, em->bm, BM_VERTS_OF_MESH, NULL) {
if (BM_elem_flag_test(v, BM_ELEM_SELECT)) {
/* disable by default, re-enable winner at end */
/* disable by default, re-enable winner at end */
BM_elem_select_set(bm, v, FALSE);
BM_ITER(e, &eiter, bm, BM_EDGES_OF_VERT, v) {