LGTM
http://codereview.chromium.org/3412034/diff/1/2 File src/parser.cc (right): http://codereview.chromium.org/3412034/diff/1/2#newcode996 src/parser.cc:996: virtual void ResumeRecording() { ASSERT(pause_count > 0? http://codereview.chromium.org/3412034/diff/1/2#newcode1376 src/parser.cc:1376: parent->type_ = type; // TODO(lrn): Remove if not needed. Please avoid TODO(username), use an issue number to track. http://codereview.chromium.org/3412034/diff/1/2#newcode1377 src/parser.cc:1377: // Initialize functiois n hijacked by DummyScope to increment scope depth. "functiois n"? http://codereview.chromium.org/3412034/diff/1/4 File src/scopes.cc (right): http://codereview.chromium.org/3412034/diff/1/4#newcode204 src/scopes.cc:204: void Scope::Leave() { This could be in the .h file. http://codereview.chromium.org/3412034/diff/1/5 File src/scopes.h (right): http://codereview.chromium.org/3412034/diff/1/5#newcode388 src/scopes.h:388: inside_with_level_(-1) { Shouldn't there be a constant for this value? http://codereview.chromium.org/3412034/diff/1/5#newcode395 src/scopes.h:395: if (inside_with && inside_with_level_ < 0) { Compare with constant instead of < 0? http://codereview.chromium.org/3412034/diff/1/5#newcode423 src/scopes.h:423: // Highest scope nesting level that is contained in a with statement, or -1 Highest -> Lowest/First? http://codereview.chromium.org/3412034/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
