[Bf-blender-cvs] [60d6cc8fae6] blender2.8: Depsgraph: Make a note about why is it OK to pass original base in relations builder

Sergey Sharybin noreply at git.blender.org
Thu Nov 30 12:47:36 CET 2017


Commit: 60d6cc8fae6a1a356acf78cc6f3379d2f55f8e91
Author: Sergey Sharybin
Date:   Thu Nov 30 12:24:57 2017 +0100
Branches: blender2.8
https://developer.blender.org/rB60d6cc8fae6a1a356acf78cc6f3379d2f55f8e91

Depsgraph: Make a note about why is it OK to pass original base in relations builder

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

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

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

diff --git a/source/blender/depsgraph/intern/builder/deg_builder_relations_view_layer.cc b/source/blender/depsgraph/intern/builder/deg_builder_relations_view_layer.cc
index 2058fd37716..fe7ccf6a24c 100644
--- a/source/blender/depsgraph/intern/builder/deg_builder_relations_view_layer.cc
+++ b/source/blender/depsgraph/intern/builder/deg_builder_relations_view_layer.cc
@@ -73,6 +73,10 @@ void DepsgraphRelationBuilder::build_view_layer(Scene *scene, ViewLayer *view_la
 	/* Setup currently building context. */
 	scene_ = scene;
 	/* Scene objects. */
+	/* NOTE: Nodes builder requires us to pass CoW base because it's being
+	 * passed to the evaluation functions. During relations builder we only
+	 * do NULL-pointer check of the base, so it's fine to pass original one.
+	 */
 	LINKLIST_FOREACH(Base *, base, &view_layer->object_bases) {
 		build_object(base, base->object);
 	}



More information about the Bf-blender-cvs mailing list