X-Git-Url: https://git.lizzy.rs/?a=blobdiff_plain;f=clippy_lints%2Fsrc%2Finherent_to_string.rs;h=9d7c3b46fed5678489f1e5206324f52c4a53f9f0;hb=5ac08b0cc94cab09fba719a17c141c68c4cbc063;hp=26b9657589f38eee3870d25162190ee7fda6e6ea;hpb=888b736560755254e33593033c3d1248c8e56085;p=rust.git diff --git a/clippy_lints/src/inherent_to_string.rs b/clippy_lints/src/inherent_to_string.rs index 26b9657589f..9d7c3b46fed 100644 --- a/clippy_lints/src/inherent_to_string.rs +++ b/clippy_lints/src/inherent_to_string.rs @@ -1,7 +1,8 @@ use if_chain::if_chain; -use rustc::hir::{ImplItem, ImplItemKind}; +use rustc::declare_lint_pass; use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass}; -use rustc::{declare_lint_pass, declare_tool_lint}; +use rustc_hir::{ImplItem, ImplItemKind}; +use rustc_session::declare_tool_lint; use crate::utils::{ get_trait_def_id, implements_trait, match_type, paths, return_ty, span_help_and_lint, trait_ref_of_method, @@ -9,7 +10,7 @@ }; declare_clippy_lint! { - /// **What id does:** Checks for the definition of inherent methods with a signature of `to_string(&self) -> String`. + /// **What it does:** Checks for the definition of inherent methods with a signature of `to_string(&self) -> String`. /// /// **Why is this bad?** This method is also implicitly defined if a type implements the `Display` trait. As the functionality of `Display` is much more versatile, it should be preferred. /// @@ -46,7 +47,7 @@ } declare_clippy_lint! { - /// **What id does:** Checks for the definition of inherent methods with a signature of `to_string(&self) -> String` and if the type implementing this method also implements the `Display` trait. + /// **What it does:** Checks for the definition of inherent methods with a signature of `to_string(&self) -> String` and if the type implementing this method also implements the `Display` trait. /// /// **Why is this bad?** This method is also implicitly defined if a type implements the `Display` trait. The less versatile inherent method will then shadow the implementation introduced by `Display`. /// @@ -87,20 +88,20 @@ /// ``` pub INHERENT_TO_STRING_SHADOW_DISPLAY, correctness, - "type implements inherent method `to_string()`, which gets shadowed by the implementation of the `Display` trait " + "type implements inherent method `to_string()`, which gets shadowed by the implementation of the `Display` trait" } declare_lint_pass!(InherentToString => [INHERENT_TO_STRING, INHERENT_TO_STRING_SHADOW_DISPLAY]); impl<'a, 'tcx> LateLintPass<'a, 'tcx> for InherentToString { - fn check_impl_item(&mut self, cx: &LateContext<'a, 'tcx>, impl_item: &'tcx ImplItem) { + fn check_impl_item(&mut self, cx: &LateContext<'a, 'tcx>, impl_item: &'tcx ImplItem<'_>) { if impl_item.span.from_expansion() { return; } if_chain! { // Check if item is a method, called to_string and has a parameter 'self' - if let ImplItemKind::Method(ref signature, _) = impl_item.node; + if let ImplItemKind::Method(ref signature, _) = impl_item.kind; if impl_item.ident.name.as_str() == "to_string"; let decl = &signature.decl; if decl.implicit_self.has_implicit_self(); @@ -119,7 +120,7 @@ fn check_impl_item(&mut self, cx: &LateContext<'a, 'tcx>, impl_item: &'tcx ImplI } } -fn show_lint(cx: &LateContext<'_, '_>, item: &ImplItem) { +fn show_lint(cx: &LateContext<'_, '_>, item: &ImplItem<'_>) { let display_trait_id = get_trait_def_id(cx, &["core", "fmt", "Display"]).expect("Failed to get trait ID of `Display`!");