[Bf-blender-cvs] [45d131f] master: Recent extrude rework caused bug with single faces

Campbell Barton noreply at git.blender.org
Tue Jan 13 19:26:16 CET 2015


Commit: 45d131ff08a577272dcd3177e8bfd230f7d8e3c9
Author: Campbell Barton
Date:   Wed Jan 14 05:24:16 2015 +1100
Branches: master
https://developer.blender.org/rB45d131ff08a577272dcd3177e8bfd230f7d8e3c9

Recent extrude rework caused bug with single faces

extrude_discrete_faces doesn't duplicate faces on extrude, see: T43237

===================================================================

M	source/blender/editors/mesh/editmesh_extrude.c

===================================================================

diff --git a/source/blender/editors/mesh/editmesh_extrude.c b/source/blender/editors/mesh/editmesh_extrude.c
index 023f776..9b1b0b9 100644
--- a/source/blender/editors/mesh/editmesh_extrude.c
+++ b/source/blender/editors/mesh/editmesh_extrude.c
@@ -335,7 +335,7 @@ static bool edbm_extrude_mesh(Object *obedit, BMEditMesh *em, wmOperator *op)
 {
 	bool changed = false;
 	const char htype = edbm_extrude_htype_from_em_select(em);
-	enum {NONE = 0, ELEM_FLAG, VERT_ONLY, EDGE_ONLY, FACE_ONLY} nr;
+	enum {NONE = 0, ELEM_FLAG, VERT_ONLY, EDGE_ONLY} nr;
 
 	if (em->selectmode & SCE_SELECT_VERTEX) {
 		if      (em->bm->totvertsel == 0) nr = NONE;
@@ -350,7 +350,6 @@ static bool edbm_extrude_mesh(Object *obedit, BMEditMesh *em, wmOperator *op)
 	}
 	else {
 		if      (em->bm->totfacesel == 0) nr = NONE;
-		else if (em->bm->totfacesel == 1) nr = FACE_ONLY;
 		else                              nr = ELEM_FLAG;
 	}
 
@@ -366,9 +365,6 @@ static bool edbm_extrude_mesh(Object *obedit, BMEditMesh *em, wmOperator *op)
 		case EDGE_ONLY:
 			changed = edbm_extrude_edges_indiv(em, op, BM_ELEM_SELECT);
 			break;
-		case FACE_ONLY:
-			changed = edbm_extrude_discrete_faces(em, op, BM_ELEM_SELECT);
-			break;
 	}
 	
 	if (changed) {




More information about the Bf-blender-cvs mailing list