Changeset 2843 in webkit for trunk/JavaScriptCore/kjs/array_object.cpp

Timestamp:

Nov 22, 2002, 11:49:05 PM (23 years ago)

Author:

darin

Message:

JavaScriptCore:

• replaced List class with a vector rather than a linked list, changed it to use a pool of instances instead of all the nodes allocated off of the heap; gives 10% gain on iBench

• kjs/list.h: Complete rewrite.
• kjs/list.cpp: Ditto.

• kjs/array_object.cpp: (compareWithCompareFunctionForQSort): Go back to doing a clear and two appends here. Fast with the new list implementation.

• kjs/collector.h: Remove _COLLECTOR hack and just make rootObjectClasses return a const void *.
• kjs/collector.cpp: Remove _COLLECTOR hack, and various other minor tweaks.

WebCore:

• khtml/ecma/kjs_window.cpp: Remove _COLLECTOR hack.

• kwq/WebCoreJavaScript.h:
• kwq/WebCoreJavaScript.mm: (+[WebCoreJavaScript rootObjectClasses]): Update for name change -- root object classes, not all live object classes.

• force-js-clean-timestamp: Make sure we don't have more build problems.

WebKit:

• Misc.subproj/WebCoreStatistics.h:
• Misc.subproj/WebCoreStatistics.m: (+[WebCoreStatistics javaScriptRootObjectClasses]): Update for name change -- root object classes, not all live object classes.

WebBrowser:

• Debug/CacheController.m: (-[CacheController refreshJavaScriptStatisticsMatrix]): Update for name change -- root object classes, not all live object classes.

File:

• trunk/JavaScriptCore/kjs/array_object.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/array_object.cpp

@@ -268 +268 @@
     CompareWithCompareFunctionArguments *args = compareWithCompareFunctionArguments;
     
-    args->arguments.replaceFirst(*(ValueImp **)a);
-    args->arguments.replaceLast(*(ValueImp **)b);
+    args->arguments.clear();
+    args->arguments.append(*(ValueImp **)a);
+    args->arguments.append(*(ValueImp **)b);
     return args->compareFunction->call(args->exec, args->globalObject, args->arguments)
         .toInt32(args->exec);