Rustup (#17389)

r? @ghost

changelog: none

Author: flip1995

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "clippy"
-version = "0.1.98"
+version = "0.1.99"
 description = "A bunch of helpful lints to avoid common pitfalls in Rust"
 repository = "https://github.com/rust-lang/rust-clippy"
 readme = "README.md"

clippy_config/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "clippy_config"
-version = "0.1.98"
+version = "0.1.99"
 edition = "2024"
 publish = false
 

clippy_lints/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "clippy_lints"
-version = "0.1.98"
+version = "0.1.99"
 description = "A bunch of helpful lints to avoid common pitfalls in Rust"
 repository = "https://github.com/rust-lang/rust-clippy"
 readme = "README.md"

clippy_lints/src/bool_assert_comparison.rs

@@ -63,7 +63,7 @@ fn is_impl_not_trait_with_bool_out<'tcx>(cx: &LateContext<'tcx>, ty: Ty<'tcx>) -
             )
         })
         .is_some_and(|assoc_item| {
-            let proj = Ty::new_projection(cx.tcx, assoc_item.def_id, cx.tcx.mk_args_trait(ty, []));
+            let proj = Ty::new_projection(cx.tcx, ty::IsRigid::No, assoc_item.def_id, cx.tcx.mk_args_trait(ty, []));
             let nty = cx
                 .tcx
                 .normalize_erasing_regions(cx.typing_env(), Unnormalized::new_wip(proj));

clippy_lints/src/combined_early_pass.rs

@@ -11,7 +11,7 @@
 //! inlined calls. No vtable, no per-node dynamic dispatch.
 //!
 //! Unlike the late combine there is no `active` gate. rustc drops fully-disabled
-//! late passes via `lints_that_dont_need_to_run`, but the early pass runner has
+//! late passes via `skippable_lints`, but the early pass runner has
 //! no such filtering, so a plain forward is equivalent and loses nothing.
 //!
 //! [`combined_late_pass`]: crate::combined_late_pass

clippy_lints/src/dereference.rs

@@ -925,21 +925,30 @@ impl TyCoercionStability {
                     continue;
                 },
                 ty::Param(_) if for_return => Self::Deref,
-                ty::Alias(ty::AliasTy {
-                    kind: ty::Free { .. } | ty::Inherent { .. },
-                    ..
-                }) => unreachable!("should have been normalized away above"),
-                ty::Alias(ty::AliasTy {
-                    kind: ty::Projection { .. },
-                    ..
-                }) if !for_return && ty.has_non_region_param() => Self::Reborrow,
+                ty::Alias(
+                    _,
+                    ty::AliasTy {
+                        kind: ty::Free { .. } | ty::Inherent { .. },
+                        ..
+                    },
+                ) => unreachable!("should have been normalized away above"),
+                ty::Alias(
+                    _,
+                    ty::AliasTy {
+                        kind: ty::Projection { .. },
+                        ..
+                    },
+                ) if !for_return && ty.has_non_region_param() => Self::Reborrow,
                 ty::Infer(_)
                 | ty::Error(_)
                 | ty::Bound(..)
-                | ty::Alias(ty::AliasTy {
-                    kind: ty::Opaque { .. },
-                    ..
-                })
+                | ty::Alias(
+                    _,
+                    ty::AliasTy {
+                        kind: ty::Opaque { .. },
+                        ..
+                    },
+                )
                 | ty::Placeholder(_)
                 | ty::Dynamic(..)
                 | ty::Param(_) => Self::Reborrow,
@@ -970,10 +979,13 @@ impl TyCoercionStability {
                 | ty::CoroutineClosure(..)
                 | ty::Never
                 | ty::Tuple(_)
-                | ty::Alias(ty::AliasTy {
-                    kind: ty::Projection { .. },
-                    ..
-                })
+                | ty::Alias(
+                    _,
+                    ty::AliasTy {
+                        kind: ty::Projection { .. },
+                        ..
+                    },
+                )
                 | ty::UnsafeBinder(_) => Self::Deref,
             };
         }

clippy_lints/src/from_over_into.rs

@@ -78,7 +78,10 @@ impl<'tcx> LateLintPass<'tcx> for FromOverInto {
             && span_is_local(item.span)
             && let middle_trait_ref = cx.tcx.impl_trait_ref(item.owner_id).instantiate_identity().skip_norm_wip()
             && cx.tcx.is_diagnostic_item(sym::Into, middle_trait_ref.def_id)
-            && !matches!(middle_trait_ref.args.type_at(1).kind(), ty::Alias(ty::AliasTy { kind: ty::Opaque{..} , .. }))
+            && !matches!(
+                middle_trait_ref.args.type_at(1).kind(),
+                ty::Alias(_, ty::AliasTy { kind: ty::Opaque{..} , .. })
+            )
             && self.msrv.meets(cx, msrvs::RE_REBALANCING_COHERENCE)
             // skip if there's a blanket From impl, the suggested impl would conflict
             && !has_blanket_from_impl(cx, middle_trait_ref.self_ty())

clippy_lints/src/future_not_send.rs

@@ -76,7 +76,7 @@ impl<'tcx> LateLintPass<'tcx> for FutureNotSend {
             return;
         }
         let ret_ty = return_ty(cx, cx.tcx.local_def_id_to_hir_id(fn_def_id).expect_owner());
-        if let ty::Alias(AliasTy { kind: ty::Opaque{def_id}, args, .. }) = *ret_ty.kind()
+        if let ty::Alias(_, AliasTy { kind: ty::Opaque{def_id}, args, .. }) = *ret_ty.kind()
             && let Some(future_trait) = cx.tcx.lang_items().future_trait()
             && let Some(send_trait) = cx.tcx.get_diagnostic_item(sym::Send)
             && let preds = cx.tcx.explicit_item_self_bounds(def_id)
@@ -149,6 +149,7 @@ impl<'tcx> TypeVisitor<TyCtxt<'tcx>> for TyParamAtTopLevelVisitor {
         match ty.kind() {
             ty::Param(_) => ControlFlow::Break(true),
             ty::Alias(
+                _,
                 ty @ AliasTy {
                     kind: ty::Projection { .. },
                     ..

clippy_lints/src/indexing_slicing.rs

@@ -280,7 +280,7 @@ fn ty_has_applicable_get_function<'tcx>(
         && let generic_ty = option_generic_param.expect_ty().peel_refs()
         // FIXME: ideally this would handle type params and projections properly, for now just assume it's the same type
         && (cx.typeck_results().expr_ty(index_expr).peel_refs() == generic_ty.peel_refs()
-            || matches!(generic_ty.peel_refs().kind(), ty::Param(_) | ty::Alias(_)))
+            || matches!(generic_ty.peel_refs().kind(), ty::Param(_) | ty::Alias(_, _)))
     {
         true
     } else {

clippy_lints/src/len_without_is_empty.rs

@@ -149,10 +149,13 @@ fn check_trait_items(cx: &LateContext<'_>, visited_trait: &Item<'_>, ident: Iden
 }
 
 fn extract_future_output<'tcx>(cx: &LateContext<'tcx>, ty: Ty<'tcx>) -> Option<&'tcx PathSegment<'tcx>> {
-    if let ty::Alias(ty::AliasTy {
-        kind: ty::Opaque { def_id },
-        ..
-    }) = *ty.kind()
+    if let ty::Alias(
+        _,
+        ty::AliasTy {
+            kind: ty::Opaque { def_id },
+            ..
+        },
+    ) = *ty.kind()
         && let Some(Node::OpaqueTy(opaque)) = cx.tcx.hir_get_if_local(def_id)
         && let OpaqueTyOrigin::AsyncFn { .. } = opaque.origin
         && let [GenericBound::Trait(trait_ref)] = &opaque.bounds