Bug 1508818, part 2 - Remove DEBUG_rickg from nsDeque.cpp. r=froydnj
authorAndrew McCreight <continuation@gmail.com>
Wed, 28 Nov 2018 21:06:07 +0000
changeset 507804 b226d4c40893fdd080f65c9d8d8a52990a5386be
parent 507803 41195aed7eef4e1575d37e806bc53876043ad376
child 507805 67b1d1e433013e79b67684b48c4c074712e2509e
push id1905
push userffxbld-merge
push dateMon, 21 Jan 2019 12:33:13 +0000
treeherdermozilla-release@c2fca1944d8c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfroydnj
bugs1508818
milestone65.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1508818, part 2 - Remove DEBUG_rickg from nsDeque.cpp. r=froydnj Clang format makes this code look pretty bad, but I think it is safe to just remove it. Depends on D13182 Differential Revision: https://phabricator.services.mozilla.com/D13183
xpcom/ds/nsDeque.cpp
--- a/xpcom/ds/nsDeque.cpp
+++ b/xpcom/ds/nsDeque.cpp
@@ -2,19 +2,16 @@
 /* vim: set ts=8 sts=2 et sw=2 tw=80: */
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 #include "nsDeque.h"
 #include "nsISupportsImpl.h"
 #include <string.h>
-#ifdef DEBUG_rickg
-#include <stdio.h>
-#endif
 
 #include "mozilla/CheckedInt.h"
 
 #define modulus(x,y) ((x)%(y))
 
 /**
  * Standard constructor
  * @param deallocator, called by Erase and ~nsDeque
@@ -31,38 +28,16 @@ nsDeque::nsDeque(nsDequeFunctor* aDeallo
 
 /**
  * Destructor
  */
 nsDeque::~nsDeque()
 {
   MOZ_COUNT_DTOR(nsDeque);
 
-#ifdef DEBUG_rickg
-  char buffer[30];
-  printf("Capacity: %i\n", mCapacity);
-
-  static int mCaps[15] = {0};
-  switch (mCapacity) {
-    case 4:     mCaps[0]++; break;
-    case 8:     mCaps[1]++; break;
-    case 16:    mCaps[2]++; break;
-    case 32:    mCaps[3]++; break;
-    case 64:    mCaps[4]++; break;
-    case 128:   mCaps[5]++; break;
-    case 256:   mCaps[6]++; break;
-    case 512:   mCaps[7]++; break;
-    case 1024:  mCaps[8]++; break;
-    case 2048:  mCaps[9]++; break;
-    case 4096:  mCaps[10]++; break;
-    default:
-      break;
-  }
-#endif
-
   Erase();
   if (mData && mData != mBuffer) {
     free(mData);
   }
   mData = 0;
   SetDeallocator(0);
 }