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

refactor(semantic): remove defunct code setting ScopeFlags twice (#4286)

Browse source 
Scope flags for functions is set when the scope is created. Remove redundant code that sets them again.
This commit is contained in:
overlookmotel 2024-07-16 07:32:36 +00:00
parent 2c7bb9f6c8
commit c5731a5431
1 changed files with 10 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
crates/oxc_semantic/src/binder.rs
View file

@ -151,28 +151,16 @@ impl<'a> Binder for Function<'a> {
} }
} }
// bind scope flags: Constructor | GetAccessor | SetAccessor // Bind scope flags: GetAccessor | SetAccessor
if let Some(kind) = builder.nodes.parent_kind(builder.current_node_id) { if let Some(AstKind::ObjectProperty(prop)) =
match kind { builder.nodes.parent_kind(builder.current_node_id)
AstKind::MethodDefinition(def) => { {
let flag = builder.scope.get_flags_mut(current_scope_id); let flag = builder.scope.get_flags_mut(current_scope_id);
*flag |= match def.kind { match prop.kind {
MethodDefinitionKind::Constructor => ScopeFlags::Constructor, PropertyKind::Get => *flag |= ScopeFlags::GetAccessor,
MethodDefinitionKind::Get => ScopeFlags::GetAccessor, PropertyKind::Set => *flag |= ScopeFlags::SetAccessor,
MethodDefinitionKind::Set => ScopeFlags::SetAccessor, PropertyKind::Init => {}
MethodDefinitionKind::Method => ScopeFlags::empty(), };
};
}
AstKind::ObjectProperty(prop) => {
let flag = builder.scope.get_flags_mut(current_scope_id);
*flag |= match prop.kind {
PropertyKind::Get => ScopeFlags::GetAccessor,
PropertyKind::Set => ScopeFlags::SetAccessor,
PropertyKind::Init => ScopeFlags::empty(),
};
}
_ => {}
}
} }
} }
} }