Skip to content

Commit 9ff868c

Browse files
committed
test test test
1 parent a935826 commit 9ff868c

File tree

3 files changed

+37
-1
lines changed

3 files changed

+37
-1
lines changed

tests/ui/needless_return.fixed

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -277,4 +277,14 @@ fn issue9947() -> Result<(), String> {
277277
do yeet "hello";
278278
}
279279

280+
// without anyhow, but triggers the same bug I believe
281+
#[expect(clippy::useless_format)]
282+
fn issue10051() -> Result<String, String> {
283+
if true {
284+
Ok(format!("ok!"))
285+
} else {
286+
Err(format!("err!"))
287+
}
288+
}
289+
280290
fn main() {}

tests/ui/needless_return.rs

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -287,4 +287,14 @@ fn issue9947() -> Result<(), String> {
287287
do yeet "hello";
288288
}
289289

290+
// without anyhow, but triggers the same bug I believe
291+
#[expect(clippy::useless_format)]
292+
fn issue10051() -> Result<String, String> {
293+
if true {
294+
return Ok(format!("ok!"));
295+
} else {
296+
return Err(format!("err!"));
297+
}
298+
}
299+
290300
fn main() {}

tests/ui/needless_return.stderr

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -386,5 +386,21 @@ LL | let _ = 42; return;
386386
|
387387
= help: remove `return`
388388

389-
error: aborting due to 46 previous errors
389+
error: unneeded `return` statement
390+
--> $DIR/needless_return.rs:294:9
391+
|
392+
LL | return Ok(format!("ok!"));
393+
| ^^^^^^^^^^^^^^^^^^^^^^^^^
394+
|
395+
= help: remove `return`
396+
397+
error: unneeded `return` statement
398+
--> $DIR/needless_return.rs:296:9
399+
|
400+
LL | return Err(format!("err!"));
401+
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
402+
|
403+
= help: remove `return`
404+
405+
error: aborting due to 48 previous errors
390406

0 commit comments

Comments
 (0)