[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