[Bf-blender-cvs] [f6b0bf1ec24] soc-2019-bevel-profiles: Bevel modifer: Add isDisabled function

Hans Goudey noreply at git.blender.org
Tue Aug 6 19:54:37 CEST 2019


Commit: f6b0bf1ec2487b48bac02c50a98dacca952a9e24
Author: Hans Goudey
Date:   Tue Aug 6 13:54:22 2019 -0400
Branches: soc-2019-bevel-profiles
https://developer.blender.org/rBf6b0bf1ec2487b48bac02c50a98dacca952a9e24

Bevel modifer: Add isDisabled function

When the width of the bevel is zero the modifier doesn't do anything
so it should be disabled.

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

M	source/blender/modifiers/intern/MOD_bevel.c

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

diff --git a/source/blender/modifiers/intern/MOD_bevel.c b/source/blender/modifiers/intern/MOD_bevel.c
index a9a75fbd928..fe403c9eccf 100644
--- a/source/blender/modifiers/intern/MOD_bevel.c
+++ b/source/blender/modifiers/intern/MOD_bevel.c
@@ -73,9 +73,7 @@ static void copyData(const ModifierData *md_src, ModifierData *md_dst, const int
   BevelModifierData *bmd_dst = (BevelModifierData *)md_dst;
 
   modifier_copyData_generic(md_src, md_dst, flag);
-//  printf("(src prwdgt: %p)", (const void *)bmd_src);
   bmd_dst->prwdgt = profilewidget_copy(bmd_src->prwdgt);
-//  printf("(dst prwdgt: %p)\n", (void *)bmd_dst);
 }
 
 static void requiredDataMask(Object *UNUSED(ob),
@@ -248,6 +246,12 @@ static void freeData(ModifierData *md)
   profilewidget_free(bmd->prwdgt);
 }
 
+static bool isDisabled(const Scene *UNUSED(scene), ModifierData *md, bool UNUSED(userRenderParams))
+{
+  BevelModifierData *bmd = (BevelModifierData *)md;
+  return (bmd->value == 0.0f);
+}
+
 ModifierTypeInfo modifierType_Bevel = {
     /* name */ "Bevel",
     /* structName */ "BevelModifierData",
@@ -264,7 +268,7 @@ ModifierTypeInfo modifierType_Bevel = {
     /* initData */ initData,
     /* requiredDataMask */ requiredDataMask,
     /* freeData */ freeData,
-    /* isDisabled */ NULL,
+    /* isDisabled */ isDisabled,
     /* updateDepsgraph */ NULL,
     /* dependsOnTime */ NULL,
     /* dependsOnNormals */ dependsOnNormals,



More information about the Bf-blender-cvs mailing list