Reviewers: arv, rossberg,
Message:
This is ready for review, though I'll need to fix the TODOs about the
debugger
before landing.
Description:
[es6] Ensure that for-in/of loops have a proper TDZ for their lexically
bound
variables
The enumerable expression in a for-in/of loop is supposed to have a TDZ for
any
lexically bound names in that loop (there can be more than one with
destructuring).
This patch accomplishes this with an almost-correct desugaring. The only
thing
missing
is proper debugger support (the let declarations added by the desugaring,
while
invisible
to code due to shadowing, are visible to the debugger).
BUG=v8:4210
LOG=n
Please review this at https://codereview.chromium.org/1218543003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+73, -31 lines):
M src/parser.cc
M test/mjsunit/es6/debug-blockscopes.js
M test/mjsunit/es6/debug-stepnext-for.js
M test/mjsunit/es6/regress/regress-2506.js
M test/mjsunit/harmony/destructuring.js
M test/mjsunit/strong/declaration-after-use.js
M test/test262-es6/test262-es6.status
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
