Skip to content

Commit 31ad96a

Browse files
JarchoJarcho
committed
Replace is_lang_ctor with is_res_lang_ctor 
1 parent 4995b4e commit 31ad96a

21 files changed

+184
-167
lines changed

‎clippy_lints/src/if_then_some_else_none.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
use clippy_utils::diagnostics::span_lint_and_help;
22
use clippy_utils::source::snippet_with_macro_callsite;
3-
use clippy_utils::{contains_return, higher, is_else_clause, is_lang_ctor, meets_msrv, msrvs, peel_blocks};
3+
use clippy_utils::{
4+
contains_return, higher, is_else_clause, is_res_lang_ctor, meets_msrv, msrvs, path_res, peel_blocks,
5+
};
46
use if_chain::if_chain;
57
use rustc_hir::LangItem::{OptionNone, OptionSome};
68
use rustc_hir::{Expr, ExprKind, Stmt, StmtKind};
@@ -75,10 +77,8 @@ impl<'tcx> LateLintPass<'tcx> for IfThenSomeElseNone {
7577
if let ExprKind::Block(then_block, _) = then.kind;
7678
if let Some(then_expr) = then_block.expr;
7779
if let ExprKind::Call(then_call, [then_arg]) = then_expr.kind;
78-
if let ExprKind::Path(ref then_call_qpath) = then_call.kind;
79-
if is_lang_ctor(cx, then_call_qpath, OptionSome);
80-
if let ExprKind::Path(ref qpath) = peel_blocks(els).kind;
81-
if is_lang_ctor(cx, qpath, OptionNone);
80+
if is_res_lang_ctor(cx, path_res(cx, then_call), OptionSome);
81+
if is_res_lang_ctor(cx, path_res(cx, peel_blocks(els)), OptionNone);
8282
if !stmts_contains_early_return(then_block.stmts);
8383
then {
8484
let cond_snip = snippet_with_macro_callsite(cx, cond.span, "[condition]");

‎clippy_lints/src/loops/manual_find.rs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
use super::utils::make_iterator_snippet;
22
use super::MANUAL_FIND;
33
use clippy_utils::{
4-
diagnostics::span_lint_and_then, higher, is_lang_ctor, path_res, peel_blocks_with_stmt,
4+
diagnostics::span_lint_and_then, higher, is_res_lang_ctor, path_res, peel_blocks_with_stmt,
55
source::snippet_with_applicability, ty::implements_trait,
66
};
77
use if_chain::if_chain;
@@ -30,8 +30,8 @@ pub(super) fn check<'tcx>(
3030
if let [stmt] = block.stmts;
3131
if let StmtKind::Semi(semi) = stmt.kind;
3232
if let ExprKind::Ret(Some(ret_value)) = semi.kind;
33-
if let ExprKind::Call(Expr { kind: ExprKind::Path(ctor), .. }, [inner_ret]) = ret_value.kind;
34-
if is_lang_ctor(cx, ctor, LangItem::OptionSome);
33+
if let ExprKind::Call(ctor, [inner_ret]) = ret_value.kind;
34+
if is_res_lang_ctor(cx, path_res(cx, ctor), LangItem::OptionSome);
3535
if path_res(cx, inner_ret) == Res::Local(binding_id);
3636
if let Some((last_stmt, last_ret)) = last_stmt_and_ret(cx, expr);
3737
then {
@@ -143,8 +143,7 @@ fn last_stmt_and_ret<'tcx>(
143143
if let Some((_, Node::Block(block))) = parent_iter.next();
144144
if let Some((last_stmt, last_ret)) = extract(block);
145145
if last_stmt.hir_id == node_hir;
146-
if let ExprKind::Path(path) = &last_ret.kind;
147-
if is_lang_ctor(cx, path, LangItem::OptionNone);
146+
if is_res_lang_ctor(cx, path_res(cx, last_ret), LangItem::OptionNone);
148147
if let Some((_, Node::Expr(_block))) = parent_iter.next();
149148
// This includes the function header
150149
if let Some((_, func)) = parent_iter.next();

‎clippy_lints/src/loops/manual_flatten.rs

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@ use super::MANUAL_FLATTEN;
33
use clippy_utils::diagnostics::span_lint_and_then;
44
use clippy_utils::higher;
55
use clippy_utils::visitors::is_local_used;
6-
use clippy_utils::{is_lang_ctor, path_to_local_id, peel_blocks_with_stmt};
6+
use clippy_utils::{path_to_local_id, peel_blocks_with_stmt};
77
use if_chain::if_chain;
88
use rustc_errors::Applicability;
9-
use rustc_hir::LangItem::{OptionSome, ResultOk};
9+
use rustc_hir::def::{DefKind, Res};
1010
use rustc_hir::{Expr, Pat, PatKind};
1111
use rustc_lint::LateContext;
12-
use rustc_middle::ty;
12+
use rustc_middle::ty::{self, DefIdTree};
1313
use rustc_span::source_map::Span;
1414

1515
/// Check for unnecessary `if let` usage in a for loop where only the `Some` or `Ok` variant of the
@@ -30,8 +30,10 @@ pub(super) fn check<'tcx>(
3030
if path_to_local_id(let_expr, pat_hir_id);
3131
// Ensure the `if let` statement is for the `Some` variant of `Option` or the `Ok` variant of `Result`
3232
if let PatKind::TupleStruct(ref qpath, _, _) = let_pat.kind;
33-
let some_ctor = is_lang_ctor(cx, qpath, OptionSome);
34-
let ok_ctor = is_lang_ctor(cx, qpath, ResultOk);
33+
if let Res::Def(DefKind::Ctor(..), ctor_id) = cx.qpath_res(qpath, let_pat.hir_id);
34+
if let Some(variant_id) = cx.tcx.opt_parent(ctor_id);
35+
let some_ctor = cx.tcx.lang_items().option_some_variant() == Some(variant_id);
36+
let ok_ctor = cx.tcx.lang_items().result_ok_variant() == Some(variant_id);
3537
if some_ctor || ok_ctor;
3638
// Ensure expr in `if let` is not used afterwards
3739
if !is_local_used(cx, if_then, pat_hir_id);

‎clippy_lints/src/loops/while_let_on_iterator.rs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,12 @@ use clippy_utils::diagnostics::span_lint_and_sugg;
33
use clippy_utils::higher;
44
use clippy_utils::source::snippet_with_applicability;
55
use clippy_utils::{
6-
get_enclosing_loop_or_closure, is_refutable, is_trait_method, match_def_path, paths, visitors::is_res_used,
6+
get_enclosing_loop_or_closure, is_refutable, is_res_lang_ctor, is_trait_method, visitors::is_res_used,
77
};
88
use if_chain::if_chain;
99
use rustc_errors::Applicability;
1010
use rustc_hir::intravisit::{walk_expr, Visitor};
11-
use rustc_hir::{def::Res, Expr, ExprKind, HirId, Local, Mutability, PatKind, QPath, UnOp};
11+
use rustc_hir::{def::Res, Expr, ExprKind, HirId, LangItem, Local, Mutability, PatKind, UnOp};
1212
use rustc_lint::LateContext;
1313
use rustc_middle::ty::adjustment::Adjust;
1414
use rustc_span::{symbol::sym, Symbol};
@@ -17,9 +17,8 @@ pub(super) fn check<'tcx>(cx: &LateContext<'tcx>, expr: &'tcx Expr<'_>) {
1717
let (scrutinee_expr, iter_expr_struct, iter_expr, some_pat, loop_expr) = if_chain! {
1818
if let Some(higher::WhileLet { if_then, let_pat, let_expr }) = higher::WhileLet::hir(expr);
1919
// check for `Some(..)` pattern
20-
if let PatKind::TupleStruct(QPath::Resolved(None, pat_path), some_pat, _) = let_pat.kind;
21-
if let Res::Def(_, pat_did) = pat_path.res;
22-
if match_def_path(cx, pat_did, &paths::OPTION_SOME);
20+
if let PatKind::TupleStruct(ref pat_path, some_pat, _) = let_pat.kind;
21+
if is_res_lang_ctor(cx, cx.qpath_res(pat_path, let_pat.hir_id), LangItem::OptionSome);
2322
// check for call to `Iterator::next`
2423
if let ExprKind::MethodCall(method_name, [iter_expr], _) = let_expr.kind;
2524
if method_name.ident.name == sym::next;

‎clippy_lints/src/manual_ok_or.rs

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
use clippy_utils::diagnostics::span_lint_and_sugg;
22
use clippy_utils::source::{indent_of, reindent_multiline, snippet_opt};
33
use clippy_utils::ty::is_type_diagnostic_item;
4-
use clippy_utils::{is_lang_ctor, path_to_local_id};
4+
use clippy_utils::{is_res_lang_ctor, path_res, path_to_local_id};
55
use if_chain::if_chain;
66
use rustc_errors::Applicability;
77
use rustc_hir::LangItem::{ResultErr, ResultOk};
@@ -55,8 +55,8 @@ impl<'tcx> LateLintPass<'tcx> for ManualOkOr {
5555
if is_type_diagnostic_item(cx, ty, sym::Option);
5656
let or_expr = &args[1];
5757
if is_ok_wrapping(cx, &args[2]);
58-
if let ExprKind::Call(Expr { kind: ExprKind::Path(err_path), .. }, &[ref err_arg]) = or_expr.kind;
59-
if is_lang_ctor(cx, err_path, ResultErr);
58+
if let ExprKind::Call(err_path, &[ref err_arg]) = or_expr.kind;
59+
if is_res_lang_ctor(cx, path_res(cx, err_path), ResultErr);
6060
if let Some(method_receiver_snippet) = snippet_opt(cx, method_receiver.span);
6161
if let Some(err_arg_snippet) = snippet_opt(cx, err_arg.span);
6262
if let Some(indent) = indent_of(cx, scrutinee.span);
@@ -82,17 +82,15 @@ impl<'tcx> LateLintPass<'tcx> for ManualOkOr {
8282
}
8383

8484
fn is_ok_wrapping(cx: &LateContext<'_>, map_expr: &Expr<'_>) -> bool {
85-
if let ExprKind::Path(ref qpath) = map_expr.kind {
86-
if is_lang_ctor(cx, qpath, ResultOk) {
87-
return true;
88-
}
85+
if is_res_lang_ctor(cx, path_res(cx, map_expr), ResultOk) {
86+
return true;
8987
}
9088
if_chain! {
9189
if let ExprKind::Closure { body, .. } = map_expr.kind;
9290
let body = cx.tcx.hir().body(body);
9391
if let PatKind::Binding(_, param_id, ..) = body.params[0].pat.kind;
94-
if let ExprKind::Call(Expr { kind: ExprKind::Path(ok_path), .. }, &[ref ok_arg]) = body.value.kind;
95-
if is_lang_ctor(cx, ok_path, ResultOk);
92+
if let ExprKind::Call(ok_path, &[ref ok_arg]) = body.value.kind;
93+
if is_res_lang_ctor(cx, path_res(cx, ok_path), ResultOk);
9694
then { path_to_local_id(ok_arg, param_id) } else { false }
9795
}
9896
}

‎clippy_lints/src/matches/collapsible_match.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
use clippy_utils::diagnostics::span_lint_and_then;
22
use clippy_utils::higher::IfLetOrMatch;
33
use clippy_utils::visitors::is_local_used;
4-
use clippy_utils::{is_lang_ctor, is_unit_expr, path_to_local, peel_blocks_with_stmt, peel_ref_operators, SpanlessEq};
4+
use clippy_utils::{
5+
is_res_lang_ctor, is_unit_expr, path_to_local, peel_blocks_with_stmt, peel_ref_operators, SpanlessEq,
6+
};
57
use if_chain::if_chain;
68
use rustc_errors::MultiSpan;
79
use rustc_hir::LangItem::OptionNone;
@@ -110,7 +112,7 @@ fn arm_is_wild_like(cx: &LateContext<'_>, arm: &Arm<'_>) -> bool {
110112
}
111113
match arm.pat.kind {
112114
PatKind::Binding(..) | PatKind::Wild => true,
113-
PatKind::Path(ref qpath) => is_lang_ctor(cx, qpath, OptionNone),
115+
PatKind::Path(ref qpath) => is_res_lang_ctor(cx, cx.qpath_res(qpath, arm.pat.hir_id), OptionNone),
114116
_ => false,
115117
}
116118
}

‎clippy_lints/src/matches/manual_map.rs

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ use clippy_utils::diagnostics::span_lint_and_sugg;
33
use clippy_utils::source::{snippet_with_applicability, snippet_with_context};
44
use clippy_utils::ty::{is_type_diagnostic_item, peel_mid_ty_refs_is_mutable, type_is_unsafe_function};
55
use clippy_utils::{
6-
can_move_expr_to_closure, is_else_clause, is_lang_ctor, is_lint_allowed, path_to_local_id, peel_blocks,
7-
peel_hir_expr_refs, peel_hir_expr_while, CaptureKind,
6+
can_move_expr_to_closure, is_else_clause, is_lint_allowed, is_res_lang_ctor, path_res, path_to_local_id,
7+
peel_blocks, peel_hir_expr_refs, peel_hir_expr_while, CaptureKind,
88
};
99
use rustc_ast::util::parser::PREC_POSTFIX;
1010
use rustc_errors::Applicability;
@@ -251,9 +251,11 @@ fn try_parse_pattern<'tcx>(cx: &LateContext<'tcx>, pat: &'tcx Pat<'_>, ctxt: Syn
251251
match pat.kind {
252252
PatKind::Wild => Some(OptionPat::Wild),
253253
PatKind::Ref(pat, _) => f(cx, pat, ref_count + 1, ctxt),
254-
PatKind::Path(ref qpath) if is_lang_ctor(cx, qpath, OptionNone) => Some(OptionPat::None),
254+
PatKind::Path(ref qpath) if is_res_lang_ctor(cx, cx.qpath_res(qpath, pat.hir_id), OptionNone) => {
255+
Some(OptionPat::None)
256+
},
255257
PatKind::TupleStruct(ref qpath, [pattern], _)
256-
if is_lang_ctor(cx, qpath, OptionSome) && pat.span.ctxt() == ctxt =>
258+
if is_res_lang_ctor(cx, cx.qpath_res(qpath, pat.hir_id), OptionSome) && pat.span.ctxt() == ctxt =>
257259
{
258260
Some(OptionPat::Some { pattern, ref_count })
259261
},
@@ -272,16 +274,14 @@ fn get_some_expr<'tcx>(
272274
) -> Option<SomeExpr<'tcx>> {
273275
// TODO: Allow more complex expressions.
274276
match expr.kind {
275-
ExprKind::Call(
276-
Expr {
277-
kind: ExprKind::Path(ref qpath),
278-
..
279-
},
280-
[arg],
281-
) if ctxt == expr.span.ctxt() && is_lang_ctor(cx, qpath, OptionSome) => Some(SomeExpr {
282-
expr: arg,
283-
needs_unsafe_block,
284-
}),
277+
ExprKind::Call(callee, [arg])
278+
if ctxt == expr.span.ctxt() && is_res_lang_ctor(cx, path_res(cx, callee), OptionSome) =>
279+
{
280+
Some(SomeExpr {
281+
expr: arg,
282+
needs_unsafe_block,
283+
})
284+
},
285285
ExprKind::Block(
286286
Block {
287287
stmts: [],
@@ -302,5 +302,5 @@ fn get_some_expr<'tcx>(
302302

303303
// Checks for the `None` value.
304304
fn is_none_expr(cx: &LateContext<'_>, expr: &Expr<'_>) -> bool {
305-
matches!(peel_blocks(expr).kind, ExprKind::Path(ref qpath) if is_lang_ctor(cx, qpath, OptionNone))
305+
is_res_lang_ctor(cx, path_res(cx, peel_blocks(expr)), OptionNone)
306306
}

‎clippy_lints/src/matches/manual_unwrap_or.rs

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,14 @@ use clippy_utils::diagnostics::span_lint_and_sugg;
33
use clippy_utils::source::{indent_of, reindent_multiline, snippet_opt};
44
use clippy_utils::ty::is_type_diagnostic_item;
55
use clippy_utils::usage::contains_return_break_continue_macro;
6-
use clippy_utils::{is_lang_ctor, path_to_local_id, sugg};
6+
use clippy_utils::{is_res_lang_ctor, path_to_local_id, sugg};
77
use if_chain::if_chain;
88
use rustc_errors::Applicability;
9-
use rustc_hir::LangItem::{OptionNone, OptionSome, ResultErr, ResultOk};
9+
use rustc_hir::def::{DefKind, Res};
10+
use rustc_hir::LangItem::{OptionNone, ResultErr};
1011
use rustc_hir::{Arm, Expr, PatKind};
1112
use rustc_lint::LateContext;
13+
use rustc_middle::ty::DefIdTree;
1214
use rustc_span::sym;
1315

1416
use super::MANUAL_UNWRAP_OR;
@@ -61,15 +63,19 @@ fn applicable_or_arm<'a>(cx: &LateContext<'_>, arms: &'a [Arm<'a>]) -> Option<&'
6163
if arms.iter().all(|arm| arm.guard.is_none());
6264
if let Some((idx, or_arm)) = arms.iter().enumerate().find(|(_, arm)| {
6365
match arm.pat.kind {
64-
PatKind::Path(ref qpath) => is_lang_ctor(cx, qpath, OptionNone),
66+
PatKind::Path(ref qpath) => is_res_lang_ctor(cx, cx.qpath_res(qpath, arm.pat.hir_id), OptionNone),
6567
PatKind::TupleStruct(ref qpath, [pat], _) =>
66-
matches!(pat.kind, PatKind::Wild) && is_lang_ctor(cx, qpath, ResultErr),
68+
matches!(pat.kind, PatKind::Wild)
69+
&& is_res_lang_ctor(cx, cx.qpath_res(qpath, arm.pat.hir_id), ResultErr),
6770
_ => false,
6871
}
6972
});
7073
let unwrap_arm = &arms[1 - idx];
7174
if let PatKind::TupleStruct(ref qpath, [unwrap_pat], _) = unwrap_arm.pat.kind;
72-
if is_lang_ctor(cx, qpath, OptionSome) || is_lang_ctor(cx, qpath, ResultOk);
75+
if let Res::Def(DefKind::Ctor(..), ctor_id) = cx.qpath_res(qpath, unwrap_arm.pat.hir_id);
76+
if let Some(variant_id) = cx.tcx.opt_parent(ctor_id);
77+
if cx.tcx.lang_items().option_some_variant() == Some(variant_id)
78+
|| cx.tcx.lang_items().result_ok_variant() == Some(variant_id);
7379
if let PatKind::Binding(_, binding_hir_id, ..) = unwrap_pat.kind;
7480
if path_to_local_id(unwrap_arm.body, binding_hir_id);
7581
if cx.typeck_results().expr_adjustments(unwrap_arm.body).is_empty();

‎clippy_lints/src/matches/match_as_ref.rs

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
use clippy_utils::diagnostics::span_lint_and_sugg;
22
use clippy_utils::source::snippet_with_applicability;
3-
use clippy_utils::{is_lang_ctor, peel_blocks};
3+
use clippy_utils::{is_res_lang_ctor, path_res, peel_blocks};
44
use rustc_errors::Applicability;
55
use rustc_hir::{Arm, BindingAnnotation, Expr, ExprKind, LangItem, PatKind, QPath};
66
use rustc_lint::LateContext;
@@ -62,19 +62,21 @@ pub(crate) fn check(cx: &LateContext<'_>, ex: &Expr<'_>, arms: &[Arm<'_>], expr:
6262

6363
// Checks if arm has the form `None => None`
6464
fn is_none_arm(cx: &LateContext<'_>, arm: &Arm<'_>) -> bool {
65-
matches!(arm.pat.kind, PatKind::Path(ref qpath) if is_lang_ctor(cx, qpath, LangItem::OptionNone))
65+
matches!(
66+
arm.pat.kind,
67+
PatKind::Path(ref qpath) if is_res_lang_ctor(cx, cx.qpath_res(qpath, arm.pat.hir_id), LangItem::OptionNone)
68+
)
6669
}
6770

6871
// Checks if arm has the form `Some(ref v) => Some(v)` (checks for `ref` and `ref mut`)
6972
fn is_ref_some_arm(cx: &LateContext<'_>, arm: &Arm<'_>) -> Option<BindingAnnotation> {
7073
if_chain! {
7174
if let PatKind::TupleStruct(ref qpath, [first_pat, ..], _) = arm.pat.kind;
72-
if is_lang_ctor(cx, qpath, LangItem::OptionSome);
75+
if is_res_lang_ctor(cx, cx.qpath_res(qpath, arm.pat.hir_id), LangItem::OptionSome);
7376
if let PatKind::Binding(rb, .., ident, _) = first_pat.kind;
7477
if rb == BindingAnnotation::Ref || rb == BindingAnnotation::RefMut;
7578
if let ExprKind::Call(e, args) = peel_blocks(arm.body).kind;
76-
if let ExprKind::Path(ref some_path) = e.kind;
77-
if is_lang_ctor(cx, some_path, LangItem::OptionSome) && args.len() == 1;
79+
if is_res_lang_ctor(cx, path_res(cx, e), LangItem::OptionSome) && args.len() == 1;
7880
if let ExprKind::Path(QPath::Resolved(_, path2)) = args[0].kind;
7981
if path2.segments.len() == 1 && ident.name == path2.segments[0].ident.name;
8082
then {

‎clippy_lints/src/matches/needless_match.rs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ use clippy_utils::diagnostics::span_lint_and_sugg;
33
use clippy_utils::source::snippet_with_applicability;
44
use clippy_utils::ty::{is_type_diagnostic_item, same_type_and_consts};
55
use clippy_utils::{
6-
eq_expr_value, get_parent_expr_for_hir, get_parent_node, higher, is_else_clause, is_lang_ctor, over,
6+
eq_expr_value, get_parent_expr_for_hir, get_parent_node, higher, is_else_clause, is_res_lang_ctor, over, path_res,
77
peel_blocks_with_stmt,
88
};
99
use rustc_errors::Applicability;
@@ -94,10 +94,7 @@ fn check_if_let_inner(cx: &LateContext<'_>, if_let: &higher::IfLet<'_>) -> bool
9494
let ret = strip_return(else_expr);
9595
let let_expr_ty = cx.typeck_results().expr_ty(if_let.let_expr);
9696
if is_type_diagnostic_item(cx, let_expr_ty, sym::Option) {
97-
if let ExprKind::Path(ref qpath) = ret.kind {
98-
return is_lang_ctor(cx, qpath, OptionNone) || eq_expr_value(cx, if_let.let_expr, ret);
99-
}
100-
return false;
97+
return is_res_lang_ctor(cx, path_res(cx, ret), OptionNone) || eq_expr_value(cx, if_let.let_expr, ret);
10198
}
10299
return eq_expr_value(cx, if_let.let_expr, ret);
103100
}

‎clippy_lints/src/matches/redundant_pattern_match.rs

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,11 @@ use clippy_utils::source::snippet;
44
use clippy_utils::sugg::Sugg;
55
use clippy_utils::ty::needs_ordered_drop;
66
use clippy_utils::{higher, match_def_path};
7-
use clippy_utils::{is_lang_ctor, is_trait_method, paths};
7+
use clippy_utils::{is_trait_method, paths};
88
use if_chain::if_chain;
99
use rustc_ast::ast::LitKind;
1010
use rustc_errors::Applicability;
11-
use rustc_hir::LangItem::{OptionNone, PollPending};
11+
use rustc_hir::def::{DefKind, Res};
1212
use rustc_hir::{
1313
intravisit::{walk_expr, Visitor},
1414
Arm, Block, Expr, ExprKind, Node, Pat, PatKind, QPath, UnOp,
@@ -163,15 +163,21 @@ fn find_sugg_for_if_let<'tcx>(
163163
}
164164
},
165165
PatKind::Path(ref path) => {
166-
let method = if is_lang_ctor(cx, path, OptionNone) {
167-
"is_none()"
168-
} else if is_lang_ctor(cx, path, PollPending) {
169-
"is_pending()"
166+
if let Res::Def(DefKind::Ctor(..), ctor_id) = cx.qpath_res(path, check_pat.hir_id)
167+
&& let Some(variant_id) = cx.tcx.opt_parent(ctor_id)
168+
{
169+
let method = if cx.tcx.lang_items().option_none_variant() == Some(variant_id) {
170+
"is_none()"
171+
} else if cx.tcx.lang_items().poll_pending_variant() == Some(variant_id) {
172+
"is_pending()"
173+
} else {
174+
return;
175+
};
176+
// `None` and `Pending` don't have an inner type.
177+
(method, cx.tcx.types.unit)
170178
} else {
171179
return;
172-
};
173-
// `None` and `Pending` don't have an inner type.
174-
(method, cx.tcx.types.unit)
180+
}
175181
},
176182
_ => return,
177183
};

‎clippy_lints/src/matches/try_err.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
use clippy_utils::diagnostics::span_lint_and_sugg;
22
use clippy_utils::source::snippet_with_applicability;
33
use clippy_utils::ty::is_type_diagnostic_item;
4-
use clippy_utils::{get_parent_expr, is_lang_ctor, match_def_path, paths};
4+
use clippy_utils::{get_parent_expr, is_res_lang_ctor, match_def_path, path_res, paths};
55
use if_chain::if_chain;
66
use rustc_errors::Applicability;
77
use rustc_hir::LangItem::ResultErr;
@@ -29,8 +29,7 @@ pub(super) fn check<'tcx>(cx: &LateContext<'tcx>, expr: &'tcx Expr<'_>, scrutine
2929
if let Some(try_arg) = try_args.get(0);
3030
if let ExprKind::Call(err_fun, err_args) = try_arg.kind;
3131
if let Some(err_arg) = err_args.get(0);
32-
if let ExprKind::Path(ref err_fun_path) = err_fun.kind;
33-
if is_lang_ctor(cx, err_fun_path, ResultErr);
32+
if is_res_lang_ctor(cx, path_res(cx, err_fun), ResultErr);
3433
if let Some(return_ty) = find_return_type(cx, &expr.kind);
3534
then {
3635
let prefix;

0 commit comments

Comments
 (0)