From ff7ddb1925c5a0bf64d3a353980e771266ebd1bf Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Sun, 26 Feb 2012 05:04:30 +0000 Subject: [PATCH] change vertex dissolve not to collapse edges when its only connected to 2, rather then joining the faces on either side. This way. IMHO is more useful since it means you can simplify the geometry between faces without joining them and it distinguishes vertex dissolve more from edge dissolve - which will join both faces still. --- source/blender/bmesh/operators/bmo_dissolve.c | 2 +- source/blender/bmesh/tools/BME_bevel.c | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/source/blender/bmesh/operators/bmo_dissolve.c b/source/blender/bmesh/operators/bmo_dissolve.c index ddef478a4aa..62c98dc2871 100644 --- a/source/blender/bmesh/operators/bmo_dissolve.c +++ b/source/blender/bmesh/operators/bmo_dissolve.c @@ -334,7 +334,7 @@ void dissolveverts_exec(BMesh *bm, BMOperator *op) if (BM_vert_edge_count(v) == 2) { /* collapse the ver */ - BM_vert_collapse_faces(bm, v->e, v, 1.0f, TRUE); + BM_vert_collapse_faces(bm, v->e, v, 1.0f, FALSE); continue; } diff --git a/source/blender/bmesh/tools/BME_bevel.c b/source/blender/bmesh/tools/BME_bevel.c index 144a0f7ea5e..903500b2b06 100644 --- a/source/blender/bmesh/tools/BME_bevel.c +++ b/source/blender/bmesh/tools/BME_bevel.c @@ -153,6 +153,9 @@ static int BME_Bevel_Dissolve_Disk(BMesh *bm, BMVert *v) e = v->e; elast = bmesh_disk_nextedge(e, v); + + /* BMESH_TODO, figure out if its possible we had a double edge here and need to splice it, + * last bool arg */ bmesh_jekv(bm, e, v, FALSE); l1 = elast->l;