[Bf-blender-cvs] [edded659c69] blender-v2.79-release: Fix T51907: New Depsgraph - Camera constraint is not evaluated properly

Sergey Sharybin noreply at git.blender.org
Mon Sep 4 14:18:56 CEST 2017


Commit: edded659c6902ac729e585971614f2efdd1eaeb4
Author: Sergey Sharybin
Date:   Thu Aug 24 14:35:48 2017 +0200
Branches: blender-v2.79-release
https://developer.blender.org/rBedded659c6902ac729e585971614f2efdd1eaeb4

Fix T51907: New Depsgraph - Camera constraint is not evaluated properly

This is more a workaround for until we've got proper visibility flush, which
will likely happen in blender2.8 branch.

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

M	source/blender/depsgraph/intern/builder/deg_builder_nodes.cc

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

diff --git a/source/blender/depsgraph/intern/builder/deg_builder_nodes.cc b/source/blender/depsgraph/intern/builder/deg_builder_nodes.cc
index a90f8ff02b6..2d275df1306 100644
--- a/source/blender/depsgraph/intern/builder/deg_builder_nodes.cc
+++ b/source/blender/depsgraph/intern/builder/deg_builder_nodes.cc
@@ -320,8 +320,11 @@ void DepsgraphNodeBuilder::build_object(Scene *scene, Base *base, Object *ob)
 	if (base != NULL) {
 		id_node->layers |= base->lay;
 	}
-	if (ob == scene->camera) {
-		/* Camera should always be updated, it used directly by viewport. */
+	if (ob->type == OB_CAMERA) {
+		/* Camera should always be updated, it used directly by viewport.
+		 *
+		 * TODO(sergey): Make it only for active scene camera.
+		 */
 		id_node->layers |= (unsigned int)(-1);
 	}
 	/* Skip rest of components if the ID node was already there. */



More information about the Bf-blender-cvs mailing list