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)
Does this actually work with const? You are doing a regular ASSIGN
below.

https://codereview.chromium.org/1149043005/diff/1/src/parser.cc#newcode3464
src/parser.cc:3464: parsing_result.declarations[0].initializer !=
nullptr) {
Don't you need to also check that the LHS is a simple variable?

https://codereview.chromium.org/1149043005/

--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to