[Bf-blender-cvs] [0e9a3882be1] blender-v2.90-release: Fix T80233: Extrude manifold can generate invalid geometry

Germano Cavalcante noreply at git.blender.org
Mon Sep 21 09:50:31 CEST 2020


Commit: 0e9a3882be1954691c7934365ac7d404ab6a062d
Author: Germano Cavalcante
Date:   Tue Sep 1 15:25:19 2020 -0300
Branches: blender-v2.90-release
https://developer.blender.org/rB0e9a3882be1954691c7934365ac7d404ab6a062d

Fix T80233: Extrude manifold can generate invalid geometry

Faces with only 2 sides were sometimes generated.

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

M	source/blender/bmesh/operators/bmo_extrude.c

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

diff --git a/source/blender/bmesh/operators/bmo_extrude.c b/source/blender/bmesh/operators/bmo_extrude.c
index eee31969971..757f9b390d3 100644
--- a/source/blender/bmesh/operators/bmo_extrude.c
+++ b/source/blender/bmesh/operators/bmo_extrude.c
@@ -614,7 +614,7 @@ void bmo_extrude_face_region_exec(BMesh *bm, BMOperator *op)
       BMEdge *e_other = BM_DISK_EDGE_NEXT(e, v);
       if ((e_other == e) || (BM_DISK_EDGE_NEXT(e_other, v) == e)) {
         /* Lose edge or BMVert is edge pair. */
-        BM_edge_collapse(bm, BMO_elem_flag_test(bm, e, EXT_TAG) ? e : e_other, v, true, false);
+        BM_edge_collapse(bm, BMO_elem_flag_test(bm, e, EXT_TAG) ? e : e_other, v, true, true);
       }
       else {
         BLI_assert(!BM_vert_is_edge_pair(v));



More information about the Bf-blender-cvs mailing list