https://codereview.chromium.org/1149043005/diff/1/src/parser.cc
File src/parser.cc (right):
https://codereview.chromium.org/1149043005/diff/1/src/parser.cc#newcode3461
src/parser.cc:3461: // special case for legacy for (var/const x =....
in)
On 2015/05/21 17:18:04, rossberg wrote:
Does this actually work with const? You are doing a regular ASSIGN
below.
It should only work for legacy const and it does (we have tests)
https://codereview.chromium.org/1149043005/diff/1/src/parser.cc#newcode3464
src/parser.cc:3464: parsing_result.declarations[0].initializer !=
nullptr) {
On 2015/05/21 17:18:04, rossberg wrote:
Don't you need to also check that the LHS is a simple variable?
No this is done in the parser (see tests in test-parsing.cc)
https://codereview.chromium.org/1149043005/
--
--
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.
