http://codereview.chromium.org/7105016/diff/1/src/heap.h
File src/heap.h (right):

http://codereview.chromium.org/7105016/diff/1/src/heap.h#newcode1057
src/heap.h:1057: inline int gc_post_process_count() { return
gc_post_process_count_; }
On 2011/06/01 11:12:35, Vyacheslav Egorov wrote:
Better make it a:

bool IsInGCPostProcessing() { return gc_post_process_count_ > 0; }

Done.

http://codereview.chromium.org/7105016/diff/1/src/heap.h#newcode1283
src/heap.h:1283: int gc_post_process_count_;
On 2011/06/01 11:12:35, Vyacheslav Egorov wrote:
Name is a bit confusing. Consider naming it:

gc_post_processing_depth_


Done.

http://codereview.chromium.org/7105016/

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

Reply via email to