move final apply outside of spawned executor

bevyengine · Jan 31, 2023 · 50cbdd0 · 50cbdd0
1 parent 5d514fb
commit 50cbdd0
Showing 1 changed file with 13 additions and 12 deletions.
diff --git a/crates/bevy_ecs/src/schedule_v3/executor/multi_threaded.rs b/crates/bevy_ecs/src/schedule_v3/executor/multi_threaded.rs
@@ -178,18 +178,6 @@ impl SystemExecutor for MultiThreadedExecutor {
                         self.rebuild_active_access();
                     }
                 }
-
-                // SAFETY: all systems have completed
-                let world = unsafe { &mut *world.get() };
-                apply_system_buffers(&mut self.unapplied_systems, systems, world);
-
-                debug_assert!(self.ready_systems.is_clear());
-                debug_assert!(self.running_systems.is_clear());
-                debug_assert!(self.unapplied_systems.is_clear());
-                self.active_access.clear();
-                self.evaluated_sets.clear();
-                self.skipped_systems.clear();
-                self.completed_systems.clear();
             };
 
             #[cfg(feature = "trace")]
@@ -198,6 +186,19 @@ impl SystemExecutor for MultiThreadedExecutor {
             let executor = executor.instrument(executor_span);
             scope.spawn(executor);
         });
+
+        // Do one final apply buffers after all systems have completed
+        // SAFETY: all systems have completed, and so no outstanding accesses remain
+        let world = unsafe { &mut *world.get() };
+        apply_system_buffers(&mut self.unapplied_systems, systems, world);
+
+        debug_assert!(self.ready_systems.is_clear());
+        debug_assert!(self.running_systems.is_clear());
+        debug_assert!(self.unapplied_systems.is_clear());
+        self.active_access.clear();
+        self.evaluated_sets.clear();
+        self.skipped_systems.clear();
+        self.completed_systems.clear();
     }
 }