StaticContext should be immutable

RESOLVED FIXED

Status

Other Applications
Narcissus
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: dherman, Assigned: dherman)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

7 years ago
The StaticContext type in the parser should be immutable and pushed/popped implicitly via the native JS stack, to avoid subtle and fragile paired calls to push() and pop() of member arrays; these arrays should be implemented as immutable lists. Also separate concerns so that each property of a StaticContext serves only one purpose in the semantics (currently, the stmtStack is used both for let-hoisting and break/continue labels).

Dave
(Assignee)

Comment 1

7 years ago
https://github.com/mozilla/narcissus/commit/8e3bd60ddc904190d7c1ec3320e4a64799386216
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.