From b33364597105dc60555957a4ae59bb780eb15147 Mon Sep 17 00:00:00 2001 From: kraktus Date: Thu, 22 Sep 2022 16:44:21 +0200 Subject: [PATCH] Add test with `unsafe` block to check #9503 is fixed too s --- tests/ui/needless_return.fixed | 10 ++++++++++ tests/ui/needless_return.rs | 10 ++++++++++ tests/ui/needless_return.stderr | 18 +++++++++++++++++- 3 files changed, 37 insertions(+), 1 deletion(-) diff --git a/tests/ui/needless_return.fixed b/tests/ui/needless_return.fixed index a5e28b9b1..112a2f57b 100644 --- a/tests/ui/needless_return.fixed +++ b/tests/ui/needless_return.fixed @@ -260,4 +260,14 @@ fn issue9192() -> i32 { } } +fn issue9503(x: usize) -> isize { + unsafe { + if x > 12 { + *(x as *const isize) + } else { + !*(x as *const isize) + } + } +} + fn main() {} diff --git a/tests/ui/needless_return.rs b/tests/ui/needless_return.rs index 49fd49c72..22aa2e11f 100644 --- a/tests/ui/needless_return.rs +++ b/tests/ui/needless_return.rs @@ -260,4 +260,14 @@ fn issue9192() -> i32 { }; } +fn issue9503(x: usize) -> isize { + unsafe { + if x > 12 { + return *(x as *const isize); + } else { + return !*(x as *const isize); + }; + }; +} + fn main() {} diff --git a/tests/ui/needless_return.stderr b/tests/ui/needless_return.stderr index d8dba766b..45090dbe2 100644 --- a/tests/ui/needless_return.stderr +++ b/tests/ui/needless_return.stderr @@ -335,5 +335,21 @@ LL | return 0; | = help: remove `return` -error: aborting due to 42 previous errors +error: unneeded `return` statement + --> $DIR/needless_return.rs:266:13 + | +LL | return *(x as *const isize); + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ + | + = help: remove `return` + +error: unneeded `return` statement + --> $DIR/needless_return.rs:268:13 + | +LL | return !*(x as *const isize); + | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ + | + = help: remove `return` + +error: aborting due to 44 previous errors