LGTM

http://codereview.chromium.org/2817018/diff/7002/14004
File src/counters.cc (right):

http://codereview.chromium.org/2817018/diff/7002/14004#newcode31
src/counters.cc:31: #include "platform.h"
Please explicitly include "isolate.h".

http://codereview.chromium.org/2817018/diff/7002/14008
File src/counters.h (right):

http://codereview.chromium.org/2817018/diff/7002/14008#newcode58
src/counters.h:58: bool HasCounterFunction() {
"const"

http://codereview.chromium.org/2817018/diff/7002/14008#newcode68
src/counters.h:68: int* FindLocation(const char* name) {
"const"

http://codereview.chromium.org/2817018/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to