forestia/bevy
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

make ScheduleGraph::initialize public (#7723)

Browse Source 
follow-up to https://github.com/bevyengine/bevy/pull/7716

# Objective

System access is only populated in `System::initialize`, so without calling `initialize` it's actually impossible to see most ambiguities.

## Solution


- make `initialize` public. The method is idempotent, so calling it multiple times doesn't hurt
This commit is contained in:
Jakob Hellermann 2023-02-17 20:25:28 +00:00
parent b24ed8bb0c
commit b2e1694c12
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
crates/bevy_ecs/src/schedule/schedule.rs
View File

@ -216,6 +216,8 @@ impl Schedule {
/// Initializes any newly-added systems and conditions, rebuilds the executable schedule, /// Initializes any newly-added systems and conditions, rebuilds the executable schedule,
/// and re-initializes the executor. /// and re-initializes the executor.
///
/// Moves all systems and run conditions out of the [`ScheduleGraph`].
pub fn initialize(&mut self, world: &mut World) -> Result<(), ScheduleBuildError> { pub fn initialize(&mut self, world: &mut World) -> Result<(), ScheduleBuildError> {
if self.graph.changed { if self.graph.changed {
self.graph.initialize(world); self.graph.initialize(world);
@ -772,7 +774,8 @@ impl ScheduleGraph {
Ok(()) Ok(())
} }
fn initialize(&mut self, world: &mut World) { /// Initializes any newly-added systems and conditions by calling [`System::initialize`]
pub fn initialize(&mut self, world: &mut World) {
for (id, i) in self.uninit.drain(..) { for (id, i) in self.uninit.drain(..) {
match id { match id {
NodeId::System(index) => { NodeId::System(index) => {