https://codereview.chromium.org/23201016/diff/1/src/runtime.cc
File src/runtime.cc (right):
https://codereview.chromium.org/23201016/diff/1/src/runtime.cc#newcode8314
src/runtime.cc:8314: if (frame->function() == function_)
has_function_activations_ = true;
On 2013/08/21 16:20:28, Michael Starzinger wrote:
The "has_function_activations" field seems to be unused.
Correct, twers for debuggin'
https://codereview.chromium.org/23201016/diff/1/src/runtime.cc#newcode8341
src/runtime.cc:8341: Handle<Code>
optimized_code(deoptimizer->compiled_code());
On 2013/08/21 16:20:28, Michael Starzinger wrote:
nit: Use an assignment instead of the copy constructor ...
Handle<JSFunction> function = deoptimizer->function();
Handle<Code> optimized_code = deoptimizer->compiled_code();
Done.
https://codereview.chromium.org/23201016/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
