bevy_log: scope print_stderr expectation to the eprintln call (#24191)

mockersf · web-flow · commit c78b1a401fa8 · 2026-05-08T13:26:23.000Z
# Objective

- `cargo clippy -p bevy_log  --no-deps -- -D warnings`
- print is only used if the trace feature is on

## Solution

- scope the expect to the print

## Testing

- CI
diff --git a/crates/bevy_log/src/lib.rs b/crates/bevy_log/src/lib.rs
@@ -293,13 +293,15 @@ impl Default for LogPlugin {
 }
 
 impl Plugin for LogPlugin {
-    #[expect(clippy::print_stderr, reason = "Allowed during logger setup")]
     fn build(&self, app: &mut App) {
         #[cfg(feature = "trace")]
         {
             let old_handler = std::panic::take_hook();
             std::panic::set_hook(Box::new(move |infos| {
-                eprintln!("{}", tracing_error::SpanTrace::capture());
+                #[expect(clippy::print_stderr, reason = "Allowed during logger setup")]
+                {
+                    eprintln!("{}", tracing_error::SpanTrace::capture());
+                }
                 old_handler(infos);
             }));
         }

Original file line number	Diff line number	Diff line change
`@@ -293,13 +293,15 @@ impl Default for LogPlugin {`
`293`	`293`	`}`
`294`	`294`
`295`	`295`	`impl Plugin for LogPlugin {`
`296`		`- #[expect(clippy::print_stderr, reason = "Allowed during logger setup")]`
`297`	`296`	`fn build(&self, app: &mut App) {`
`298`	`297`	`#[cfg(feature = "trace")]`
`299`	`298`	`{`
`300`	`299`	`let old_handler = std::panic::take_hook();`
`301`	`300`	`std::panic::set_hook(Box::new(move \|infos\| {`
`302`		`- eprintln!("{}", tracing_error::SpanTrace::capture());`
	`301`	`+ #[expect(clippy::print_stderr, reason = "Allowed during logger setup")]`
	`302`	`+ {`
	`303`	`+ eprintln!("{}", tracing_error::SpanTrace::capture());`
	`304`	`+ }`
`303`	`305`	`old_handler(infos);`
`304`	`306`	`}));`
`305`	`307`	`}`