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 05:17:49 +0000 (16:17 +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 e9758cb1a5a8f87f08a142a3cc4b0851658a8d06..50ea452f7f36b6cfc6dc14c6d1ea6288cc1315e9 100644 (file)
@@ -71,7 +71,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") &&