Skip to content

Commit a4dd873

Browse files
authored
Fix uninlined format argument lint (#19561)
# Objective Fixes #19370 ## Solution Implemented the lint as suggested by clippy. ## Testing CI tests passed and lint was resolved.
1 parent 1294b71 commit a4dd873

File tree

2 files changed

+9
-15
lines changed

2 files changed

+9
-15
lines changed

crates/bevy_ecs/src/schedule/schedule.rs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ impl Schedules {
166166
writeln!(message, "{}", components.get_name(*id).unwrap()).unwrap();
167167
}
168168

169-
info!("{}", message);
169+
info!("{message}");
170170
}
171171

172172
/// Adds one or more systems to the [`Schedule`] matching the provided [`ScheduleLabel`].
@@ -1705,10 +1705,7 @@ impl ScheduleGraph {
17051705
match self.settings.hierarchy_detection {
17061706
LogLevel::Ignore => unreachable!(),
17071707
LogLevel::Warn => {
1708-
error!(
1709-
"Schedule {schedule_label:?} has redundant edges:\n {}",
1710-
message
1711-
);
1708+
error!("Schedule {schedule_label:?} has redundant edges:\n {message}");
17121709
Ok(())
17131710
}
17141711
LogLevel::Error => Err(ScheduleBuildError::HierarchyRedundancy(message)),
@@ -1910,7 +1907,7 @@ impl ScheduleGraph {
19101907
match self.settings.ambiguity_detection {
19111908
LogLevel::Ignore => Ok(()),
19121909
LogLevel::Warn => {
1913-
warn!("Schedule {schedule_label:?} has ambiguities.\n{}", message);
1910+
warn!("Schedule {schedule_label:?} has ambiguities.\n{message}");
19141911
Ok(())
19151912
}
19161913
LogLevel::Error => Err(ScheduleBuildError::Ambiguity(message)),

crates/bevy_ecs/src/schedule/stepping.rs

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -475,9 +475,8 @@ impl Stepping {
475475
Some(state) => state.clear_behaviors(),
476476
None => {
477477
warn!(
478-
"stepping is not enabled for schedule {:?}; \
479-
use `.add_stepping({:?})` to enable stepping",
480-
label, label
478+
"stepping is not enabled for schedule {label:?}; \
479+
use `.add_stepping({label:?})` to enable stepping"
481480
);
482481
}
483482
},
@@ -486,9 +485,8 @@ impl Stepping {
486485
Some(state) => state.set_behavior(system, behavior),
487486
None => {
488487
warn!(
489-
"stepping is not enabled for schedule {:?}; \
490-
use `.add_stepping({:?})` to enable stepping",
491-
label, label
488+
"stepping is not enabled for schedule {label:?}; \
489+
use `.add_stepping({label:?})` to enable stepping"
492490
);
493491
}
494492
}
@@ -498,9 +496,8 @@ impl Stepping {
498496
Some(state) => state.clear_behavior(system),
499497
None => {
500498
warn!(
501-
"stepping is not enabled for schedule {:?}; \
502-
use `.add_stepping({:?})` to enable stepping",
503-
label, label
499+
"stepping is not enabled for schedule {label:?}; \
500+
use `.add_stepping({label:?})` to enable stepping"
504501
);
505502
}
506503
}

0 commit comments

Comments
 (0)