ITStudio_OUC/OxideJS-New
0
1
Fork 0
mirror of https://github.com/pjh456/OxideJS.git synced 2026-10-25 11:11:11 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: 分析时跳过继承作用域，避免将外层 var 误判为嵌套函数本地变量

Browse Source
This commit is contained in:
彭俊皓
2026-06-30 22:05:46 +08:00
parent 6df603002d
commit 655bc8516e
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/oxide_compiler/src/compiler.rs
+7 -3
View File
@@ -700,7 +700,9 @@ impl Compiler {
match expr { match expr {
Expression::Identifier(ident) => { Expression::Identifier(ident) => {
let name = ident.name.as_str(); let name = ident.name.as_str();
if nested_symbols.lookup_any_binding(name).is_some() { // Only skip if name is in the nested function's OWN scopes (not inherited scopes[0])
let is_local = nested_symbols.scopes.iter().skip(1).any(|s| s.bindings.contains_key(name));
if is_local {
return; return;
} }
if let Some((binding, _)) = parent_ctx.scopes.symbols.lookup_any_binding(name) { if let Some((binding, _)) = parent_ctx.scopes.symbols.lookup_any_binding(name) {
@@ -719,7 +721,8 @@ impl Compiler {
Expression::AssignmentExpression(ae) => { Expression::AssignmentExpression(ae) => {
if let oxide_parser::AssignmentTarget::AssignmentTargetIdentifier(ati) = &ae.left { if let oxide_parser::AssignmentTarget::AssignmentTargetIdentifier(ati) = &ae.left {
let name = ati.name.as_str(); let name = ati.name.as_str();
if nested_symbols.lookup_any_binding(name).is_none() { let is_local = nested_symbols.scopes.iter().skip(1).any(|s| s.bindings.contains_key(name));
if !is_local {
if let Some((binding, _)) = parent_ctx.scopes.symbols.lookup_any_binding(name) { if let Some((binding, _)) = parent_ctx.scopes.symbols.lookup_any_binding(name) {
if !seen.contains_key(name) { if !seen.contains_key(name) {
let cell_idx = captures.len() as u8; let cell_idx = captures.len() as u8;
@@ -745,7 +748,8 @@ impl Compiler {
Expression::UpdateExpression(ue) => { Expression::UpdateExpression(ue) => {
if let oxide_parser::SimpleAssignmentTarget::AssignmentTargetIdentifier(ati) = &ue.argument { if let oxide_parser::SimpleAssignmentTarget::AssignmentTargetIdentifier(ati) = &ue.argument {
let name = ati.name.as_str(); let name = ati.name.as_str();
if nested_symbols.lookup_any_binding(name).is_none() { let is_local = nested_symbols.scopes.iter().skip(1).any(|s| s.bindings.contains_key(name));
if !is_local {
if let Some((binding, _)) = parent_ctx.scopes.symbols.lookup_any_binding(name) { if let Some((binding, _)) = parent_ctx.scopes.symbols.lookup_any_binding(name) {
if !seen.contains_key(name) { if !seen.contains_key(name) {
let cell_idx = captures.len() as u8; let cell_idx = captures.len() as u8;