[Bf-blender-cvs] [1b371d199eb] blender2.8: Depsgraph: Ensure dependency graph is allocated for evaluation context

Sergey Sharybin noreply at git.blender.org
Wed Dec 13 12:55:42 CET 2017


Commit: 1b371d199eb0846fb2abd50477690d3e0bff0119
Author: Sergey Sharybin
Date:   Wed Dec 13 12:38:31 2017 +0100
Branches: blender2.8
https://developer.blender.org/rB1b371d199eb0846fb2abd50477690d3e0bff0119

Depsgraph: Ensure dependency graph is allocated for evaluation context

This is something what we would need to ensure anyway, so doesn't seem
to make sense to NOT allocate depsgraph and then worry about this externally.

Steps to reproduce: add cube, change it's size in redo panel.

Found by Campbell during code review session.

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

M	source/blender/depsgraph/intern/depsgraph_eval.cc

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

diff --git a/source/blender/depsgraph/intern/depsgraph_eval.cc b/source/blender/depsgraph/intern/depsgraph_eval.cc
index 1d389b902b8..2ed403c463a 100644
--- a/source/blender/depsgraph/intern/depsgraph_eval.cc
+++ b/source/blender/depsgraph/intern/depsgraph_eval.cc
@@ -85,11 +85,10 @@ void DEG_evaluation_context_init_from_scene(EvaluationContext *eval_ctx,
                                             eEvaluationMode mode)
 {
 	DEG_evaluation_context_init(eval_ctx, mode);
-	eval_ctx->depsgraph = BKE_scene_get_depsgraph(scene, view_layer, false);
+	eval_ctx->depsgraph = BKE_scene_get_depsgraph(scene, view_layer, true);
 	eval_ctx->view_layer = view_layer;
 	eval_ctx->engine_type = engine_type;
 	eval_ctx->ctime = BKE_scene_frame_get(scene);
-	BLI_assert(eval_ctx->depsgraph != NULL);
 }
 
 /* Free evaluation context. */



More information about the Bf-blender-cvs mailing list