fix: iter_on_single_item FP on let stmt with type annotation

Author: profetia

clippy_lints/src/methods/iter_on_single_or_empty_collections.rs

@@ -97,7 +97,8 @@ pub(super) fn check<'tcx>(cx: &LateContext<'tcx>, expr: &'tcx Expr<'tcx>, method
             | ExprKind::Break(_, _) => true,
             _ => false,
         },
-        Some((Node::Stmt(_) | Node::LetStmt(_), _)) => false,
+        Some((Node::LetStmt(let_stmt), _)) => let_stmt.ty.is_some(),
+        Some((Node::Stmt(_), _)) => false,
         _ => true,
     };
 

tests/ui/iter_on_single_items.fixed

@@ -67,13 +67,26 @@ fn main() {
     in_macros!();
 }
 
-fn issue14981() {
+mod issue14981 {
     use std::option::IntoIter;
-    fn some_func(_: IntoIter<i32>) -> IntoIter<i32> {
-        todo!()
+    fn takes_into_iter(_: impl IntoIterator<Item = i32>) {}
+
+    fn let_stmt() {
+        macro_rules! x {
+            ($e:expr) => {
+                let _: IntoIter<i32> = $e;
+            };
+        }
+        x!(Some(5).into_iter());
     }
-    some_func(Some(5).into_iter());
 
-    const C: fn(IntoIter<i32>) -> IntoIter<i32> = <IntoIter<i32> as IntoIterator>::into_iter;
-    C(Some(5).into_iter());
+    fn fn_ptr() {
+        fn some_func(_: IntoIter<i32>) -> IntoIter<i32> {
+            todo!()
+        }
+        some_func(Some(5).into_iter());
+
+        const C: fn(IntoIter<i32>) -> IntoIter<i32> = <IntoIter<i32> as IntoIterator>::into_iter;
+        C(Some(5).into_iter());
+    }
 }

tests/ui/iter_on_single_items.rs

@@ -67,13 +67,26 @@ fn main() {
     in_macros!();
 }
 
-fn issue14981() {
+mod issue14981 {
     use std::option::IntoIter;
-    fn some_func(_: IntoIter<i32>) -> IntoIter<i32> {
-        todo!()
+    fn takes_into_iter(_: impl IntoIterator<Item = i32>) {}
+
+    fn let_stmt() {
+        macro_rules! x {
+            ($e:expr) => {
+                let _: IntoIter<i32> = $e;
+            };
+        }
+        x!(Some(5).into_iter());
     }
-    some_func(Some(5).into_iter());
 
-    const C: fn(IntoIter<i32>) -> IntoIter<i32> = <IntoIter<i32> as IntoIterator>::into_iter;
-    C(Some(5).into_iter());
+    fn fn_ptr() {
+        fn some_func(_: IntoIter<i32>) -> IntoIter<i32> {
+            todo!()
+        }
+        some_func(Some(5).into_iter());
+
+        const C: fn(IntoIter<i32>) -> IntoIter<i32> = <IntoIter<i32> as IntoIterator>::into_iter;
+        C(Some(5).into_iter());
+    }
 }