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_;
On 2012/07/06 13:30:32, Jakob wrote:
nit: {} please

Done.

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6511
src/api.cc:6511: ASSERT((last_block_limit_ >= blocks_.first()) &&
On 2012/07/06 13:30:32, Jakob wrote:
last_block_limit_ should probably be renamed to first_block_limit_
now.

Done.

http://codereview.chromium.org/10696125/diff/1/src/api.cc#newcode6516
src/api.cc:6516: for (int i = 1; i < (blocks_.length() - 1); i++) {
On 2012/07/06 13:30:32, Jakob wrote:
This skips over the last block.

Done.

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

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

Reply via email to