Change default for bevel to match previous behavior.
authorHoward Trickey <howard.trickey@gmail.com>
Sun, 5 Jul 2015 17:31:26 +0000 (13:31 -0400)
committerHoward Trickey <howard.trickey@gmail.com>
Sun, 5 Jul 2015 17:31:26 +0000 (13:31 -0400)
Have reconsidered and feel it best to try matching previous behavior
(doing "loop slides" where possible) as default. This will avoid the
need to change regression tests, among other things.

source/blender/editors/mesh/editmesh_bevel.c
source/blender/makesdna/DNA_modifier_types.h
source/blender/makesrna/intern/rna_modifier.c
source/blender/modifiers/intern/MOD_bevel.c

index 709977629096715cda0cf8ef52ca64a1d928d650..dda21b39264491356369910a2b46e0d27ae0cbaa 100644 (file)
@@ -502,6 +502,6 @@ void MESH_OT_bevel(wmOperatorType *ot)
        RNA_def_boolean(ot->srna, "vertex_only", false, "Vertex Only", "Bevel only vertices");
        RNA_def_boolean(ot->srna, "clamp_overlap", false, "Clamp Overlap",
                        "Do not allow beveled edges/vertices to overlap each other");
-       RNA_def_boolean(ot->srna, "loop_slide", false, "Loop Slide", "Prefer slide along edge to even widths");
+       RNA_def_boolean(ot->srna, "loop_slide", true, "Loop Slide", "Prefer slide along edge to even widths");
        RNA_def_int(ot->srna, "material", -1, -1, INT_MAX, "Material", "Material for bevel faces (-1 means use adjacent faces)", -1, 100);
 }
index 8319dc3a874c211a32a136d05ba7ffbf4ffc3279..322df66c7fcfcb0ec92740dde50550ae7db1c5af 100644 (file)
@@ -338,7 +338,7 @@ enum {
 /*     MOD_BEVEL_EVEN          = (1 << 11), */
 /*     MOD_BEVEL_DIST          = (1 << 12), */  /* same as above */
        MOD_BEVEL_OVERLAP_OK    = (1 << 13),
-       MOD_BEVEL_LOOP_SLIDE    = (1 << 14),
+       MOD_BEVEL_EVEN_WIDTHS   = (1 << 14),
 };
 
 /* BevelModifierData->val_flags (not used as flags any more) */
index 7bbf975ac85ead4af4d5d716dc56db6dcf9ca057..affaef3aa7b9f3668337dd13b4c1cad58e4aef87 100644 (file)
@@ -2826,7 +2826,7 @@ static void rna_def_modifier_bevel(BlenderRNA *brna)
        RNA_def_property_update(prop, 0, "rna_Modifier_update");
 
        prop = RNA_def_property(srna, "loop_slide", PROP_BOOLEAN, PROP_NONE);
-       RNA_def_property_boolean_sdna(prop, NULL, "flags", MOD_BEVEL_LOOP_SLIDE);
+       RNA_def_property_boolean_negative_sdna(prop, NULL, "flags", MOD_BEVEL_EVEN_WIDTHS);
        RNA_def_property_ui_text(prop, "Loop Slide", "Prefer sliding along edges to having even widths");
        RNA_def_property_update(prop, 0, "rna_Modifier_update");
 }
index 63e4b15d52e70ad13098ac6c69f2b286c20aac21..5874029ae0862279b9eec1964bf0ab7b8842b7b2 100644 (file)
@@ -112,7 +112,7 @@ static DerivedMesh *applyModifier(ModifierData *md, struct Object *ob,
        const bool do_clamp = !(bmd->flags & MOD_BEVEL_OVERLAP_OK);
        const int offset_type = bmd->val_flags;
        const int mat = CLAMPIS(bmd->mat, -1, ob->totcol - 1);
-       const bool loop_slide = (bmd->flags & MOD_BEVEL_LOOP_SLIDE) != 0;
+       const bool loop_slide = (bmd->flags & MOD_BEVEL_EVEN_WIDTHS) == 0;
 
        bm = DM_to_bmesh(dm, true);
        if ((bmd->lim_flags & MOD_BEVEL_VGROUP) && bmd->defgrp_name[0])