Reviewers: Søren Gjesse, Erik Corry,
Message:
Just FYI. I tried to unify PageIterator, but there are couple of places
where
has_next is used to determine if that's the last page.
And actually I placed two unnecessary .has_next's before when iterating by
PageIterator, they are removed now.
Description:
Merge ObjectIterator::has_next and ObjectIterator::next methods.
This reduces chances of improper usage, see
http://code.google.com/p/v8/issues/detail?id=586
for more details.
BUG=586
Please review this at http://codereview.chromium.org/555072
SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/
Affected files:
M src/debug.cc
M src/heap-profiler.cc
M src/heap.h
M src/heap.cc
M src/log.cc
M src/mark-compact.cc
M src/runtime.cc
M src/spaces-inl.h
M src/spaces.h
M src/spaces.cc
M test/cctest/test-api.cc
M test/cctest/test-debug.cc
M test/cctest/test-heap-profiler.cc
M test/cctest/test-heap.cc
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
