dogfood

Author: y21

clippy_lints/src/could_be_assoc_type_bounds.rs

@@ -31,7 +31,7 @@ declare_clippy_lint! {
     ///
     /// fn example2<I: Iterator<Item = T>, T: Copy>() {}
     ///
-    /// fn example3<I, T>() -> impl IntoIterator<Item = impl Copy> {}
+    /// fn example3(_: impl IntoIterator<Item = impl Copy>) {}
     /// ```
     ///
     /// Use instead:
@@ -43,7 +43,7 @@ declare_clippy_lint! {
     ///
     /// fn example2<I: Iterator<Item: Copy>>() {}
     ///
-    /// fn example3<I, T>() -> impl IntoIterator<Item: Copy> {}
+    /// fn example3(_: impl IntoIterator<Item: Copy>) {}
     /// ```
     ///
     /// [associated type bounds]: https://blog.rust-lang.org/2024/06/13/Rust-1.79.0.html#bounds-in-associated-type-position
@@ -115,12 +115,17 @@ impl ItemState {
                 .params
                 .iter()
                 .enumerate()
-                .filter(|(_, p)| matches!(p.kind, hir::GenericParamKind::Type { .. }))
-                .map(|(param_index, &hir::GenericParam { def_id, .. })| TyParamState {
-                    def_id,
-                    param_index,
-                    use_count: 0,
-                    constrained_assoc_type: None,
+                .filter_map(|(param_index, &hir::GenericParam { def_id, kind, .. })| {
+                    if let hir::GenericParamKind::Type { .. } = kind {
+                        Some(TyParamState {
+                            def_id,
+                            param_index,
+                            use_count: 0,
+                            constrained_assoc_type: None,
+                        })
+                    } else {
+                        None
+                    }
                 })
                 .collect(),
         }