Fixed a goof in bmalloc Vector sizing
authorggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 12 Sep 2014 23:47:56 +0000 (23:47 +0000)
committerggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 12 Sep 2014 23:47:56 +0000 (23:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=136795

Reviewed by Gavin Barraclough and Sam Weinig.

We want our minimum vector to be page-sized since the OS will give us
a page no matter what -- but we want that many bytes, and not enough
bytes to store that many elements.

* bmalloc/Vector.h: Math is hard.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@173589 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/bmalloc/ChangeLog
Source/bmalloc/bmalloc/Vector.h

index 9cbed4c..5fffda4 100644 (file)
@@ -1,3 +1,16 @@
+2014-09-12  Geoffrey Garen  <ggaren@apple.com>
+
+        Fixed a goof in bmalloc Vector sizing
+        https://bugs.webkit.org/show_bug.cgi?id=136795
+
+        Reviewed by Gavin Barraclough and Sam Weinig.
+
+        We want our minimum vector to be page-sized since the OS will give us
+        a page no matter what -- but we want that many bytes, and not enough
+        bytes to store that many elements.
+
+        * bmalloc/Vector.h: Math is hard.
+
 2014-09-11  Geoffrey Garen  <ggaren@apple.com>
 
         bmalloc should segregate medium-sized objects by line like it does for small-sized objects
index ebcf94d..5a0688e 100644 (file)
@@ -66,7 +66,7 @@ public:
 private:
     static const size_t growFactor = 2;
     static const size_t shrinkFactor = 4;
-    static const size_t initialCapacity = vmPageSize;
+    static const size_t initialCapacity = vmPageSize / sizeof(T);
 
     void growCapacity(size_t size);
     void shrinkCapacity();