Auto merge of #8382 - tamaroning:suggest_iter_instead_of_into_iter, r=giraffate

[explicit_counter_loop] suggests `.into_iter()`, despite that triggering [into_iter_on_ref] in some cases

I have modified `fn make_iterator_snippet` in clippy_lints/src/loops/utils.rs ,so this change has some little influence on another lint [manual_flatten] .

fixes #8155

---
changelog: Fix that [`explicit_counter_loop`] suggests `into_iter()` despite that triggering [`into_iter_on_ref`] in some cases

3 files changed, 15 insertions, 17 deletions

diff --git a/tests/ui/explicit_counter_loop.stderr b/tests/ui/explicit_counter_loop.stderr
index 9edddea651c..f9f8407d577 100644
--- a/tests/ui/explicit_counter_loop.stderr
+++ b/tests/ui/explicit_counter_loop.stderr
@@ -46,13 +46,13 @@ error: the variable `idx_usize` is used as a loop counter
   --> $DIR/explicit_counter_loop.rs:170:9
    |
 LL |         for _item in slice {
-   |         ^^^^^^^^^^^^^^^^^^ help: consider using: `for (idx_usize, _item) in slice.into_iter().enumerate()`
+   |         ^^^^^^^^^^^^^^^^^^ help: consider using: `for (idx_usize, _item) in slice.iter().enumerate()`
 
 error: the variable `idx_u32` is used as a loop counter
   --> $DIR/explicit_counter_loop.rs:182:9
    |
 LL |         for _item in slice {
-   |         ^^^^^^^^^^^^^^^^^^ help: consider using: `for (idx_u32, _item) in (0_u32..).zip(slice.into_iter())`
+   |         ^^^^^^^^^^^^^^^^^^ help: consider using: `for (idx_u32, _item) in (0_u32..).zip(slice.iter())`
    |
    = note: `idx_u32` is of type `u32`, making it ineligible for `Iterator::enumerate`
 
diff --git a/tests/ui/manual_flatten.rs b/tests/ui/manual_flatten.rs
index 7db6b730963..6c5232ec5f5 100644
--- a/tests/ui/manual_flatten.rs
+++ b/tests/ui/manual_flatten.rs
@@ -26,8 +26,6 @@ fn main() {
     }
 
     // Test for loop over an implicit reference
-    // Note: if `clippy::manual_flatten` is made autofixable, this case will
-    // lead to a follow-up lint `clippy::into_iter_on_ref`
     let z = &y;
     for n in z {
         if let Ok(n) = n {
diff --git a/tests/ui/manual_flatten.stderr b/tests/ui/manual_flatten.stderr
index be5f8a1d818..392e1a39393 100644
--- a/tests/ui/manual_flatten.stderr
+++ b/tests/ui/manual_flatten.stderr
@@ -63,10 +63,10 @@ LL | |         }
    | |_________^
 
 error: unnecessary `if let` since only the `Ok` variant of the iterator element is used
-  --> $DIR/manual_flatten.rs:32:5
+  --> $DIR/manual_flatten.rs:30:5
    |
 LL |       for n in z {
-   |       ^        - help: try: `z.into_iter().flatten()`
+   |       ^        - help: try: `z.iter().flatten()`
    |  _____|
    | |
 LL | |         if let Ok(n) = n {
@@ -76,7 +76,7 @@ LL | |     }
    | |_____^
    |
 help: ...and remove the `if let` statement in the for loop
-  --> $DIR/manual_flatten.rs:33:9
+  --> $DIR/manual_flatten.rs:31:9
    |
 LL | /         if let Ok(n) = n {
 LL | |             println!("{}", n);
@@ -84,7 +84,7 @@ LL | |         }
    | |_________^
 
 error: unnecessary `if let` since only the `Some` variant of the iterator element is used
-  --> $DIR/manual_flatten.rs:41:5
+  --> $DIR/manual_flatten.rs:39:5
    |
 LL |       for n in z {
    |       ^        - help: try: `z.flatten()`
@@ -97,7 +97,7 @@ LL | |     }
    | |_____^
    |
 help: ...and remove the `if let` statement in the for loop
-  --> $DIR/manual_flatten.rs:42:9
+  --> $DIR/manual_flatten.rs:40:9
    |
 LL | /         if let Some(m) = n {
 LL | |             println!("{}", m);
@@ -105,7 +105,7 @@ LL | |         }
    | |_________^
 
 error: unnecessary `if let` since only the `Some` variant of the iterator element is used
-  --> $DIR/manual_flatten.rs:74:5
+  --> $DIR/manual_flatten.rs:72:5
    |
 LL |       for n in &vec_of_ref {
    |       ^        ----------- help: try: `vec_of_ref.iter().copied().flatten()`
@@ -118,7 +118,7 @@ LL | |     }
    | |_____^
    |
 help: ...and remove the `if let` statement in the for loop
-  --> $DIR/manual_flatten.rs:75:9
+  --> $DIR/manual_flatten.rs:73:9
    |
 LL | /         if let Some(n) = n {
 LL | |             println!("{:?}", n);
@@ -126,10 +126,10 @@ LL | |         }
    | |_________^
 
 error: unnecessary `if let` since only the `Some` variant of the iterator element is used
-  --> $DIR/manual_flatten.rs:81:5
+  --> $DIR/manual_flatten.rs:79:5
    |
 LL |       for n in vec_of_ref {
-   |       ^        ---------- help: try: `vec_of_ref.into_iter().copied().flatten()`
+   |       ^        ---------- help: try: `vec_of_ref.iter().copied().flatten()`
    |  _____|
    | |
 LL | |         if let Some(n) = n {
@@ -139,7 +139,7 @@ LL | |     }
    | |_____^
    |
 help: ...and remove the `if let` statement in the for loop
-  --> $DIR/manual_flatten.rs:82:9
+  --> $DIR/manual_flatten.rs:80:9
    |
 LL | /         if let Some(n) = n {
 LL | |             println!("{:?}", n);
@@ -147,10 +147,10 @@ LL | |         }
    | |_________^
 
 error: unnecessary `if let` since only the `Some` variant of the iterator element is used
-  --> $DIR/manual_flatten.rs:88:5
+  --> $DIR/manual_flatten.rs:86:5
    |
 LL |       for n in slice_of_ref {
-   |       ^        ------------ help: try: `slice_of_ref.into_iter().copied().flatten()`
+   |       ^        ------------ help: try: `slice_of_ref.iter().copied().flatten()`
    |  _____|
    | |
 LL | |         if let Some(n) = n {
@@ -160,7 +160,7 @@ LL | |     }
    | |_____^
    |
 help: ...and remove the `if let` statement in the for loop
-  --> $DIR/manual_flatten.rs:89:9
+  --> $DIR/manual_flatten.rs:87:9
    |
 LL | /         if let Some(n) = n {
 LL | |             println!("{:?}", n);