On 2012/07/10 18:46:08, Jakob wrote:
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.
Good point.
http://chromiumcodereview.appspot.com/10701116/
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
