fix FP with MBEs

Author: ada4a

clippy_lints/src/needless_parens_on_range_literals.rs

@@ -1,5 +1,5 @@
 use clippy_utils::diagnostics::span_lint_and_sugg;
-use clippy_utils::source::{SpanRangeExt, snippet_with_applicability};
+use clippy_utils::source::{SpanRangeExt, snippet_with_context};
 
 use rustc_ast::{Expr, ExprKind};
 use rustc_errors::Applicability;
@@ -39,7 +39,8 @@ declare_clippy_lint! {
 declare_lint_pass!(NeedlessParensOnRangeLiterals => [NEEDLESS_PARENS_ON_RANGE_LITERALS]);
 
 fn check_for_parens(cx: &EarlyContext<'_>, e: &Expr, is_start: bool) {
-    if let ExprKind::Paren(literal) = &e.kind
+    if !e.span.from_expansion()
+        && let ExprKind::Paren(literal) = &e.kind
         && let ExprKind::Lit(lit) = &literal.kind
     {
         if is_start
@@ -52,7 +53,7 @@ fn check_for_parens(cx: &EarlyContext<'_>, e: &Expr, is_start: bool) {
         }
 
         let mut applicability = Applicability::MachineApplicable;
-        let suggestion = snippet_with_applicability(cx, literal.span, "_", &mut applicability);
+        let suggestion = snippet_with_context(cx, literal.span, e.span.ctxt(), "_", &mut applicability).0;
         span_lint_and_sugg(
             cx,
             NEEDLESS_PARENS_ON_RANGE_LITERALS,

tests/ui/needless_parens_on_range_literals.fixed

@@ -18,4 +18,28 @@ fn main() {
     //~^ needless_parens_on_range_literals
     let _ = ..'z';
     //~^ needless_parens_on_range_literals
+
+    macro_rules! verbatim {
+        ($e:expr) => {
+            $e
+        };
+    }
+    macro_rules! add_paren {
+        ($e:expr) => {
+            ($e)
+        };
+    }
+
+    // lint: the paren was added by the user
+    let _ = verbatim!(0)..1;
+    //~^ needless_parens_on_range_literals
+    let _ = 0..verbatim!(1);
+    //~^ needless_parens_on_range_literals
+
+    // lint: the macro doesn't have anything to do with the paren
+    let _ = 0..1;
+    //~^ needless_parens_on_range_literals
+
+    // don't lint: the paren was added by the macro
+    let _ = add_paren!(0)..1;
 }

tests/ui/needless_parens_on_range_literals.rs

@@ -18,4 +18,28 @@ fn main() {
     //~^ needless_parens_on_range_literals
     let _ = ..('z');
     //~^ needless_parens_on_range_literals
+
+    macro_rules! verbatim {
+        ($e:expr) => {
+            $e
+        };
+    }
+    macro_rules! add_paren {
+        ($e:expr) => {
+            ($e)
+        };
+    }
+
+    // lint: the paren was added by the user
+    let _ = verbatim!((0))..1;
+    //~^ needless_parens_on_range_literals
+    let _ = 0..verbatim!((1));
+    //~^ needless_parens_on_range_literals
+
+    // lint: the macro doesn't have anything to do with the paren
+    let _ = (verbatim!(0))..1;
+    //~^ needless_parens_on_range_literals
+
+    // don't lint: the paren was added by the macro
+    let _ = add_paren!(0)..1;
 }

tests/ui/needless_parens_on_range_literals.stderr

@@ -43,5 +43,23 @@ error: needless parenthesis on range literals can be removed
 LL |     let _ = ..('z');
    |               ^^^^^ help: try: `'z'`
 
-error: aborting due to 7 previous errors
+error: needless parenthesis on range literals can be removed
+  --> tests/ui/needless_parens_on_range_literals.rs:34:23
+   |
+LL |     let _ = verbatim!((0))..1;
+   |                       ^^^ help: try: `0`
+
+error: needless parenthesis on range literals can be removed
+  --> tests/ui/needless_parens_on_range_literals.rs:36:26
+   |
+LL |     let _ = 0..verbatim!((1));
+   |                          ^^^ help: try: `1`
+
+error: needless parenthesis on range literals can be removed
+  --> tests/ui/needless_parens_on_range_literals.rs:40:13
+   |
+LL |     let _ = (verbatim!(0))..1;
+   |             ^^^^^^^^^^^^^^ help: try: `0`
+
+error: aborting due to 10 previous errors