From 1f77f8cc3cca968d47d15c9e621d48f0193f8ab1 Mon Sep 17 00:00:00 2001
From: y21 <30553356+y21@users.noreply.github.com>
Date: Mon, 3 Jul 2023 13:40:33 +0200
Subject: [PATCH] [`useless_vec`]: use the source span

---
 clippy_lints/src/vec.rs |  2 +-
 tests/ui/vec.fixed      | 11 +++++++++++
 tests/ui/vec.rs         | 11 +++++++++++
 tests/ui/vec.stderr     | 12 +++++++++---
 4 files changed, 32 insertions(+), 4 deletions(-)

diff --git a/clippy_lints/src/vec.rs b/clippy_lints/src/vec.rs
index 2a594e750b9..ca0daa9ecee 100644
--- a/clippy_lints/src/vec.rs
+++ b/clippy_lints/src/vec.rs
@@ -181,7 +181,7 @@ fn check_vec_macro<'tcx>(
                     if args.len() as u64 * size_of(cx, last) > self.too_large_for_stack {
                         return;
                     }
-                    let span = args[0].span.to(last.span);
+                    let span = args[0].span.source_callsite().to(last.span.source_callsite());
                     let args = snippet_with_applicability(cx, span, "..", &mut applicability);
 
                     match suggest_slice {
diff --git a/tests/ui/vec.fixed b/tests/ui/vec.fixed
index fcdc917c1b1..14ab8bd8897 100644
--- a/tests/ui/vec.fixed
+++ b/tests/ui/vec.fixed
@@ -115,6 +115,17 @@ fn main() {
     let _x = vec![1; 201];
 }
 
+fn issue11075() {
+    macro_rules! repro {
+        ($e:expr) => {
+            stringify!($e)
+        };
+    }
+    for _string in [repro!(true), repro!(null)] {
+        unimplemented!();
+    }
+}
+
 #[clippy::msrv = "1.53"]
 fn above() {
     for a in [1, 2, 3] {
diff --git a/tests/ui/vec.rs b/tests/ui/vec.rs
index 0404d8cdb84..e57cb4374e3 100644
--- a/tests/ui/vec.rs
+++ b/tests/ui/vec.rs
@@ -115,6 +115,17 @@ fn main() {
     let _x = vec![1; 201];
 }
 
+fn issue11075() {
+    macro_rules! repro {
+        ($e:expr) => {
+            stringify!($e)
+        };
+    }
+    for _string in vec![repro!(true), repro!(null)] {
+        unimplemented!();
+    }
+}
+
 #[clippy::msrv = "1.53"]
 fn above() {
     for a in vec![1, 2, 3] {
diff --git a/tests/ui/vec.stderr b/tests/ui/vec.stderr
index 33d565b2d52..1b82d5cbea8 100644
--- a/tests/ui/vec.stderr
+++ b/tests/ui/vec.stderr
@@ -85,16 +85,22 @@ LL |     for _ in vec![1, 2, 3] {}
    |              ^^^^^^^^^^^^^ help: you can use an array directly: `[1, 2, 3]`
 
 error: useless use of `vec!`
-  --> $DIR/vec.rs:120:14
+  --> $DIR/vec.rs:124:20
+   |
+LL |     for _string in vec![repro!(true), repro!(null)] {
+   |                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: you can use an array directly: `[repro!(true), repro!(null)]`
+
+error: useless use of `vec!`
+  --> $DIR/vec.rs:131:14
    |
 LL |     for a in vec![1, 2, 3] {
    |              ^^^^^^^^^^^^^ help: you can use an array directly: `[1, 2, 3]`
 
 error: useless use of `vec!`
-  --> $DIR/vec.rs:124:14
+  --> $DIR/vec.rs:135:14
    |
 LL |     for a in vec![String::new(), String::new()] {
    |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: you can use an array directly: `[String::new(), String::new()]`
 
-error: aborting due to 16 previous errors
+error: aborting due to 17 previous errors