Changeset 36435 in webkit for trunk/JavaScriptCore

Timestamp:

Sep 15, 2008, 3:02:56 AM (17 years ago)

Author:

[email protected]

Message:

2008-09-15 Maciej Stachowiak <​[email protected]>

Reviewed by Cameron Zwarich.

• fix JSC test failures introduced by fix for 20849

• kjs/PropertyMap.cpp: (JSC::PropertyMap::getEnumerablePropertyNames): Use the correct count.

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• kjs/PropertyMap.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-09-15  Maciej Stachowiak  <[email protected]>
+
+        Reviewed by Cameron Zwarich.
+        
+        - fix JSC test failures introduced by fix for 20849
+
+        * kjs/PropertyMap.cpp:
+        (JSC::PropertyMap::getEnumerablePropertyNames): Use the correct count.
+
 2008-09-15  Cameron Zwarich  <[email protected]>
 

trunk/JavaScriptCore/kjs/PropertyMap.cpp

@@ -509 +509 @@
     }
 
+    size_t enumerableCount = p - sortedEnumerables.data();
     // Sort the entries by index.
-    qsort(sortedEnumerables.data(), p - sortedEnumerables.data(), sizeof(Entry*), comparePropertyMapEntryIndices);
+    qsort(sortedEnumerables.data(), enumerableCount, sizeof(Entry*), comparePropertyMapEntryIndices);
+    sortedEnumerables.resize(enumerableCount);
 
     // Put the keys of the sorted entries into the list.