LGTM with comments addressed
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_; } Better make it a: bool IsInGCPostProcessing() { return gc_post_process_count_ > 0; } http://codereview.chromium.org/7105016/diff/1/src/heap.h#newcode1283 src/heap.h:1283: int gc_post_process_count_; Name is a bit confusing. Consider naming it: gc_post_processing_depth_ http://codereview.chromium.org/7105016/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
