Reviewers: ulan,
Message:
ulan, ptal
Description:
Tests and fixes for (pre)parse errors related to strict reserved words.
This contains the following fixes:
- We had strict_reserved_word and unexpected_strict_reserved, which one to
use
was totally mixed in Parser and PreParser. Removed strict_reserved_word.
- When we saw a strict future reserved word when expecting something
completely
different (such as "(" in "function foo interface"), Parser reports
unexpected
identifier, whereas PreParser used to report unexpected strict reserved
word. Fixed PreParser to report unexpected identifier too.
- Unified parser and preparser error locations when the name of a function
is a
strict reserved word. Now both point to the name.
BUG=3126
LOG=N
[email protected]
Please review this at https://codereview.chromium.org/149253010/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+202, -52 lines):
M src/messages.js
M src/parser.cc
M src/preparser.cc
M test/cctest/test-parsing.cc
--
--
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/groups/opt_out.
