https://codereview.chromium.org/1272673003/diff/230001/src/parser.h
File src/parser.h (right):

https://codereview.chromium.org/1272673003/diff/230001/src/parser.h#newcode1324
src/parser.h:1324: !is_rest && pattern->IsVariableProxy() && initializer
== nullptr;
On 2015/09/02 20:11:31, caitp wrote:
This makes sure the empty string is used instead of the variable name,
so that
the variable doesn't get declared twice. This seems to fix the bug in
the tests.
I'm not sure if this code should be here, or maybe live somewhere
else, but it
should be okay either way.

Yes, this looks right to me.

https://codereview.chromium.org/1272673003/

--
--
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