[Bf-blender-cvs] [b4fb1e6] master: Bevel Modifier: fix bug T37916, bad result after mirror modifier.

Howard Trickey noreply at git.blender.org
Fri Feb 7 16:50:19 CET 2014


Commit: b4fb1e6d9f318aa7bed6af8902dd839c0ce76948
Author: Howard Trickey
Date:   Fri Feb 7 10:43:28 2014 -0500
https://developer.blender.org/rBb4fb1e6d9f318aa7bed6af8902dd839c0ce76948

Bevel Modifier: fix bug T37916, bad result after mirror modifier.

Bevel needs the vertex normals to be correct and they are not
normally recalculated after a previous modifier, like mirror.
Adding 'dependsOnNormals' -> returns true to modifier info
for Bevel fixes the problem.

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

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 2a6c594..e5e7e00 100644
--- a/source/blender/modifiers/intern/MOD_bevel.c
+++ b/source/blender/modifiers/intern/MOD_bevel.c
@@ -183,6 +183,11 @@ static DerivedMesh *applyModifier(ModifierData *md, struct Object *ob,
 	return result;
 }
 
+static bool dependsOnNormals(ModifierData *UNUSED(md))
+{
+	return true;
+}
+
 ModifierTypeInfo modifierType_Bevel = {
 	/* name */              "Bevel",
 	/* structName */        "BevelModifierData",
@@ -205,7 +210,7 @@ ModifierTypeInfo modifierType_Bevel = {
 	/* isDisabled */        NULL,
 	/* updateDepgraph */    NULL,
 	/* dependsOnTime */     NULL,
-	/* dependsOnNormals */  NULL,
+	/* dependsOnNormals */  dependsOnNormals,
 	/* foreachObjectLink */ NULL,
 	/* foreachIDLink */     NULL,
 	/* foreachTexLink */    NULL,




More information about the Bf-blender-cvs mailing list