LGTM.

http://chromiumcodereview.appspot.com/10701116/diff/1/src/preparser.cc
File src/preparser.cc (right):

http://chromiumcodereview.appspot.com/10701116/diff/1/src/preparser.cc#newcode605
src/preparser.cc:605: Expect(i::Token::COLON, CHECK_OK);
nit: The "Expect(...COLON...)" line is in both the if and the else
blocks, so you could move it out and put it after line 609, but I don't
mind either way.

http://chromiumcodereview.appspot.com/10701116/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to