Changeset 1821 in webkit for trunk/JavaScriptCore/kjs/array_object.h

Timestamp:

Aug 14, 2002, 5:59:18 PM (23 years ago)

Author:

mjs

Message:

Fix major JavaScript memory leak. run-plt says cvs-base improved
by 2% and cvs-js-performance improved by 7%. However, this was
within the possible noise level in each case.

The fix was to store ValueImp *'s in the array instead of Value
objects, since the Value wrapper will keep a ref and make the
object immortal.

• kjs/array_object.cpp: (ArrayInstanceImp::ArrayInstanceImp): (ArrayInstanceImp::get): (ArrayInstanceImp::put): (ArrayInstanceImp::hasProperty): (ArrayInstanceImp::deleteProperty): (ArrayInstanceImp::setLength): (ArrayInstanceImp::mark):
• kjs/array_object.h:

File:

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

trunk/JavaScriptCore/kjs/array_object.h

@@ -55 +55 @@
     unsigned length;
     unsigned capacity;
-    Value *storage;
+    ValueImp **storage;
   };