PTAL
https://codereview.chromium.org/811113002/diff/40001/test/mjsunit/harmony/templates.js
File test/mjsunit/harmony/templates.js (left):
https://codereview.chromium.org/811113002/diff/40001/test/mjsunit/harmony/templates.js#oldcode256
test/mjsunit/harmony/templates.js:256: (function(s) { calls++;
assertEquals("\u005Cx", s.raw[0]); })`\x`;
On 2014/12/17 19:35:56, caitp wrote:
Maybe just replace `\x` with `\z` or something?
Done.
https://codereview.chromium.org/811113002/diff/40001/test/mjsunit/harmony/templates.js
File test/mjsunit/harmony/templates.js (right):
https://codereview.chromium.org/811113002/diff/40001/test/mjsunit/harmony/templates.js#newcode475
test/mjsunit/harmony/templates.js:475: assertThrows("`\\01`",
SyntaxError);
On 2014/12/17 19:35:56, caitp wrote:
This looks like a duplicate
\00, \01, \02, \03, \04, \05, \06, \07, \08, and \09 should all be
errors ---
but \0a, \0., \0^ etc should all be okay.
Adding more tests
https://codereview.chromium.org/811113002/diff/40001/test/mjsunit/harmony/templates.js#newcode478
test/mjsunit/harmony/templates.js:478: assertEquals('\\123',
String.raw`\123`);
On 2014/12/17 19:35:56, caitp wrote:
My reading of the spec is that this should be a SyntaxError as well
You are right. Fixing.
https://codereview.chromium.org/811113002/
--
--
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.
