LGTM!

http://codereview.chromium.org/6777011/diff/3001/src/heap.cc
File src/heap.cc (right):

http://codereview.chromium.org/6777011/diff/3001/src/heap.cc#newcode1252
src/heap.cc:1252: static inline void DoScavengeObject(Map* map,
This probably needs INLINE to prevent surprises with gcc.

http://codereview.chromium.org/6777011/diff/3001/src/heap.cc#newcode1531
src/heap.cc:1531: // However this does not lead any bugs as such isolate
does not have
"lead to"

http://codereview.chromium.org/6777011/diff/3001/src/heap.cc#newcode1533
src/heap.cc:1533: // to see table it in the consistent state.
"table it" -> "the table"

http://codereview.chromium.org/6777011/diff/3001/src/heap.cc#newcode4823
src/heap.cc:4823: initialized_gc = true;
nit: Is indentation off here?

http://codereview.chromium.org/6777011/

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

Reply via email to