Bug 552329: Fix compile warnings in xpcshell test TestOOM.cpp. r=doug…

…t, test-only change on CLOSED TREE to see if tinderbox still wants to be red
mozilla · Mar 16, 2010 · 5eaa541 · 5eaa541
1 parent a393bc1
commit 5eaa541
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/xpcom/tests/TestOOM.cpp b/xpcom/tests/TestOOM.cpp
@@ -62,6 +62,7 @@ int main(int argc, char **argv)
   // the predicate should work with out having to rely on
   // that.
   void *big_alloc = malloc(1024 * 1024 * 16);
+  (void)big_alloc; // Tell compiler we're not using big_alloc, to fix warning
 
   const int highpower = 500000;
 
@@ -77,10 +78,10 @@ int main(int argc, char **argv)
 
     // You have to touch the buffer
     if (!buffers[i])
-      printf("Could not allocate a buffer of size %ld\n", s);
+      printf("Could not allocate a buffer of size %lu\n", (unsigned long)s);
     else
     {
-      for (int j=0; j<s; j++)
+      for (size_t j=0; j<s; j++)
         buffers[i][j] = 'a';
     }
 
@@ -89,10 +90,10 @@ int main(int argc, char **argv)
     PRIntervalTime cost = PR_IntervalNow() - start;
 
 
-    printf("Total Allocated: %ld. \tLow Memory now?  %s\t Took (%d)\n",
+    printf("Total Allocated: %lu. \tLow Memory now?  %s\t Took (%d)\n",
-           s*i,
+           (unsigned long)s*i,
-	   lowMem ? "Yes" : "No",
+           lowMem ? "Yes" : "No",
-	   PR_IntervalToMilliseconds(cost));
+           PR_IntervalToMilliseconds(cost));
 
     if (lowMem)
       break;