post-commit comments:
https://codereview.chromium.org/722793005/diff/100001/src/parser.cc
File src/parser.cc (right):
https://codereview.chromium.org/722793005/diff/100001/src/parser.cc#newcode3897
src/parser.cc:3897: ClassLiteral* Parser::ParseClassLiteral(const
AstRawString* name,
Is there a reason why ParseClassLiteral is not in ParserBase, but both
in Parser and PreParser (so that there are 2 versions of it)? Is
something missing in the PreParser? Is there some mess in the code that
makes it hard?
All other Expression parsing is unified (so, implemented only once, in
ParserBase), so this is moving the situation backwards...
https://codereview.chromium.org/722793005/
--
--
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.
