diff --git a/src/ast/checks.rs b/src/ast/checks.rs index faf081d82b..a64cbe46c1 100644 --- a/src/ast/checks.rs +++ b/src/ast/checks.rs @@ -148,13 +148,13 @@ pub fn check_ambiguous_variable_name(target: &Expr) -> Vec { match &target.node { ExprKind::Tuple { elts, .. } | ExprKind::List { elts, .. } => { for elt in elts { - if let Some(check) = check_target(&elt) { + if let Some(check) = check_target(elt) { checks.push(check); }; } } _ => { - if let Some(check) = check_target(&target) { + if let Some(check) = check_target(target) { checks.push(check); }; } diff --git a/src/check_ast.rs b/src/check_ast.rs index c3331b9600..e4a19155e4 100644 --- a/src/check_ast.rs +++ b/src/check_ast.rs @@ -407,15 +407,15 @@ where StmtKind::For { target, .. } => { if self.settings.select.contains(&CheckCode::E741) { self.checks - .extend(checks::check_ambiguous_variable_name(&target)); + .extend(checks::check_ambiguous_variable_name(target)); } } StmtKind::With { items, .. } | StmtKind::AsyncWith { items, .. } => { if self.settings.select.contains(&CheckCode::E741) { for item in items { - if let Some(optional_vars) = &item.optional_vars { + if let Some(vars) = &item.optional_vars { self.checks - .extend(checks::check_ambiguous_variable_name(optional_vars)) + .extend(checks::check_ambiguous_variable_name(vars)) } } }