Skip to content

Commit cceedb7

Browse files
Rename ReportExecutionOrderAmbiguities to ExecutionOrderAmbiguities
1 parent 98d6d8a commit cceedb7

File tree

6 files changed

+44
-44
lines changed

6 files changed

+44
-44
lines changed

crates/bevy_ecs/src/lib.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ pub mod prelude {
2828
event::{EventReader, EventWriter},
2929
query::{Added, AnyOf, ChangeTrackers, Changed, Or, QueryState, With, Without},
3030
schedule::{
31-
AmbiguitySetLabel, ExclusiveSystemDescriptorCoercion, ParallelSystemDescriptorCoercion,
32-
ReportExecutionOrderAmbiguities, RunCriteria, RunCriteriaDescriptorCoercion,
31+
AmbiguitySetLabel, ExclusiveSystemDescriptorCoercion, ExecutionOrderAmbiguities,
32+
ParallelSystemDescriptorCoercion, RunCriteria, RunCriteriaDescriptorCoercion,
3333
RunCriteriaLabel, RunCriteriaPiping, Schedule, Stage, StageLabel, State, SystemLabel,
3434
SystemSet, SystemStage,
3535
},

crates/bevy_ecs/src/schedule/ambiguity_detection.rs

Lines changed: 32 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ use fixedbitset::FixedBitSet;
3131
/// App::new()
3232
/// .insert_resource(ReportExecutionOrderAmbiguities::verbose().ignore(&["my_external_crate"]));
3333
/// ```
34-
pub enum ReportExecutionOrderAmbiguities {
34+
pub enum ExecutionOrderAmbiguities {
3535
/// Disables all checks for execution order ambiguities
3636
Allow,
3737
/// Displays only the number of unresolved ambiguities detected by the ambiguity checker
@@ -164,16 +164,16 @@ pub fn find_ambiguities(
164164
systems: &[impl SystemContainer],
165165
crates_filter: &[String],
166166
// Should explicit attempts to ignore ambiguities be obeyed?
167-
report_level: ReportExecutionOrderAmbiguities,
167+
report_level: ExecutionOrderAmbiguities,
168168
) -> Vec<(usize, usize, Vec<ComponentId>)> {
169169
fn should_ignore_ambiguity(
170170
systems: &[impl SystemContainer],
171171
index_a: usize,
172172
index_b: usize,
173173
crates_filter: &[String],
174-
report_level: ReportExecutionOrderAmbiguities,
174+
report_level: ExecutionOrderAmbiguities,
175175
) -> bool {
176-
if report_level == ReportExecutionOrderAmbiguities::Forbid {
176+
if report_level == ExecutionOrderAmbiguities::Forbid {
177177
return false;
178178
}
179179

@@ -259,9 +259,9 @@ pub fn find_ambiguities(
259259
ambiguities
260260
}
261261

262-
impl Default for ReportExecutionOrderAmbiguities {
262+
impl Default for ExecutionOrderAmbiguities {
263263
fn default() -> Self {
264-
ReportExecutionOrderAmbiguities::Warn
264+
ExecutionOrderAmbiguities::Warn
265265
}
266266
}
267267

@@ -278,19 +278,19 @@ impl SystemStage {
278278
// FIXME: these methods should not have tor rely on &mut World, or any specific World
279279
// see https://github.com/bevyengine/bevy/issues/4364
280280
world: &mut World,
281-
report_level: ReportExecutionOrderAmbiguities,
281+
report_level: ExecutionOrderAmbiguities,
282282
) -> Vec<SystemOrderAmbiguity> {
283283
self.initialize(world);
284284

285-
if report_level == ReportExecutionOrderAmbiguities::Allow {
285+
if report_level == ExecutionOrderAmbiguities::Allow {
286286
return Vec::default();
287287
}
288288

289289
// System order must be fresh
290290
debug_assert!(!self.systems_modified);
291291

292292
// TODO: remove all internal ambiguities and remove this logic
293-
let ignored_crates = if report_level != ReportExecutionOrderAmbiguities::WarnInternal {
293+
let ignored_crates = if report_level != ExecutionOrderAmbiguities::WarnInternal {
294294
vec![
295295
// Rendering
296296
"bevy_render".to_string(),
@@ -385,7 +385,7 @@ impl SystemStage {
385385
pub fn n_ambiguities(
386386
&mut self,
387387
world: &mut World,
388-
report_level: ReportExecutionOrderAmbiguities,
388+
report_level: ExecutionOrderAmbiguities,
389389
) -> usize {
390390
self.ambiguities(world, report_level).len()
391391
}
@@ -394,7 +394,7 @@ impl SystemStage {
394394
pub fn report_ambiguities(
395395
&mut self,
396396
world: &mut World,
397-
report_level: ReportExecutionOrderAmbiguities,
397+
report_level: ExecutionOrderAmbiguities,
398398
) {
399399
let ambiguities = self.ambiguities(world, report_level);
400400
let unresolved_count = ambiguities.len();
@@ -409,7 +409,7 @@ impl SystemStage {
409409
You may want to add `.before()` or `.after()` ordering constraints between some of these systems to prevent bugs.\n");
410410
}
411411

412-
if report_level == ReportExecutionOrderAmbiguities::Warn {
412+
if report_level == ExecutionOrderAmbiguities::Warn {
413413
println!("Set the level of the `ReportExecutionOrderAmbiguities` resource to `AmbiguityReportLevel::Verbose` for more details.");
414414
} else {
415415
for (i, ambiguity) in ambiguities.iter().enumerate() {
@@ -434,8 +434,8 @@ impl SystemStage {
434434
println!("");
435435
}
436436

437-
if report_level == ReportExecutionOrderAmbiguities::Deny
438-
|| report_level == ReportExecutionOrderAmbiguities::Forbid
437+
if report_level == ExecutionOrderAmbiguities::Deny
438+
|| report_level == ExecutionOrderAmbiguities::Forbid
439439
{
440440
panic!("The ReportExecutionOrderAmbiguities resource is set to a level that forbids the app from running with unresolved system execution order ambiguities.")
441441
}
@@ -594,7 +594,7 @@ mod tests {
594594
.add_system(event_writer_system);
595595

596596
assert_eq!(
597-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
597+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
598598
0
599599
);
600600
}
@@ -619,7 +619,7 @@ mod tests {
619619
.add_system(read_world_system);
620620

621621
assert_eq!(
622-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
622+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
623623
0
624624
);
625625
}
@@ -637,7 +637,7 @@ mod tests {
637637
.add_system(read_world_system);
638638

639639
assert_eq!(
640-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
640+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
641641
3
642642
);
643643
}
@@ -650,7 +650,7 @@ mod tests {
650650
test_stage.add_system(resmut_system).add_system(res_system);
651651

652652
assert_eq!(
653-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
653+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
654654
1
655655
);
656656
}
@@ -665,7 +665,7 @@ mod tests {
665665
.add_system(nonsend_system);
666666

667667
assert_eq!(
668-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
668+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
669669
1
670670
);
671671
}
@@ -680,7 +680,7 @@ mod tests {
680680
.add_system(write_component_system);
681681

682682
assert_eq!(
683-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
683+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
684684
1
685685
);
686686
}
@@ -695,7 +695,7 @@ mod tests {
695695
.add_system(without_filtered_component_system);
696696

697697
assert_eq!(
698-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
698+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
699699
0
700700
);
701701
}
@@ -712,7 +712,7 @@ mod tests {
712712
.add_system(event_resource_system);
713713

714714
assert_eq!(
715-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
715+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
716716
3
717717
);
718718
}
@@ -732,7 +732,7 @@ mod tests {
732732
.add_system(write_world_system.exclusive_system().before_commands());
733733

734734
assert_eq!(
735-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
735+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
736736
3
737737
);
738738
}
@@ -749,7 +749,7 @@ mod tests {
749749
.add_system(event_resource_system.after(event_writer_system));
750750

751751
assert_eq!(
752-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
752+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
753753
// All of these systems clash
754754
0
755755
);
@@ -764,7 +764,7 @@ mod tests {
764764
.add_system(res_system);
765765

766766
assert_eq!(
767-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
767+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
768768
0
769769
);
770770
}
@@ -778,7 +778,7 @@ mod tests {
778778
.add_system(res_system.label("IGNORE_ME"));
779779

780780
assert_eq!(
781-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
781+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
782782
0
783783
);
784784
}
@@ -815,7 +815,7 @@ mod tests {
815815
.add_system(system_b);
816816

817817
assert_eq!(
818-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
818+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
819819
0
820820
);
821821
}
@@ -825,7 +825,7 @@ mod tests {
825825
let mut world = World::new();
826826
let mut test_stage = make_test_stage(&mut world);
827827
assert_eq!(
828-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
828+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
829829
0
830830
);
831831
}
@@ -835,7 +835,7 @@ mod tests {
835835
let mut world = World::new();
836836
let mut test_stage = make_test_stage(&mut world);
837837
assert_eq!(
838-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
838+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
839839
2
840840
);
841841
}
@@ -845,7 +845,7 @@ mod tests {
845845
let mut world = World::new();
846846
let mut test_stage = make_test_stage(&mut world);
847847
assert_eq!(
848-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
848+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
849849
2
850850
);
851851
}
@@ -855,7 +855,7 @@ mod tests {
855855
let mut world = World::new();
856856
let mut test_stage = make_test_stage(&mut world);
857857
assert_eq!(
858-
test_stage.n_ambiguities(&mut world, ReportExecutionOrderAmbiguities::Forbid),
858+
test_stage.n_ambiguities(&mut world, ExecutionOrderAmbiguities::Forbid),
859859
6
860860
);
861861
}
@@ -869,7 +869,7 @@ mod tests {
869869
let mut world = World::new();
870870
let mut test_stage = make_test_stage(&mut world);
871871
let ambiguities =
872-
test_stage.ambiguities(&mut world, ReportExecutionOrderAmbiguities::WarnVerbose);
872+
test_stage.ambiguities(&mut world, ExecutionOrderAmbiguities::WarnVerbose);
873873
assert_eq!(
874874
// We don't care if the reported order varies
875875
HashSet::from_iter(ambiguities),

crates/bevy_ecs/src/schedule/stage.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,11 @@ use crate::{
33
schedule::{
44
graph_utils::{self, DependencyGraphError},
55
BoxedRunCriteria, BoxedRunCriteriaLabel, BoxedSystemLabel, DuplicateLabelStrategy,
6-
ExclusiveSystemContainer, GraphNode, InsertionPoint, IntoSystemDescriptor,
7-
ParallelExecutor, ParallelSystemContainer, ParallelSystemExecutor,
8-
ReportExecutionOrderAmbiguities, RunCriteriaContainer, RunCriteriaDescriptor,
9-
RunCriteriaDescriptorOrLabel, RunCriteriaInner, ShouldRun, SingleThreadedExecutor,
10-
SystemContainer, SystemDescriptor, SystemSet,
6+
ExclusiveSystemContainer, ExecutionOrderAmbiguities, GraphNode, InsertionPoint,
7+
IntoSystemDescriptor, ParallelExecutor, ParallelSystemContainer, ParallelSystemExecutor,
8+
RunCriteriaContainer, RunCriteriaDescriptor, RunCriteriaDescriptorOrLabel,
9+
RunCriteriaInner, ShouldRun, SingleThreadedExecutor, SystemContainer, SystemDescriptor,
10+
SystemSet,
1111
},
1212
world::{World, WorldId},
1313
};
@@ -112,7 +112,7 @@ impl SystemStage {
112112
self.executor_modified = false;
113113
// This cannot panic, so we insert the default value if it was not present
114114
let report_level = world
115-
.get_resource_or_insert_with(ReportExecutionOrderAmbiguities::default)
115+
.get_resource_or_insert_with(ExecutionOrderAmbiguities::default)
116116
.clone();
117117
self.report_ambiguities(world, report_level);
118118
} else if self.executor_modified {

examples/ecs/ambiguity_checker.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ fn main() {
66
// This resource allows to control how Ambiguity Checker will report unresolved ambiguities.
77
// By default only a warning with the number of unresolved ambiguities is shown, but
88
// a more complete report will be displayed if we explicitly set this resource to verbose.
9-
.insert_resource(ReportExecutionOrderAmbiguities::WarnVerbose)
9+
.insert_resource(ExecutionOrderAmbiguities::WarnVerbose)
1010
.insert_resource(MyStartupResource(0))
1111
// `startup_system_a` and `startup_system_b` will both compete for the same resource. Since there is no ordering between
1212
// them (e.g., `.before()` or `.after()`), which one will run first is not deterministic.

examples/ecs/ecs_guide.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
use bevy::{
22
app::{AppExit, ScheduleRunnerPlugin, ScheduleRunnerSettings},
3-
ecs::schedule::ReportExecutionOrderAmbiguities,
3+
ecs::schedule::ExecutionOrderAmbiguities,
44
prelude::*,
55
utils::Duration,
66
};
@@ -351,7 +351,7 @@ fn main() {
351351
// in the console by adding the following resource to our App
352352
// Execution order ambiguities can be resolved by adding ordering constraints,
353353
// or explicitly ignored (when there's a false positive).
354-
.insert_resource(ReportExecutionOrderAmbiguities::WarnVerbose)
354+
.insert_resource(ExecutionOrderAmbiguities::WarnVerbose)
355355
// This call to run() starts the app we just built!
356356
.run();
357357
}

examples/tools/internal_ambiguities.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,6 @@ use bevy::prelude::*;
66
fn main() {
77
App::new()
88
.add_plugins(DefaultPlugins)
9-
.insert_resource(ReportExecutionOrderAmbiguities::WarnInternal)
9+
.insert_resource(ExecutionOrderAmbiguities::WarnInternal)
1010
.run()
1111
}

0 commit comments

Comments
 (0)