LGTM
http://codereview.chromium.org/10270007/diff/1/src/preparser.h File src/preparser.h (right): http://codereview.chromium.org/10270007/diff/1/src/preparser.h#newcode474 src/preparser.h:474: class WithinWith { nit: not sure I like the name... how about InsideWith? http://codereview.chromium.org/10270007/diff/1/src/preparser.h#newcode476 src/preparser.h:476: explicit WithinWith(Scope* scope): scope_(scope) { nit: space before ':' http://codereview.chromium.org/10270007/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
