Spin Tool: don't flip when using a negative angle
authorCampbell Barton <ideasman42@gmail.com>
Fri, 4 Jan 2019 05:17:49 +0000 (16:17 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 4 Jan 2019 06:07:12 +0000 (17:07 +1100)
The intention was to flip normals when extruding in the opposite
direction, however the sign of the angle isn't meaningful unless
the geometry center and region normal are taken into account.

Disable, may add back in a way that works more predictably.

source/blender/editors/mesh/editmesh_extrude_spin.c

index a9e78f740125fb6198569686ed7300eb38c62ddb..b7c09c2b029b390c5704ce502f085235439e0c8f 100644 (file)
@@ -66,7 +66,7 @@ static int edbm_spin_exec(bContext *C, wmOperator *op)
        RNA_float_get_array(op->ptr, "axis", axis);
        const int steps = RNA_int_get(op->ptr, "steps");
        const float angle = RNA_float_get(op->ptr, "angle");
        RNA_float_get_array(op->ptr, "axis", axis);
        const int steps = RNA_int_get(op->ptr, "steps");
        const float angle = RNA_float_get(op->ptr, "angle");
-       const bool use_normal_flip = RNA_boolean_get(op->ptr, "use_normal_flip") ^ (angle < 0.0f);
+       const bool use_normal_flip = RNA_boolean_get(op->ptr, "use_normal_flip");
        const bool dupli = RNA_boolean_get(op->ptr, "dupli");
        const bool use_auto_merge = (
                RNA_boolean_get(op->ptr, "use_auto_merge") &&
        const bool dupli = RNA_boolean_get(op->ptr, "dupli");
        const bool use_auto_merge = (
                RNA_boolean_get(op->ptr, "use_auto_merge") &&