Also walk bindings created by if-let guards
This commit is contained in:
parent
fb5ed726f7
commit
ab60a7df64
@ -1351,6 +1351,9 @@ impl<'a, 'tcx> Visitor<'tcx> for Liveness<'a, 'tcx> {
|
||||
|
||||
fn visit_arm(&mut self, arm: &'tcx hir::Arm<'tcx>) {
|
||||
self.check_unused_vars_in_pat(arm.pat, None, None, |_, _, _, _| {});
|
||||
if let Some(hir::Guard::IfLet(let_expr)) = arm.guard {
|
||||
self.check_unused_vars_in_pat(let_expr.pat, None, None, |_, _, _, _| {});
|
||||
}
|
||||
intravisit::walk_arm(self, arm);
|
||||
}
|
||||
}
|
||||
|
9
tests/ui/lint/unused/issue-119383.rs
Normal file
9
tests/ui/lint/unused/issue-119383.rs
Normal file
@ -0,0 +1,9 @@
|
||||
#![feature(if_let_guard)]
|
||||
#![deny(unused_variables)]
|
||||
|
||||
fn main() {
|
||||
match () {
|
||||
() if let Some(b) = Some(()) => {} //~ ERROR unused variable: `b`
|
||||
_ => {}
|
||||
}
|
||||
}
|
14
tests/ui/lint/unused/issue-119383.stderr
Normal file
14
tests/ui/lint/unused/issue-119383.stderr
Normal file
@ -0,0 +1,14 @@
|
||||
error: unused variable: `b`
|
||||
--> $DIR/issue-119383.rs:6:24
|
||||
|
|
||||
LL | () if let Some(b) = Some(()) => {}
|
||||
| ^ help: if this is intentional, prefix it with an underscore: `_b`
|
||||
|
|
||||
note: the lint level is defined here
|
||||
--> $DIR/issue-119383.rs:2:9
|
||||
|
|
||||
LL | #![deny(unused_variables)]
|
||||
| ^^^^^^^^^^^^^^^^
|
||||
|
||||
error: aborting due to 1 previous error
|
||||
|
Loading…
x
Reference in New Issue
Block a user