lgtm, though as discussed offline please add a test case to the regression test
for the undefined case.

https://codereview.chromium.org/1309303006/diff/20001/src/parser.cc
File src/parser.cc (right):

https://codereview.chromium.org/1309303006/diff/20001/src/parser.cc#newcode2993
src/parser.cc:2993: factory()->NewBlock(NULL, 2, false,
RelocInfo::kNoPosition);
We really ought to change this param to an enum so the callsites are
clearer, but that's a cleanup for a future CL.

https://codereview.chromium.org/1309303006/

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