Bug 1011355 (part 3) - Tiny FreeSpan tweaks. r=billm.
authorNicholas Nethercote <nnethercote@mozilla.com>
Thu, 15 May 2014 22:16:27 -0700
changeset 183431 5385817bdae605fab0c8415e12fe6b1965d79df3
parent 183430 6a3df055066a6c942c20b1e600be06c3eacadd29
child 183432 5e18fd30243f4e6709a77be6ce612ea6ca671f42
push idunknown
push userunknown
push dateunknown
reviewersbillm
bugs1011355
milestone32.0a1
Bug 1011355 (part 3) - Tiny FreeSpan tweaks. r=billm.
js/src/gc/Heap.h
--- a/js/src/gc/Heap.h
+++ b/js/src/gc/Heap.h
@@ -151,18 +151,16 @@ class FreeSpan
     friend class ArenaCellIterImpl;
     friend class CompactFreeSpan;
     friend class FreeList;
 
     uintptr_t   first;
     uintptr_t   last;
 
   public:
-    FreeSpan() {}
-
     // This inits just |first| and |last|; if the span is non-empty it doesn't
     // do anything with the next span stored at |last|.
     void initBoundsUnchecked(uintptr_t first, uintptr_t last) {
         this->first = first;
         this->last = last;
     }
 
     void initBounds(uintptr_t first, uintptr_t last) {
@@ -231,16 +229,17 @@ class FreeSpan
 
             /* If we find it before the end of the span, it's free. */
             if (thing <= span->last)
                 return true;
         }
         return false;
     }
 
+  private:
     // Some callers can pass in |thingSize| easily, and we can do stronger
     // checking in that case.
     void checkSpan(size_t thingSize = 0) const {
 #ifdef DEBUG
         if (!first || !last) {
             JS_ASSERT(!first && !last);
             // An empty span.
             return;