Skip to content

Commit 8d5078e

Browse files
committed
Introduce trait_item_of
1 parent ba107fc commit 8d5078e

File tree

3 files changed

+4
-11
lines changed

3 files changed

+4
-11
lines changed

clippy_lints/src/functions/renamed_function_params.rs

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
use clippy_utils::diagnostics::span_lint_and_then;
22
use rustc_errors::{Applicability, MultiSpan};
3-
use rustc_hir::def_id::{DefId, DefIdSet};
4-
use rustc_hir::hir_id::OwnerId;
3+
use rustc_hir::def_id::DefIdSet;
54
use rustc_hir::{Impl, ImplItem, ImplItemKind, ItemKind, Node, TraitRef};
65
use rustc_lint::LateContext;
76
use rustc_span::Span;
@@ -19,7 +18,7 @@ pub(super) fn check_impl_item(cx: &LateContext<'_>, item: &ImplItem<'_>, ignored
1918
of_trait: Some(of_trait),
2019
..
2120
}) = &parent_item.kind
22-
&& let Some(did) = trait_item_def_id_of_impl(cx, item.owner_id)
21+
&& let Some(did) = cx.tcx.trait_item_of(item.owner_id)
2322
&& !is_from_ignored_trait(&of_trait.trait_ref, ignored_traits)
2423
{
2524
let mut param_idents_iter = cx.tcx.hir_body_param_idents(body_id);
@@ -87,11 +86,6 @@ impl RenamedFnArgs {
8786
}
8887
}
8988

90-
/// Get the [`trait_item_def_id`](ImplItemRef::trait_item_def_id) of a relevant impl item.
91-
fn trait_item_def_id_of_impl(cx: &LateContext<'_>, target: OwnerId) -> Option<DefId> {
92-
cx.tcx.associated_item(target).trait_item_def_id
93-
}
94-
9589
fn is_from_ignored_trait(of_trait: &TraitRef<'_>, ignored_traits: &DefIdSet) -> bool {
9690
of_trait
9791
.trait_def_id()

clippy_lints/src/only_used_in_recursion.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ impl<'tcx> LateLintPass<'tcx> for OnlyUsedInRecursion {
248248
.tcx
249249
.impl_trait_ref(item.owner_id)
250250
.map(EarlyBinder::instantiate_identity)
251-
&& let Some(trait_item_id) = cx.tcx.associated_item(owner_id).trait_item_def_id
251+
&& let Some(trait_item_id) = cx.tcx.trait_item_of(owner_id)
252252
{
253253
(
254254
trait_item_id,

clippy_lints/src/use_self.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -151,8 +151,7 @@ impl<'tcx> LateLintPass<'tcx> for UseSelf {
151151
// trait, not in the impl of the trait.
152152
let trait_method = cx
153153
.tcx
154-
.associated_item(impl_item.owner_id)
155-
.trait_item_def_id
154+
.trait_item_of(impl_item.owner_id)
156155
.expect("impl method matches a trait method");
157156
let trait_method_sig = cx.tcx.fn_sig(trait_method).instantiate_identity();
158157
let trait_method_sig = cx.tcx.instantiate_bound_regions_with_erased(trait_method_sig);

0 commit comments

Comments
 (0)