PTAL

cq+ if it looks good to you.


https://codereview.chromium.org/905003003/diff/1/src/preparser.h
File src/preparser.h (right):

https://codereview.chromium.org/905003003/diff/1/src/preparser.h#newcode1622
src/preparser.h:1622: SourceElements ParseStatementList(int end_token,
bool* ok);
On 2015/02/06 22:24:15, adamk wrote:
You could also rename the return value, although it's pretty confusing
no matter
what the name is.

Actually, could you just get rid of the enum altogether and make this
return
void? We already have to redefine the CHECK_OK macro to make the
SourceElements
thing work.

Done.

The return value was never used.

https://codereview.chromium.org/905003003/

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