https://codereview.chromium.org/12223071/diff/1/src/heap.cc
File src/heap.cc (right):

https://codereview.chromium.org/12223071/diff/1/src/heap.cc#newcode5180
src/heap.cc:5180: STATIC_ASSERT(HeapNumber::kSize <=
Page::kNonCodeObjectAreaSize);
On 2013/02/12 10:03:53, Michael Starzinger wrote:
Both comment and assertion are out-dated and use the wrong size
constant.

Oops, done.

https://codereview.chromium.org/12223071/diff/1/test/cctest/test-symbols.cc
File test/cctest/test-symbols.cc (right):

https://codereview.chromium.org/12223071/diff/1/test/cctest/test-symbols.cc#newcode11
test/cctest/test-symbols.cc:11: #include "cctest.h"
On 2013/02/12 10:03:53, Michael Starzinger wrote:
Can we alpha-sort the includes?

Done. Also removed obsolete import of "zone-inl.h".

https://codereview.chromium.org/12223071/

--
--
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.


Reply via email to