lgtm
http://codereview.chromium.org/8508052/diff/6020/src/scopes.cc File src/scopes.cc (right): http://codereview.chromium.org/8508052/diff/6020/src/scopes.cc#newcode269 src/scopes.cc:269: !top->outer_scope()->already_resolved()) { On 2011/11/15 11:53:23, Steven wrote:
I've updated the comment with a more thorough description. On 2011/11/14 15:20:52, fschneider wrote: > This loop condition seems inconsistent with the comment.
Maybe say: Traverse until the first unresolved scope or the global scope. http://codereview.chromium.org/8508052/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
