From 80bf8b4b089d4227a81000c12227bf3a506cf281 Mon Sep 17 00:00:00 2001 From: harupy Date: Sat, 10 Sep 2022 18:37:00 +0900 Subject: [PATCH] clean up --- src/ast/checks.rs | 4 ++-- src/check_ast.rs | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) 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)) } } }