[Bf-blender-cvs] [25e4faa] depsgraph_refactor: Depsgraph: Fix/workaround for a drivers using lamp settings as variables

Sergey Sharybin noreply at git.blender.org
Fri Feb 27 13:49:46 CET 2015


Commit: 25e4faabbd55acb5c03ffd2dcbc1d9f833bd366a
Author: Sergey Sharybin
Date:   Fri Feb 27 17:48:01 2015 +0500
Branches: depsgraph_refactor
https://developer.blender.org/rB25e4faabbd55acb5c03ffd2dcbc1d9f833bd366a

Depsgraph: Fix/workaround for a drivers using lamp settings as variables

Not sure what's the designed way dealing with this, so if Joshua can clarify this a
bit that'd be really nice :)

For now it's so called "seems to work in the test file", added a TODO in the code so
we don't forget to doublecheck on this before merge.

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

M	source/blender/depsgraph/intern/depsgraph_build_nodes.cpp

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

diff --git a/source/blender/depsgraph/intern/depsgraph_build_nodes.cpp b/source/blender/depsgraph/intern/depsgraph_build_nodes.cpp
index 8f9e837..111b5ad 100644
--- a/source/blender/depsgraph/intern/depsgraph_build_nodes.cpp
+++ b/source/blender/depsgraph/intern/depsgraph_build_nodes.cpp
@@ -876,6 +876,10 @@ void DepsgraphNodeBuilder::build_lamp(Object *ob)
 	/* node for obdata */
 	ComponentDepsNode *param_node = add_component_node(lamp_id, DEPSNODE_TYPE_PARAMETERS);
 
+	/* TODO(sergey): Is it really how we're supposed to work with drivers? */
+	add_operation_node(lamp_id, DEPSNODE_TYPE_PARAMETERS, DEPSOP_TYPE_EXEC, NULL,
+	                   DEG_OPCODE_PLACEHOLDER, "Parameters Eval");
+
 	/* lamp's nodetree */
 	if (la->nodetree) {
 		build_nodetree(param_node, la->nodetree);




More information about the Bf-blender-cvs mailing list