On 2015/06/04 20:54:15, caitp wrote:

https://codereview.chromium.org/1168643005/diff/20001/test/cctest/test-parsing.cc
File test/cctest/test-parsing.cc (right):


https://codereview.chromium.org/1168643005/diff/20001/test/cctest/test-parsing.cc#newcode6616
test/cctest/test-parsing.cc:6616: "{ x: new.target }",
On 2015/06/04 20:08:04, arv wrote:
> I think this is valid syntax inside a function but it would throw at runtime
> since it is a const binding.

IsValidSimpleAssignmentTarget returns false for `new.target`, making it an
early
error when used as a DestructuringAssignmentTarget, iirc --- However, I think
`new.target.foo` would be acceptable, if that's legal to parse

OK. Early error > late error.

https://codereview.chromium.org/1168643005/

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

Reply via email to