From 8bdac5606d702dc7d3dedd97c11533c117081f84 Mon Sep 17 00:00:00 2001 From: shulaoda <165626830+shulaoda@users.noreply.github.com> Date: Tue, 18 Mar 2025 15:48:46 +0000 Subject: [PATCH] refactor(linter): improve `ast_util::is_method_call` (#9874) Simplify the method code. --- crates/oxc_linter/src/ast_util.rs | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/crates/oxc_linter/src/ast_util.rs b/crates/oxc_linter/src/ast_util.rs index 6fd3e2abbe84d..a4aec9ff4ca5d 100644 --- a/crates/oxc_linter/src/ast_util.rs +++ b/crates/oxc_linter/src/ast_util.rs @@ -343,18 +343,12 @@ pub fn is_method_call<'a>( } } - let callee_without_parentheses = call_expr.callee.without_parentheses(); - let member_expr = match callee_without_parentheses { - match_member_expression!(Expression) => callee_without_parentheses.to_member_expression(), - Expression::ChainExpression(chain) => match chain.expression.member_expression() { - Some(e) => e, - None => return false, - }, - _ => return false, + let Some(member_expr) = call_expr.callee.get_member_expr() else { + return false; }; if let Some(objects) = objects { - let Expression::Identifier(ident) = member_expr.object().without_parentheses() else { + let Expression::Identifier(ident) = member_expr.object().get_inner_expression() else { return false; }; if !objects.contains(&ident.name.as_str()) {