Depsgraph: Cleanup, promote is_evaluating query
authorSergey Sharybin <sergey.vfx@gmail.com>
Thu, 10 Oct 2019 09:03:58 +0000 (11:03 +0200)
committerSergey Sharybin <sergey.vfx@gmail.com>
Fri, 11 Oct 2019 08:42:45 +0000 (10:42 +0200)
This way it might be used for sanity checks in RNA API as well.

source/blender/blenkernel/intern/DerivedMesh.c
source/blender/depsgraph/DEG_depsgraph.h
source/blender/depsgraph/intern/depsgraph.cc
source/blender/depsgraph/intern/depsgraph.h
source/blender/depsgraph/intern/depsgraph_debug.cc
source/blender/depsgraph/intern/eval/deg_eval.cc

index 4849d63149378297050d22c46c1bd07eea008fc3..2f61cfcbc156cac6da3180e870febcfacc06c3bc 100644 (file)
@@ -1952,7 +1952,7 @@ Mesh *mesh_get_eval_final(struct Depsgraph *depsgraph,
                           const CustomData_MeshMasks *dataMask)
 {
   /* This function isn't thread-safe and can't be used during evaluation. */
-  BLI_assert(DEG_debug_is_evaluating(depsgraph) == false);
+  BLI_assert(DEG_is_evaluating(depsgraph) == false);
 
   /* Evaluated meshes aren't supposed to be created on original instances. If you do,
    * they aren't cleaned up properly on mode switch, causing crashes, e.g T58150. */
@@ -1985,7 +1985,7 @@ Mesh *mesh_get_eval_deform(struct Depsgraph *depsgraph,
                            const CustomData_MeshMasks *dataMask)
 {
   /* This function isn't thread-safe and can't be used during evaluation. */
-  BLI_assert(DEG_debug_is_evaluating(depsgraph) == false);
+  BLI_assert(DEG_is_evaluating(depsgraph) == false);
 
   /* Evaluated meshes aren't supposed to be created on original instances. If you do,
    * they aren't cleaned up properly on mode switch, causing crashes, e.g T58150. */
index e44dddbcf542eac8790e5c98873651d2b5048ec8..d5a93d21b99c94a6a2dcce15e6c8bbdc2f9734d9 100644 (file)
@@ -179,14 +179,14 @@ void DEG_editors_set_update_cb(DEG_EditorUpdateIDCb id_func, DEG_EditorUpdateSce
 
 /* Evaluation  ----------------------------------- */
 
+bool DEG_is_evaluating(struct Depsgraph *depsgraph);
+
 bool DEG_is_active(const struct Depsgraph *depsgraph);
 void DEG_make_active(struct Depsgraph *depsgraph);
 void DEG_make_inactive(struct Depsgraph *depsgraph);
 
 /* Evaluation Debug ------------------------------ */
 
-bool DEG_debug_is_evaluating(struct Depsgraph *depsgraph);
-
 void DEG_debug_print_begin(struct Depsgraph *depsgraph);
 
 void DEG_debug_print_eval(struct Depsgraph *depsgraph,
index dcdea87fe1a10f032fa34d37e7f75b9de092923b..8f9595d2476ded9e2bee6b4e0b553e6b6362e561 100644 (file)
@@ -76,7 +76,7 @@ Depsgraph::Depsgraph(Main *bmain, Scene *scene, ViewLayer *view_layer, eEvaluati
       ctime(BKE_scene_frame_get(scene)),
       scene_cow(NULL),
       is_active(false),
-      debug_is_evaluating(false),
+      is_evaluating(false),
       is_render_pipeline_depsgraph(false)
 {
   BLI_spin_init(&lock);
@@ -334,6 +334,12 @@ void DEG_graph_free(Depsgraph *graph)
   OBJECT_GUARDED_DELETE(deg_depsgraph, Depsgraph);
 }
 
+bool DEG_is_evaluating(struct Depsgraph *depsgraph)
+{
+  DEG::Depsgraph *deg_graph = reinterpret_cast<DEG::Depsgraph *>(depsgraph);
+  return deg_graph->is_evaluating;
+}
+
 bool DEG_is_active(const struct Depsgraph *depsgraph)
 {
   if (depsgraph == NULL) {
index 30ae4edde34e78f2f3cb2b4628c797e93ff3c3a2..43829f4e045ac3af24225121809c7f8845a7d012 100644 (file)
@@ -198,7 +198,7 @@ struct Depsgraph {
   int debug_flags;
   string debug_name;
 
-  bool debug_is_evaluating;
+  bool is_evaluating;
 
   /* Is set to truth for dependency graph which are used for post-processing (compositor and
    * sequencer).
index d079c958e04f1e53164cbb039fed9c4325ef3fb7..bb60db5209cb7af43a251e6a88b59ea170e60885 100644 (file)
@@ -246,12 +246,6 @@ void DEG_stats_simple(const Depsgraph *graph,
   }
 }
 
-bool DEG_debug_is_evaluating(struct Depsgraph *depsgraph)
-{
-  DEG::Depsgraph *deg_graph = reinterpret_cast<DEG::Depsgraph *>(depsgraph);
-  return deg_graph->debug_is_evaluating;
-}
-
 static DEG::string depsgraph_name_for_logging(struct Depsgraph *depsgraph)
 {
   const char *name = DEG_debug_name_get(depsgraph);
index 6b7d102c67541242859002c8429d033342c5f094..d6b3c54a149585608b28d62d254f566ae7198101 100644 (file)
@@ -257,7 +257,7 @@ void deg_evaluate_on_refresh(Depsgraph *graph)
   }
   const bool do_time_debug = ((G.debug & G_DEBUG_DEPSGRAPH_TIME) != 0);
   const double start_time = do_time_debug ? PIL_check_seconds_timer() : 0;
-  graph->debug_is_evaluating = true;
+  graph->is_evaluating = true;
   depsgraph_ensure_view_layer(graph);
   /* Set up evaluation state. */
   DepsgraphEvalState state;
@@ -298,7 +298,7 @@ void deg_evaluate_on_refresh(Depsgraph *graph)
   if (need_free_scheduler) {
     BLI_task_scheduler_free(task_scheduler);
   }
-  graph->debug_is_evaluating = false;
+  graph->is_evaluating = false;
   if (do_time_debug) {
     printf("Depsgraph updated in %f seconds.\n", PIL_check_seconds_timer() - start_time);
   }