LGTM
http://codereview.chromium.org/6598023/diff/1/src/messages.js File src/messages.js (right): http://codereview.chromium.org/6598023/diff/1/src/messages.js#newcode229 src/messages.js:229: strict_function: ["In strict mode code, functions can only be declared at top level or immediately within another function." ], Seems like a very long (but informative) error message. The language here is very different from the other error messages. Can we use the same style: "Function declaration in local scope disallowed in strict mode" or something like that? http://codereview.chromium.org/6598023/diff/1/src/parser.cc File src/parser.cc (right): http://codereview.chromium.org/6598023/diff/1/src/parser.cc#newcode1111 src/parser.cc:1111: if (peek() == Token::FUNCTION) { You should add a comment here as well to make it clear why this parse statement call is treated specially. http://codereview.chromium.org/6598023/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
