[Bf-blender-cvs] [19782e4f680] master: Fix missing autosmooth update when animated
Sergey Sharybin
noreply at git.blender.org
Fri Jun 7 09:14:23 CEST 2019
Commit: 19782e4f680e7db86a72cbfa78d23afd7e7c93ce
Author: Sergey Sharybin
Date: Thu Jun 6 12:27:28 2019 +0200
Branches: master
https://developer.blender.org/rB19782e4f680e7db86a72cbfa78d23afd7e7c93ce
Fix missing autosmooth update when animated
Consists of few simple steps.
Don't tag object data's bounding box as dirty before re-evaluaiton
Most of the time this actually tags evaluated mesh which will be tossed
away few lines below anyway. And this causes issues if the evaluated
mesh is actually owned by the mesh datablock.
The bounding box now has clear separation between original object data
and object data after modifiers, so this should not be causing any
issues.
Free evaluated mesh when owner mesh changes
Ensures that evaluated mesh shares the same settings as its owner when
updates related on animation system happens.
Depsgraph: Update mesh when its geometry settings changes
Allows to have options like autosmooth animated.
Reviewers: brecht, mont29
Differential Revision: https://developer.blender.org/D5030
===================================================================
M source/blender/depsgraph/intern/builder/deg_builder_rna.cc
===================================================================
diff --git a/source/blender/depsgraph/intern/builder/deg_builder_rna.cc b/source/blender/depsgraph/intern/builder/deg_builder_rna.cc
index 5c7fa98ccf1..dc408351a1a 100644
--- a/source/blender/depsgraph/intern/builder/deg_builder_rna.cc
+++ b/source/blender/depsgraph/intern/builder/deg_builder_rna.cc
@@ -277,7 +277,7 @@ RNANodeIdentifier RNANodeQuery::construct_node_identifier(const PointerRNA *ptr,
return node_identifier;
}
}
- else if (RNA_struct_is_a(ptr->type, &RNA_Modifier) ||
+ else if (RNA_struct_is_a(ptr->type, &RNA_Mesh) || RNA_struct_is_a(ptr->type, &RNA_Modifier) ||
RNA_struct_is_a(ptr->type, &RNA_GpencilModifier) ||
RNA_struct_is_a(ptr->type, &RNA_Spline)) {
/* When modifier is used as FROM operation this is likely referencing to
More information about the Bf-blender-cvs
mailing list