http://codereview.chromium.org/5720003/diff/12001/src/runtime-profiler.cc File src/runtime-profiler.cc (right):
http://codereview.chromium.org/5720003/diff/12001/src/runtime-profiler.cc#newcode353 src/runtime-profiler.cc:353: static void AddStateSample(SamplerState current_state) { On 2010/12/10 14:40:40, Vitaly wrote:
Rename to "UpdateStateRatio"?
Done. http://codereview.chromium.org/5720003/diff/12001/src/runtime-profiler.cc#newcode374 src/runtime-profiler.cc:374: AddStateSample(IN_JS_STATE); On 2010/12/10 14:40:40, Vitaly wrote:
Nice place to use ?: operator. Up to you though.
Done. http://codereview.chromium.org/5720003/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
