Reviewers: Vyacheslav Egorov,

Description:
Fixed an assertion in PagedSpace::Verify.

The formerly unused variable was found by GCC 4.6.1, BTW...

Please review this at http://codereview.chromium.org/7979005/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/spaces.cc


Index: src/spaces.cc
===================================================================
--- src/spaces.cc       (revision 9346)
+++ src/spaces.cc       (working copy)
@@ -785,7 +785,7 @@
   if (was_swept_conservatively_) return;

   bool allocation_pointer_found_in_space =
-      (allocation_info_.top != allocation_info_.limit);
+      (allocation_info_.top == allocation_info_.limit);
   PageIterator page_iterator(this);
   while (page_iterator.has_next()) {
     Page* page = page_iterator.next();
@@ -826,6 +826,7 @@

     CHECK_LE(black_size, page->LiveBytes());
   }
+  ASSERT(allocation_pointer_found_in_space);
 }
 #endif



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

Reply via email to