LGTM, nice cleanup.
http://codereview.chromium.org/6542017/diff/1/src/handles.cc File src/handles.cc (right): http://codereview.chromium.org/6542017/diff/1/src/handles.cc#newcode859 src/handles.cc:859: return CompileLazyFunction(function, flag, false /* in loop */); The C style comment is OK, but I don't think we have any examples of the style in the code already: You could also consider: const kInLoop = false; return CompileLazyFunction(function, flag, kInLoop); but it's not obviously better. http://codereview.chromium.org/6542017/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
