create path_ty, asref_def_id when constructing the lint pass

Author: ada4a

clippy_lints/src/lib.rs

@@ -831,6 +831,6 @@ pub fn register_lint_passes(store: &mut rustc_lint::LintStore, conf: &'static Co
     store.register_late_pass(|_| Box::new(cloned_ref_to_slice_refs::ClonedRefToSliceRefs::new(conf)));
     store.register_late_pass(|_| Box::new(infallible_try_from::InfallibleTryFrom));
     store.register_late_pass(|_| Box::new(coerce_container_to_any::CoerceContainerToAny));
-    store.register_late_pass(|_| Box::new(needless_path_new::NeedlessPathNew));
+    store.register_late_pass(|tcx| Box::new(needless_path_new::NeedlessPathNew::new(tcx)));
     // add lints here, do not remove this comment, it's used in `new_lint`
 }

clippy_lints/src/needless_path_new.rs

@@ -6,8 +6,8 @@ use rustc_errors::Applicability;
 use rustc_hir::def::{DefKind, Res};
 use rustc_hir::{Expr, ExprKind, QPath};
 use rustc_lint::{LateContext, LateLintPass};
-use rustc_middle::ty::{self, List, Ty};
-use rustc_session::declare_lint_pass;
+use rustc_middle::ty::{List, Ty, TyCtxt};
+use rustc_session::impl_lint_pass;
 use rustc_span::sym;
 use std::iter;
 
@@ -36,12 +36,35 @@ declare_clippy_lint! {
     being enclosed in `Path::new` when the argument implements the trait"
 }
 
-declare_lint_pass!(NeedlessPathNew => [NEEDLESS_PATH_NEW]);
+impl_lint_pass!(NeedlessPathNew<'_> => [NEEDLESS_PATH_NEW]);
 
-impl<'tcx> LateLintPass<'tcx> for NeedlessPathNew {
+pub struct NeedlessPathNew<'tcx> {
+    path_ty: Option<Ty<'tcx>>,
+    asref_def_id: Option<DefId>,
+}
+
+impl<'tcx> NeedlessPathNew<'tcx> {
+    pub fn new(tcx: TyCtxt<'tcx>) -> Self {
+        Self {
+            path_ty: (tcx.get_diagnostic_item(sym::Path))
+                .map(|path_def_id| Ty::new_adt(tcx, tcx.adt_def(path_def_id), List::empty())),
+            asref_def_id: tcx.get_diagnostic_item(sym::AsRef),
+        }
+    }
+}
+
+impl<'tcx> LateLintPass<'tcx> for NeedlessPathNew<'tcx> {
     fn check_expr(&mut self, cx: &LateContext<'tcx>, e: &'tcx Expr<'tcx>) {
         let tcx = cx.tcx;
 
+        let Some(path_ty) = self.path_ty else {
+            return;
+        };
+
+        let Some(asref_def_id) = self.asref_def_id else {
+            return;
+        };
+
         let (fn_did, arguments) = match e.kind {
             ExprKind::Call(callee, args)
                 if let Res::Def(DefKind::Fn | DefKind::AssocFn, did) = path_res(cx, callee) =>
@@ -72,17 +95,6 @@ impl<'tcx> LateLintPass<'tcx> for NeedlessPathNew {
             }
         };
 
-        let path_ty = {
-            let Some(path_def_id) = tcx.get_diagnostic_item(sym::Path) else {
-                return;
-            };
-            Ty::new_adt(tcx, tcx.adt_def(path_def_id), List::empty())
-        };
-
-        let Some(asref_def_id) = tcx.get_diagnostic_item(sym::AsRef) else {
-            return;
-        };
-
         let implements_asref_path = |arg| implements_trait(cx, arg, asref_def_id, &[path_ty.into()]);
 
         let parameters = sig.inputs();