dan/oxc
1
0
Fork 0
mirror of https://github.com/danbulant/oxc synced 2026-05-19 04:08:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

refactor(linter/consistent-function-scoping): remove Visit::enter_node usage (#8538)

Browse source 
Use `Visit::visit_*` method instead of `Visit::enter_node` / `leave_node`. A step towards solving #8461, but also may improve performance.
This commit is contained in:
overlookmotel 2025-01-16 11:00:11 +00:00
parent 30c0689dfc
commit 8dd0013e72
1 changed files with 8 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
crates/oxc_linter/src/rules/unicorn/consistent_function_scoping.rs
View file

@ -1,9 +1,10 @@
use oxc_ast::{AstKind, Visit};
use rustc_hash::FxHashSet;
use oxc_ast::{visit::walk, AstKind, Visit};
use oxc_diagnostics::OxcDiagnostic;
use oxc_macros::declare_oxc_lint;
use oxc_semantic::ReferenceId;
use oxc_semantic::{ReferenceId, ScopeFlags};
use oxc_span::{GetSpan, Span};
use rustc_hash::FxHashSet;
use crate::{
ast_util::{get_function_like_declaration, nth_outermost_paren_parent, outermost_paren_parent},
@ -283,16 +284,10 @@ impl<'a> Visit<'a> for ReferencesFinder {
}
}
fn enter_node(&mut self, kind: AstKind<'a>) {
if let AstKind::Function(_) = kind {
self.in_function += 1;
}
}
fn leave_node(&mut self, kind: AstKind<'a>) {
if let AstKind::Function(_) = kind {
self.in_function -= 1;
}
fn visit_function(&mut self, func: &oxc_ast::ast::Function<'a>, flags: ScopeFlags) {
self.in_function += 1;
walk::walk_function(self, func, flags);
self.in_function -= 1;
}
}