[Bf-blender-cvs] [b11d0d55cd4] temp-viewport-compositor-compiler: Viewport Compositor: Fix double scheduling of nodes

Omar Emara noreply at git.blender.org
Thu Mar 31 16:22:12 CEST 2022


Commit: b11d0d55cd4d34b35933b536c2fef233d9c3222f
Author: Omar Emara
Date:   Thu Mar 31 13:01:20 2022 +0200
Branches: temp-viewport-compositor-compiler
https://developer.blender.org/rBb11d0d55cd4d34b35933b536c2fef233d9c3222f

Viewport Compositor: Fix double scheduling of nodes

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

M	source/blender/nodes/intern/node_compositor_execute.cc

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

diff --git a/source/blender/nodes/intern/node_compositor_execute.cc b/source/blender/nodes/intern/node_compositor_execute.cc
index 8e31b4c6264..41e01bf18ce 100644
--- a/source/blender/nodes/intern/node_compositor_execute.cc
+++ b/source/blender/nodes/intern/node_compositor_execute.cc
@@ -1519,8 +1519,7 @@ void Scheduler::compute_schedule(DNode node)
     const DInputSocket input{node.context(), input_ref};
     input.foreach_origin_socket([&](const DSocket origin) {
       /* The origin node was added before or was already schedule, so skip it. The number of
-       * origin nodes is very small, so linear search is okay.
-       */
+       * origin nodes is very small, so linear search is okay. */
       if (sorted_origin_nodes.contains(origin.node()) || schedule_.contains(origin.node())) {
         return;
       }
@@ -1543,7 +1542,7 @@ void Scheduler::compute_schedule(DNode node)
     compute_schedule(origin_node);
   }
 
-  schedule_.add_new(node);
+  schedule_.add(node);
 }
 
 /* --------------------------------------------------------------------



More information about the Bf-blender-cvs mailing list