From 40e2f6e25cba6965334c5987b5da5c755f628e2b Mon Sep 17 00:00:00 2001
From: overlookmotel <557937+overlookmotel@users.noreply.github.com>
Date: Fri, 23 Aug 2024 09:28:14 +0000
Subject: [PATCH] refactor(traverse): remove unnecessary branch in
 `ChildScopeCollector` (#5111)

Fix a mistake in `ChildScopeCollector`. `Class` always has a scope. No need to check whether it does.
---
 crates/oxc_traverse/src/context/scoping.rs | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/crates/oxc_traverse/src/context/scoping.rs b/crates/oxc_traverse/src/context/scoping.rs
index 41772f9ea..e795fabe7 100644
--- a/crates/oxc_traverse/src/context/scoping.rs
+++ b/crates/oxc_traverse/src/context/scoping.rs
@@ -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>) {