wildcard_match_arm: expand lint scope.

We're not only working with Results.
This commit is contained in:
Alex Hamilton 2019-01-25 10:39:09 -06:00
parent 1b3c3d073a
commit 20ba476ea8

View file

@ -464,17 +464,14 @@ fn check_wild_err_arm(cx: &LateContext<'_, '_>, ex: &Expr, arms: &[Arm]) {
} }
fn check_wild_arm(cx: &LateContext<'_, '_>, ex: &Expr, arms: &[Arm]) { fn check_wild_arm(cx: &LateContext<'_, '_>, ex: &Expr, arms: &[Arm]) {
let ex_ty = walk_ptrs_ty(cx.tables.expr_ty(ex)); for arm in arms {
if match_type(cx, ex_ty, &paths::RESULT) { if is_wild(&arm.pats[0]) {
for arm in arms { span_note_and_lint(cx,
if is_wild(&arm.pats[0]) { WILDCARD_MATCH_ARM,
span_note_and_lint(cx, arm.pats[0].span,
WILDCARD_MATCH_ARM, "wildcard match will miss any future added variants.",
arm.pats[0].span, arm.pats[0].span,
"wildcard match will miss any future added variants.", "to resolve, match each variant explicitly");
arm.pats[0].span,
"to resolve, match each variant explicitly");
}
} }
} }
} }