Some cleanup of the mark-sweep/compact collector following the symbol...

src/mark-compact.h

Index: src/mark-compact.h
===================================================================
--- src/mark-compact.h	(revision 1853)
+++ src/mark-compact.h	(working copy)
@@ -142,7 +142,6 @@
 
   friend class RootMarkingVisitor;
   friend class MarkingVisitor;
-  friend class UnmarkingVisitor;
 
   // Marking operations for objects reachable from roots.
   static void MarkLiveObjects();
@@ -156,7 +155,7 @@
   static inline void SetMark(HeapObject* obj) {
     tracer_->increment_marked_count();
 #ifdef DEBUG
-    UpdateLiveObjectCount(obj, 1);
+    UpdateLiveObjectCount(obj);
 #endif
     obj->SetMark();
   }
@@ -204,13 +203,10 @@
   static void RefillMarkingStack();
 
   // Callback function for telling whether the object *p must be marked.

[Kasper Lund, 2009/05/05 09:23:14]

Update comment?

[Kevin Millikin (Chromium), 2009/05/05 09:29:53]

Thanks, done.

-  static bool MustBeMarked(Object** p);
+  static bool IsUnmarkedHeapObject(Object** p);
 
 #ifdef DEBUG
-  // The scale argument is positive 1 if we are marking an object and
-  // -1 if we are clearing the mark bit of an object that we didn't
-  // actually want marked.
-  static void UpdateLiveObjectCount(HeapObject* obj, int scale);
+  static void UpdateLiveObjectCount(HeapObject* obj);
 #endif
 
   // We sweep the large object space in the same way whether we are