http://codereview.chromium.org/650126/diff/4/5 File src/compiler.cc (right):
http://codereview.chromium.org/650126/diff/4/5#newcode227 src/compiler.cc:227: Debugger::OnAfterCompile(script, fun, false); On 2010/02/22 10:41:20, Søren Gjesse wrote:
It looks as if this is the only place this function is called, so I
don't see
why the extra argument is needed.
That's right. The actual usage is in one of my next CLs. This way I am trying to split one huge change into a smaller separately-compilable pieces. However I'm fine with merging this CL with the one that actually uses the 3rd argument. http://codereview.chromium.org/650126 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
