fix incorrect suggestions related to parentheses in needless_return

Author: lapla-cogito

clippy_lints/src/returns.rs

@@ -1,6 +1,6 @@
 use clippy_utils::diagnostics::{span_lint_and_sugg, span_lint_hir_and_then};
-use clippy_utils::source::{SpanRangeExt, snippet_with_context};
-use clippy_utils::sugg::has_enclosing_paren;
+use clippy_utils::source::SpanRangeExt;
+use clippy_utils::sugg::{Sugg, has_enclosing_paren};
 use clippy_utils::visitors::{Descend, for_each_expr};
 use clippy_utils::{
     binary_expr_needs_parentheses, fn_def_id, is_from_proc_macro, is_inside_let_else, is_res_lang_ctor,
@@ -137,7 +137,6 @@ enum RetReplacement<'tcx> {
     Empty,
     Block,
     Unit,
-    NeedsPar(Cow<'tcx, str>, Applicability),
     Expr(Cow<'tcx, str>, Applicability),
 }
 
@@ -147,13 +146,12 @@ impl RetReplacement<'_> {
             Self::Empty | Self::Expr(..) => "remove `return`",
             Self::Block => "replace `return` with an empty block",
             Self::Unit => "replace `return` with a unit value",
-            Self::NeedsPar(..) => "remove `return` and wrap the sequence with parentheses",
         }
     }
 
     fn applicability(&self) -> Applicability {
         match self {
-            Self::Expr(_, ap) | Self::NeedsPar(_, ap) => *ap,
+            Self::Expr(_, ap) => *ap,
             _ => Applicability::MachineApplicable,
         }
     }
@@ -165,7 +163,6 @@ impl Display for RetReplacement<'_> {
             Self::Empty => write!(f, ""),
             Self::Block => write!(f, "{{}}"),
             Self::Unit => write!(f, "()"),
-            Self::NeedsPar(inner, _) => write!(f, "({inner})"),
             Self::Expr(inner, _) => write!(f, "{inner}"),
         }
     }
@@ -365,12 +362,11 @@ fn check_final_expr<'tcx>(
                 }
 
                 let mut applicability = Applicability::MachineApplicable;
-                let (snippet, _) = snippet_with_context(cx, inner_expr.span, ret_span.ctxt(), "..", &mut applicability);
-                if binary_expr_needs_parentheses(inner_expr) {
-                    RetReplacement::NeedsPar(snippet, applicability)
-                } else {
-                    RetReplacement::Expr(snippet, applicability)
-                }
+                let snippet = Sugg::hir_with_context(cx, inner_expr, ret_span.ctxt(), "..", &mut applicability)
+                    .maybe_par()
+                    .to_string();
+
+                RetReplacement::Expr(snippet.into(), applicability)
             } else {
                 match match_ty_opt {
                     Some(match_ty) => {

tests/ui/needless_return.fixed

@@ -6,7 +6,8 @@
     clippy::single_match,
     clippy::needless_bool,
     clippy::equatable_if_let,
-    clippy::needless_else
+    clippy::needless_else,
+    clippy::missing_safety_doc
 )]
 #![warn(clippy::needless_return)]
 
@@ -297,9 +298,9 @@ fn issue9192() -> i32 {
 fn issue9503(x: usize) -> isize {
     unsafe {
         if x > 12 {
-            *(x as *const isize)
+            (*(x as *const isize))
         } else {
-            !*(x as *const isize)
+            (!*(x as *const isize))
         }
     }
 }
@@ -331,7 +332,7 @@ fn issue10051() -> Result<String, String> {
 
 mod issue10049 {
     fn single() -> u32 {
-        if true { 1 } else { 2 }
+        (if true { 1 } else { 2 })
     }
 
     fn multiple(b1: bool, b2: bool, b3: bool) -> u32 {
@@ -388,3 +389,11 @@ fn b(x: Option<u8>) -> Option<u8> {
         },
     }
 }
+
+unsafe fn todo() -> *const u8 {
+    todo!()
+}
+
+pub unsafe fn issue_12157() -> *const i32 {
+    (unsafe { todo() } as *const i32)
+}

tests/ui/needless_return.rs

@@ -6,7 +6,8 @@
     clippy::single_match,
     clippy::needless_bool,
     clippy::equatable_if_let,
-    clippy::needless_else
+    clippy::needless_else,
+    clippy::missing_safety_doc
 )]
 #![warn(clippy::needless_return)]
 
@@ -398,3 +399,11 @@ fn b(x: Option<u8>) -> Option<u8> {
         },
     }
 }
+
+unsafe fn todo() -> *const u8 {
+    todo!()
+}
+
+pub unsafe fn issue_12157() -> *const i32 {
+    return unsafe { todo() } as *const i32;
+}