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

refactor(traverse): remove unnecessary branch in ChildScopeCollector (#5111)

Browse source 
Fix a mistake in `ChildScopeCollector`. `Class` always has a scope. No need to check whether it does.
This commit is contained in:
overlookmotel 2024-08-23 09:28:14 +00:00
parent c100826b42
commit 40e2f6e25c
1 changed files with 2 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
crates/oxc_traverse/src/context/scoping.rs
View file

@ -2,10 +2,7 @@ use std::{cell::Cell, str};
use compact_str::{format_compact, CompactString};
#[allow(clippy::wildcard_imports)]
use oxc_ast::{
ast::*,
visit::{walk, Visit},
};
use oxc_ast::{ast::*, visit::Visit};
use oxc_semantic::{AstNodeId, Reference, ScopeTree, SymbolTable};
use oxc_span::{Atom, CompactStr, Span, SPAN};
use oxc_syntax::{
@ -567,11 +564,7 @@ impl<'a> Visit<'a> for ChildScopeCollector {
}
fn visit_class(&mut self, class: &Class<'a>) {
if let Some(scope_id) = class.scope_id.get() {
self.scope_ids.push(scope_id);
} else {
walk::walk_class(self, class);
}
self.scope_ids.push(class.scope_id.get().unwrap());
}
fn visit_static_block(&mut self, block: &StaticBlock<'a>) {