lgtm
https://chromiumcodereview.appspot.com/9121033/diff/1/src/heap-inl.h File src/heap-inl.h (right): https://chromiumcodereview.appspot.com/9121033/diff/1/src/heap-inl.h#newcode730 src/heap-inl.h:730: void VerifyPointersVisitor::VisitPointers(Object** start, Object** end) { I think this will not compile in release mode (class is not defined). https://chromiumcodereview.appspot.com/9121033/diff/1/src/heap-inl.h#newcode784 src/heap-inl.h:784: pDisableAssertNoAllocation::~DisableAssertNoAllocation() { } I think this will not compile. https://chromiumcodereview.appspot.com/9121033/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
