LGTM

https://chromiumcodereview.appspot.com/10078015/diff/1/test/cctest/test-heap-profiler.cc
File test/cctest/test-heap-profiler.cc (right):

https://chromiumcodereview.appspot.com/10078015/diff/1/test/cctest/test-heap-profiler.cc#newcode5
test/cctest/test-heap-profiler.cc:5: #include <ctype.h>
Sorry, didn't spot that in your original CL. "v8.h" is a "project
header", thus it goes after system headers.

https://chromiumcodereview.appspot.com/10078015/

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

Reply via email to