http://codereview.chromium.org/10696125/diff/1/src/api.cc
File src/api.cc (right):

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6474
src/api.cc:6474: deferred_iterator = deferred_iterator->previous_;
nit: {} please

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6511
src/api.cc:6511: ASSERT((last_block_limit_ >= blocks_.first()) &&
last_block_limit_ should probably be renamed to first_block_limit_ now.

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6516
src/api.cc:6516: for (int i = 1; i < (blocks_.length() - 1); i++) {
This skips over the last block.

http://codereview.chromium.org/10696125/

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

Reply via email to