2007-08-02 Mark Rowe <mrowe@apple.com>
[WebKit-https.git] / JavaScriptCore / kjs / array_object.cpp
index 6371782..d2b8893 100644 (file)
@@ -2,7 +2,9 @@
 /*
  *  This file is part of the KDE libraries
  *  Copyright (C) 1999-2000 Harri Porten (porten@kde.org)
- *  Copyright (C) 2003 Apple Computer, Inc.
+ *  Copyright (C) 2003, 2007 Apple Inc. All rights reserved.
+ *  Copyright (C) 2003 Peter Kelly (pmk@post.com)
+ *  Copyright (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  *
  *  This library is free software; you can redistribute it and/or
  *  modify it under the terms of the GNU Lesser General Public
  *
  *  You should have received a copy of the GNU Lesser General Public
  *  License along with this library; if not, write to the Free Software
- *  Foundation, Inc., 51 Franklin Steet, Fifth Floor, Boston, MA  02110-1301  USA
+ *  Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
  *
  */
 
 #include "config.h"
 #include "array_object.h"
+#include "array_object.lut.h"
 
 #include "error_object.h"
-#include "internal.h"
-#include "interpreter.h"
-#include "object.h"
+#include "lookup.h"
 #include "operations.h"
-#include "reference_list.h"
-#include "types.h"
-#include "value.h"
-
-#include "array_object.lut.h"
-
+#include "PropertyNameArray.h"
+#include <wtf/HashSet.h>
 #include <stdio.h>
-#include <assert.h>
+
 
 using namespace KJS;
 
-// ------------------------------ ArrayInstanceImp -----------------------------
+// ------------------------------ ArrayInstance -----------------------------
 
 const unsigned sparseArrayCutoff = 10000;
 
-const ClassInfo ArrayInstanceImp::info = {"Array", 0, 0, 0};
+const ClassInfo ArrayInstance::info = {"Array", 0, 0, 0};
+
+static inline JSValue** allocateStorage(size_t capacity)
+{
+  if (capacity == 0)
+      return 0;
+
+  // store capacity in extra space before the beginning of the storage array to save space
+  JSValue** storage = static_cast<JSValue**>(fastCalloc(capacity + 1, sizeof(JSValue *))) + 1;
+  storage[-1] = reinterpret_cast<JSValue*>(capacity);
+  return storage;
+}
+
+static inline void reallocateStorage(JSValue**& storage, size_t newCapacity)
+{
+  if (!storage) {
+    storage =  allocateStorage(newCapacity);
+    return;
+  }
 
-ArrayInstanceImp::ArrayInstanceImp(ObjectImp *proto, unsigned initialLength)
-  : ObjectImp(proto)
+  // store capacity in extra space before the beginning of the storage array to save space
+  storage = static_cast<JSValue**>(fastRealloc(storage - 1, (newCapacity + 1) * sizeof (JSValue*))) + 1;
+  storage[-1] = reinterpret_cast<JSValue*>(newCapacity);
+}
+
+static inline void freeStorage(JSValue** storage)
+{
+  if (storage)
+    fastFree(storage - 1);
+}
+
+ArrayInstance::ArrayInstance(JSObject *proto, unsigned initialLength)
+  : JSObject(proto)
   , length(initialLength)
   , storageLength(initialLength < sparseArrayCutoff ? initialLength : 0)
-  , capacity(storageLength)
-  , storage(capacity ? (ValueImp **)fastCalloc(capacity, sizeof(ValueImp *)) : 0)
+  , storage(allocateStorage(storageLength))
 {
+  Collector::reportExtraMemoryCost(storageLength * sizeof(JSValue*));
 }
 
-ArrayInstanceImp::ArrayInstanceImp(ObjectImp *proto, const List &list)
-  : ObjectImp(proto)
+ArrayInstance::ArrayInstance(JSObject *proto, const List &list)
+  : JSObject(proto)
   , length(list.size())
   , storageLength(length)
-  , capacity(storageLength)
-  , storage(capacity ? (ValueImp **)fastMalloc(sizeof(ValueImp *) * capacity) : 0)
+  , storage(allocateStorage(storageLength))
 {
   ListIterator it = list.begin();
   unsigned l = length;
   for (unsigned i = 0; i < l; ++i) {
     storage[i] = it++;
   }
+  // When the array is created non-empty its cells are filled so it's really no worse than
+  // a property map. Therefore don't report extra memory cost.
+}
+
+ArrayInstance::~ArrayInstance()
+{
+  freeStorage(storage);
 }
 
-ArrayInstanceImp::~ArrayInstanceImp()
+JSValue* ArrayInstance::getItem(unsigned i) const
 {
-  fastFree(storage);
+    if (i >= length)
+        return jsUndefined();
+    
+    JSValue* val = (i < storageLength) ? 
+                            storage[i] :
+                            getDirect(Identifier::from(i));
+
+    return val ? val : jsUndefined();
 }
 
-ValueImp *ArrayInstanceImp::lengthGetter(ExecState *exec, const Identifier& propertyName, const PropertySlot& slot)
+JSValue *ArrayInstance::lengthGetter(ExecState*, JSObject*, const Identifier&, const PropertySlot& slot)
 {
-  return Number(static_cast<ArrayInstanceImp *>(slot.slotBase())->length);
+  return jsNumber(static_cast<ArrayInstance *>(slot.slotBase())->length);
 }
 
-bool ArrayInstanceImp::getOwnPropertySlot(ExecState *exec, const Identifier& propertyName, PropertySlot& slot)
+bool ArrayInstance::getOwnPropertySlot(ExecState* exec, const Identifier& propertyName, PropertySlot& slot)
 {
-  if (propertyName == lengthPropertyName) {
+  if (propertyName == exec->propertyNames().length) {
     slot.setCustom(this, lengthGetter);
     return true;
   }
@@ -91,37 +130,45 @@ bool ArrayInstanceImp::getOwnPropertySlot(ExecState *exec, const Identifier& pro
     if (index >= length)
       return false;
     if (index < storageLength) {
-      ValueImp *v = storage[index];
-      if (!v || v->isUndefined())
+      JSValue *v = storage[index];
+      if (!v)
         return false;      
       slot.setValueSlot(this, &storage[index]);
       return true;
     }
   }
 
-  return ObjectImp::getOwnPropertySlot(exec, propertyName, slot);
+  return JSObject::getOwnPropertySlot(exec, propertyName, slot);
 }
 
-bool ArrayInstanceImp::getOwnPropertySlot(ExecState *exec, unsigned index, PropertySlot& slot)
+bool ArrayInstance::getOwnPropertySlot(ExecState *exec, unsigned index, PropertySlot& slot)
 {
+  if (index > MAX_ARRAY_INDEX)
+    return getOwnPropertySlot(exec, Identifier::from(index), slot);
+
   if (index >= length)
     return false;
   if (index < storageLength) {
-    ValueImp *v = storage[index];
-    if (!v || v->isUndefined())
+    JSValue *v = storage[index];
+    if (!v)
       return false;
     slot.setValueSlot(this, &storage[index]);
     return true;
   }
 
-  return ObjectImp::getOwnPropertySlot(exec, index, slot);
+  return JSObject::getOwnPropertySlot(exec, index, slot);
 }
 
 // Special implementation of [[Put]] - see ECMA 15.4.5.1
-void ArrayInstanceImp::put(ExecState *exec, const Identifier &propertyName, ValueImp *value, int attr)
+void ArrayInstance::put(ExecState* exec, const Identifier& propertyName, JSValue* value, int attr)
 {
-  if (propertyName == lengthPropertyName) {
-    setLength(value->toUInt32(exec), exec);
+  if (propertyName == exec->propertyNames().length) {
+    unsigned int newLen = value->toUInt32(exec);
+    if (value->toNumber(exec) != double(newLen)) {
+      throwError(exec, RangeError, "Invalid array length.");
+      return;
+    }
+    setLength(newLen, exec);
     return;
   }
   
@@ -132,11 +179,18 @@ void ArrayInstanceImp::put(ExecState *exec, const Identifier &propertyName, Valu
     return;
   }
   
-  ObjectImp::put(exec, propertyName, value, attr);
+  JSObject::put(exec, propertyName, value, attr);
 }
 
-void ArrayInstanceImp::put(ExecState *exec, unsigned index, ValueImp *value, int attr)
+void ArrayInstance::put(ExecState *exec, unsigned index, JSValue *value, int attr)
 {
+  //0xFFFF FFFF is a bit weird --- it should be treated as a non-array index, even when
+  //it's a string 
+  if (index > MAX_ARRAY_INDEX) {
+    put(exec, Identifier::from(index), value, attr);
+    return;
+  }
+
   if (index < sparseArrayCutoff && index >= storageLength) {
     resizeStorage(index + 1);
   }
@@ -151,16 +205,16 @@ void ArrayInstanceImp::put(ExecState *exec, unsigned index, ValueImp *value, int
   }
   
   assert(index >= sparseArrayCutoff);
-  ObjectImp::put(exec, Identifier::from(index), value, attr);
+  JSObject::put(exec, Identifier::from(index), value, attr);
 }
 
-bool ArrayInstanceImp::deleteProperty(ExecState *exec, const Identifier &propertyName)
+bool ArrayInstance::deleteProperty(ExecState* exec, const Identifier &propertyName)
 {
-  if (propertyName == lengthPropertyName)
+  if (propertyName == exec->propertyNames().length)
     return false;
   
   bool ok;
-  uint32_t index = propertyName.toUInt32(&ok);
+  uint32_t index = propertyName.toArrayIndex(&ok);
   if (ok) {
     if (index >= length)
       return true;
@@ -170,11 +224,14 @@ bool ArrayInstanceImp::deleteProperty(ExecState *exec, const Identifier &propert
     }
   }
   
-  return ObjectImp::deleteProperty(exec, propertyName);
+  return JSObject::deleteProperty(exec, propertyName);
 }
 
-bool ArrayInstanceImp::deleteProperty(ExecState *exec, unsigned index)
+bool ArrayInstance::deleteProperty(ExecState *exec, unsigned index)
 {
+  if (index > MAX_ARRAY_INDEX)
+    return deleteProperty(exec, Identifier::from(index));
+
   if (index >= length)
     return true;
   if (index < storageLength) {
@@ -182,31 +239,30 @@ bool ArrayInstanceImp::deleteProperty(ExecState *exec, unsigned index)
     return true;
   }
   
-  return ObjectImp::deleteProperty(exec, Identifier::from(index));
+  return JSObject::deleteProperty(exec, Identifier::from(index));
 }
 
-ReferenceList ArrayInstanceImp::propList(ExecState *exec, bool recursive)
+void ArrayInstance::getPropertyNames(ExecState* exec, PropertyNameArray& propertyNames)
 {
-  ReferenceList properties = ObjectImp::propList(exec,recursive);
-
   // avoid fetching this every time through the loop
-  ValueImp *undefined = jsUndefined();
-
+  JSValue* undefined = jsUndefined();
+  
   for (unsigned i = 0; i < storageLength; ++i) {
-    ValueImp *imp = storage[i];
-    if (imp && imp != undefined) {
-      properties.append(Reference(this, i));
-    }
+    JSValue* value = storage[i];
+    if (value && value != undefined)
+      propertyNames.add(Identifier::from(i));
   }
-  return properties;
+  JSObject::getPropertyNames(exec, propertyNames);
 }
 
-void ArrayInstanceImp::resizeStorage(unsigned newLength)
+void ArrayInstance::resizeStorage(unsigned newLength)
 {
     if (newLength < storageLength) {
-      memset(storage + newLength, 0, sizeof(ValueImp *) * (storageLength - newLength));
+      memset(storage + newLength, 0, sizeof(JSValue *) * (storageLength - newLength));
     }
-    if (newLength > capacity) {
+    size_t cap = capacity();
+    if (newLength > cap) {
       unsigned newCapacity;
       if (newLength > sparseArrayCutoff) {
         newCapacity = newLength;
@@ -216,56 +272,56 @@ void ArrayInstanceImp::resizeStorage(unsigned newLength)
           newCapacity = sparseArrayCutoff;
         }
       }
-      storage = (ValueImp **)fastRealloc(storage, newCapacity * sizeof (ValueImp *));
-      memset(storage + capacity, 0, sizeof(ValueImp *) * (newCapacity - capacity));
-      capacity = newCapacity;
+      
+      reallocateStorage(storage, newCapacity);
+      memset(storage + cap, 0, sizeof(JSValue*) * (newCapacity - cap));
     }
     storageLength = newLength;
 }
 
-void ArrayInstanceImp::setLength(unsigned newLength, ExecState *exec)
+void ArrayInstance::setLength(unsigned newLength, ExecState *exec)
 {
   if (newLength <= storageLength) {
     resizeStorage(newLength);
   }
 
   if (newLength < length) {
-    ReferenceList sparseProperties;
+    PropertyNameArray sparseProperties;
+    
+    _prop.getSparseArrayPropertyNames(sparseProperties);
     
-    _prop.addSparseArrayPropertiesToReferenceList(sparseProperties, this);
+    PropertyNameArrayIterator end = sparseProperties.end();
     
-    ReferenceListIterator it = sparseProperties.begin();
-    while (it != sparseProperties.end()) {
-      Reference ref = it++;
+    for (PropertyNameArrayIterator it = sparseProperties.begin(); it != end; ++it) {
+      Identifier name = *it;
       bool ok;
-      unsigned index = ref.getPropertyName(exec).toArrayIndex(&ok);
-      if (ok && index > newLength) {
-       ref.deleteValue(exec);
-      }
+      unsigned index = name.toArrayIndex(&ok);
+      if (ok && index > newLength)
+        deleteProperty(exec, name);
     }
   }
   
   length = newLength;
 }
 
-void ArrayInstanceImp::mark()
+void ArrayInstance::mark()
 {
-  ObjectImp::mark();
+  JSObject::mark();
   unsigned l = storageLength;
   for (unsigned i = 0; i < l; ++i) {
-    ValueImp *imp = storage[i];
+    JSValue *imp = storage[i];
     if (imp && !imp->marked())
       imp->mark();
   }
 }
 
-static ExecState *execForCompareByStringForQSort;
+static ExecState* execForCompareByStringForQSort = 0;
 
 static int compareByStringForQSort(const void *a, const void *b)
 {
     ExecState *exec = execForCompareByStringForQSort;
-    ValueImp *va = *(ValueImp **)a;
-    ValueImp *vb = *(ValueImp **)b;
+    JSValue *va = *(JSValue **)a;
+    JSValue *vb = *(JSValue **)b;
     if (va->isUndefined()) {
         return vb->isUndefined() ? 0 : 1;
     }
@@ -275,39 +331,58 @@ static int compareByStringForQSort(const void *a, const void *b)
     return compare(va->toString(exec), vb->toString(exec));
 }
 
-void ArrayInstanceImp::sort(ExecState *exec)
+void ArrayInstance::sort(ExecState* exec)
 {
-    int lengthNotIncludingUndefined = pushUndefinedObjectsToEnd(exec);
-    
+    size_t lengthNotIncludingUndefined = compactForSorting();
+      
+    ExecState* oldExec = execForCompareByStringForQSort;
     execForCompareByStringForQSort = exec;
-    qsort(storage, lengthNotIncludingUndefined, sizeof(ValueImp *), compareByStringForQSort);
-    execForCompareByStringForQSort = 0;
+#if HAVE(MERGESORT)
+    // mergesort usually does fewer compares, so it is actually faster than qsort for JS sorts.
+    // however, becuase it requires extra copies of the storage buffer, don't use it for very
+    // large arrays
+    // FIXME: for sorting by string value, the fastest thing would actually be to convert all the
+    // values to string once up front, and then use a radix sort. That would be O(N) rather than 
+    // O(N log N).
+    if (lengthNotIncludingUndefined < sparseArrayCutoff) {
+        JSValue** storageCopy = allocateStorage(capacity());
+        memcpy(storageCopy, storage, capacity() * sizeof(JSValue*));
+        mergesort(storageCopy, lengthNotIncludingUndefined, sizeof(JSValue *), compareByStringForQSort);
+        freeStorage(storage);
+        storage = storageCopy;
+        execForCompareByStringForQSort = oldExec;
+        return;
+    }
+#endif
+
+    qsort(storage, lengthNotIncludingUndefined, sizeof(JSValue*), compareByStringForQSort);
+    execForCompareByStringForQSort = oldExec;
 }
 
 struct CompareWithCompareFunctionArguments {
-    CompareWithCompareFunctionArguments(ExecState *e, ObjectImp *cf)
+    CompareWithCompareFunctionArguments(ExecState *e, JSObject *cf)
         : exec(e)
         , compareFunction(cf)
         , globalObject(e->dynamicInterpreter()->globalObject())
     {
-        arguments.append(Undefined());
-        arguments.append(Undefined());
+        arguments.append(jsUndefined());
+        arguments.append(jsUndefined());
     }
 
     ExecState *exec;
-    ObjectImp *compareFunction;
+    JSObject *compareFunction;
     List arguments;
-    ObjectImp *globalObject;
+    JSObject *globalObject;
 };
 
-static CompareWithCompareFunctionArguments *compareWithCompareFunctionArguments;
+static CompareWithCompareFunctionArguments* compareWithCompareFunctionArguments = 0;
 
 static int compareWithCompareFunctionForQSort(const void *a, const void *b)
 {
     CompareWithCompareFunctionArguments *args = compareWithCompareFunctionArguments;
 
-    ValueImp *va = *(ValueImp **)a;
-    ValueImp *vb = *(ValueImp **)b;
+    JSValue *va = *(JSValue **)a;
+    JSValue *vb = *(JSValue **)b;
     if (va->isUndefined()) {
         return vb->isUndefined() ? 0 : 1;
     }
@@ -323,189 +398,205 @@ static int compareWithCompareFunctionForQSort(const void *a, const void *b)
     return compareResult < 0 ? -1 : compareResult > 0 ? 1 : 0;
 }
 
-void ArrayInstanceImp::sort(ExecState *exec, ObjectImp *compareFunction)
+void ArrayInstance::sort(ExecState* exec, JSObject* compareFunction)
 {
-    int lengthNotIncludingUndefined = pushUndefinedObjectsToEnd(exec);
-    
+    size_t lengthNotIncludingUndefined = compactForSorting();
+
+    CompareWithCompareFunctionArguments* oldArgs = compareWithCompareFunctionArguments;
     CompareWithCompareFunctionArguments args(exec, compareFunction);
     compareWithCompareFunctionArguments = &args;
-    qsort(storage, lengthNotIncludingUndefined, sizeof(ValueImp *), compareWithCompareFunctionForQSort);
-    compareWithCompareFunctionArguments = 0;
+#if HAVE(MERGESORT)
+    // mergesort usually does fewer compares, so it is actually faster than qsort for JS sorts.
+    // however, becuase it requires extra copies of the storage buffer, don't use it for very
+    // large arrays
+    // FIXME: a tree sort using a perfectly balanced tree (e.g. an AVL tree) could do an even
+    // better job of minimizing compares
+    if (lengthNotIncludingUndefined < sparseArrayCutoff) {
+        JSValue** storageCopy = allocateStorage(capacity());
+        memcpy(storageCopy, storage, capacity() * sizeof(JSValue*));
+        mergesort(storageCopy, lengthNotIncludingUndefined, sizeof(JSValue *), compareWithCompareFunctionForQSort);
+        freeStorage(storage);
+        storage = storageCopy;
+        compareWithCompareFunctionArguments = oldArgs;
+        return;
+    }
+#endif
+    qsort(storage, lengthNotIncludingUndefined, sizeof(JSValue*), compareWithCompareFunctionForQSort);
+    compareWithCompareFunctionArguments = oldArgs;
 }
 
-unsigned ArrayInstanceImp::pushUndefinedObjectsToEnd(ExecState *exec)
+unsigned ArrayInstance::compactForSorting()
 {
-    ValueImp *undefined = jsUndefined();
+    JSValue *undefined = jsUndefined();
 
     unsigned o = 0;
     
     for (unsigned i = 0; i != storageLength; ++i) {
-        ValueImp *v = storage[i];
+        JSValue *v = storage[i];
         if (v && v != undefined) {
             if (o != i)
                 storage[o] = v;
             o++;
         }
     }
+   
+    PropertyNameArray sparseProperties;
+    _prop.getSparseArrayPropertyNames(sparseProperties);
+    unsigned newLength = o + sparseProperties.size();
     
-    ReferenceList sparseProperties;
-    _prop.addSparseArrayPropertiesToReferenceList(sparseProperties, this);
-    unsigned newLength = o + sparseProperties.length();
-
-    if (newLength > storageLength) {
-      resizeStorage(newLength);
-    } 
-
-    ReferenceListIterator it = sparseProperties.begin();
-    while (it != sparseProperties.end()) {
-      Reference ref = it++;
-      storage[o] = ref.getValue(exec);
-      ObjectImp::deleteProperty(exec, ref.getPropertyName(exec));
-      o++;
+    if (newLength > storageLength)
+        resizeStorage(newLength);
+    
+    PropertyNameArrayIterator end = sparseProperties.end();
+    for (PropertyNameArrayIterator it = sparseProperties.begin(); it != end; ++it) {
+        Identifier name = *it;
+        storage[o] = getDirect(name);
+        _prop.remove(name);
+        o++;
     }
     
     if (newLength != storageLength)
-        memset(storage + o, 0, sizeof(ValueImp *) * (storageLength - o));
+        memset(storage + o, 0, sizeof(JSValue *) * (storageLength - o));
     
     return o;
 }
 
-// ------------------------------ ArrayPrototypeImp ----------------------------
+// ------------------------------ ArrayPrototype ----------------------------
 
-const ClassInfo ArrayPrototypeImp::info = {"Array", &ArrayInstanceImp::info, &arrayTable, 0};
+const ClassInfo ArrayPrototype::info = {"Array", &ArrayInstance::info, &arrayTable, 0};
 
 /* Source for array_object.lut.h
 @begin arrayTable 16
-  toString       ArrayProtoFuncImp::ToString       DontEnum|Function 0
-  toLocaleString ArrayProtoFuncImp::ToLocaleString DontEnum|Function 0
-  concat         ArrayProtoFuncImp::Concat         DontEnum|Function 1
-  join           ArrayProtoFuncImp::Join           DontEnum|Function 1
-  pop            ArrayProtoFuncImp::Pop            DontEnum|Function 0
-  push           ArrayProtoFuncImp::Push           DontEnum|Function 1
-  reverse        ArrayProtoFuncImp::Reverse        DontEnum|Function 0
-  shift          ArrayProtoFuncImp::Shift          DontEnum|Function 0
-  slice          ArrayProtoFuncImp::Slice          DontEnum|Function 2
-  sort           ArrayProtoFuncImp::Sort           DontEnum|Function 1
-  splice         ArrayProtoFuncImp::Splice         DontEnum|Function 2
-  unshift        ArrayProtoFuncImp::UnShift        DontEnum|Function 1
-  every          ArrayProtoFuncImp::Every          DontEnum|Function 5
-  forEach        ArrayProtoFuncImp::ForEach        DontEnum|Function 5
-  some           ArrayProtoFuncImp::Some           DontEnum|Function 5
+  toString       ArrayProtoFunc::ToString       DontEnum|Function 0
+  toLocaleString ArrayProtoFunc::ToLocaleString DontEnum|Function 0
+  concat         ArrayProtoFunc::Concat         DontEnum|Function 1
+  join           ArrayProtoFunc::Join           DontEnum|Function 1
+  pop            ArrayProtoFunc::Pop            DontEnum|Function 0
+  push           ArrayProtoFunc::Push           DontEnum|Function 1
+  reverse        ArrayProtoFunc::Reverse        DontEnum|Function 0
+  shift          ArrayProtoFunc::Shift          DontEnum|Function 0
+  slice          ArrayProtoFunc::Slice          DontEnum|Function 2
+  sort           ArrayProtoFunc::Sort           DontEnum|Function 1
+  splice         ArrayProtoFunc::Splice         DontEnum|Function 2
+  unshift        ArrayProtoFunc::UnShift        DontEnum|Function 1
+  every          ArrayProtoFunc::Every          DontEnum|Function 1
+  forEach        ArrayProtoFunc::ForEach        DontEnum|Function 1
+  some           ArrayProtoFunc::Some           DontEnum|Function 1
+  indexOf        ArrayProtoFunc::IndexOf        DontEnum|Function 1
+  lastIndexOf    ArrayProtoFunc::LastIndexOf    DontEnum|Function 1
+  filter         ArrayProtoFunc::Filter         DontEnum|Function 1
+  map            ArrayProtoFunc::Map            DontEnum|Function 1
 @end
 */
 
 // ECMA 15.4.4
-ArrayPrototypeImp::ArrayPrototypeImp(ExecState *exec,
-                                     ObjectPrototypeImp *objProto)
-  : ArrayInstanceImp(objProto, 0)
+ArrayPrototype::ArrayPrototype(ExecState*, ObjectPrototype* objProto)
+  : ArrayInstance(objProto, 0)
 {
-  setInternalValue(Null());
 }
 
-bool ArrayPrototypeImp::getOwnPropertySlot(ExecState *exec, const Identifier& propertyName, PropertySlot& slot)
+bool ArrayPrototype::getOwnPropertySlot(ExecState* exec, const Identifier& propertyName, PropertySlot& slot)
 {
-  return getStaticFunctionSlot<ArrayProtoFuncImp, ArrayInstanceImp>(exec, &arrayTable, this, propertyName, slot);
+  return getStaticFunctionSlot<ArrayProtoFunc, ArrayInstance>(exec, &arrayTable, this, propertyName, slot);
 }
 
-// ------------------------------ ArrayProtoFuncImp ----------------------------
-
-ArrayProtoFuncImp::ArrayProtoFuncImp(ExecState *exec, int i, int len)
-  : InternalFunctionImp(
-    static_cast<FunctionPrototypeImp*>(exec->lexicalInterpreter()->builtinFunctionPrototype())
-    ), id(i)
-{
-  put(exec,lengthPropertyName,Number(len),DontDelete|ReadOnly|DontEnum);
-}
+// ------------------------------ ArrayProtoFunc ----------------------------
 
-bool ArrayProtoFuncImp::implementsCall() const
+ArrayProtoFunc::ArrayProtoFunc(ExecState* exec, int i, int len, const Identifier& name)
+  : InternalFunctionImp(static_cast<FunctionPrototype*>
+                        (exec->lexicalInterpreter()->builtinFunctionPrototype()), name)
+  , id(i)
 {
-  return true;
+  put(exec, exec->propertyNames().length, jsNumber(len), DontDelete | ReadOnly | DontEnum);
 }
 
-static ValueImp *getProperty(ExecState *exec, ObjectImp *obj, unsigned index)
+static JSValue *getProperty(ExecState *exec, JSObject *obj, unsigned index)
 {
     PropertySlot slot;
     if (!obj->getPropertySlot(exec, index, slot))
         return NULL;
-    return slot.getValue(exec, index);
+    return slot.getValue(exec, obj, index);
 }
 
 // ECMA 15.4.4
-ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj, const List &args)
+JSValue* ArrayProtoFunc::callAsFunction(ExecState* exec, JSObject* thisObj, const List& args)
 {
-  unsigned length = thisObj->get(exec,lengthPropertyName)->toUInt32(exec);
+  unsigned length = thisObj->get(exec, exec->propertyNames().length)->toUInt32(exec);
 
-  ValueImp *result = 0; // work around gcc 4.0 bug in uninitialized variable warning
+  JSValue *result = 0; // work around gcc 4.0 bug in uninitialized variable warning
   
   switch (id) {
   case ToLocaleString:
   case ToString:
 
-    if (!thisObj->inherits(&ArrayInstanceImp::info))
+    if (!thisObj->inherits(&ArrayInstance::info))
       return throwError(exec, TypeError);
 
     // fall through
   case Join: {
+    static HashSet<JSObject*> visitedElems;
+    if (visitedElems.contains(thisObj))
+        return jsString("");
     UString separator = ",";
     UString str = "";
 
-    if (!args[0]->isUndefined())
-      separator = args[0]->toString(exec);
+    visitedElems.add(thisObj);
+    if (id == Join && !args[0]->isUndefined())
+        separator = args[0]->toString(exec);
     for (unsigned int k = 0; k < length; k++) {
-      if (k >= 1)
-        str += separator;
-      
-      ValueImp *element = thisObj->get(exec, k);
-      if (element->isUndefinedOrNull())
-        continue;
+        if (k >= 1)
+            str += separator;
+        if (str.isNull()) {
+            JSObject *error = Error::create(exec, GeneralError, "Out of memory");
+            exec->setException(error);
+            break;
+        }
 
-      bool fallback = false;
-      if (id == ToLocaleString) {
-        ObjectImp *o = element->toObject(exec);
-        ValueImp *conversionFunction = o->get(exec, toLocaleStringPropertyName);
-        if (conversionFunction->isObject() && static_cast<ObjectImp *>(conversionFunction)->implementsCall()) {
-          str += static_cast<ObjectImp *>(conversionFunction)->call(exec, o, List())->toString(exec);
-        } else {
-          // try toString() fallback
-          fallback = true;
+        JSValue* element = thisObj->get(exec, k);
+        if (element->isUndefinedOrNull())
+            continue;
+
+        bool fallback = false;
+        if (id == ToLocaleString) {
+            JSObject* o = element->toObject(exec);
+            JSValue* conversionFunction = o->get(exec, exec->propertyNames().toLocaleString);
+            if (conversionFunction->isObject() && static_cast<JSObject*>(conversionFunction)->implementsCall())
+                str += static_cast<JSObject*>(conversionFunction)->call(exec, o, List())->toString(exec);
+            else
+                // try toString() fallback
+                fallback = true;
         }
-      }
 
-      if (id == ToString || id == Join || fallback) {
-        if (element->isObject()) {
-          ObjectImp *o = static_cast<ObjectImp *>(element);
-          ValueImp *conversionFunction = o->get(exec, toStringPropertyName);
-          if (conversionFunction->isObject() && static_cast<ObjectImp *>(conversionFunction)->implementsCall()) {
-            str += static_cast<ObjectImp *>(conversionFunction)->call(exec, o, List())->toString(exec);
-          } else {
-            return throwError(exec, RangeError, "Can't convert " + o->className() + " object to string");
-          }
-        } else {
-          str += element->toString(exec);
+        if (id == ToString || id == Join || fallback)
+            str += element->toString(exec);
+
+        if (str.isNull()) {
+            JSObject *error = Error::create(exec, GeneralError, "Out of memory");
+            exec->setException(error);
         }
-      }
 
-      if ( exec->hadException() )
-        break;
+        if (exec->hadException())
+            break;
     }
-    result = String(str);
+    visitedElems.remove(thisObj);
+    result = jsString(str);
     break;
   }
   case Concat: {
-    ObjectImp *arr = static_cast<ObjectImp *>(exec->lexicalInterpreter()->builtinArray()->construct(exec,List::empty()));
+    JSObject *arr = static_cast<JSObject *>(exec->lexicalInterpreter()->builtinArray()->construct(exec,List::empty()));
     int n = 0;
-    ValueImp *curArg = thisObj;
-    ObjectImp *curObj = static_cast<ObjectImp *>(thisObj);
+    JSValue *curArg = thisObj;
+    JSObject *curObj = static_cast<JSObject *>(thisObj);
     ListIterator it = args.begin();
     for (;;) {
       if (curArg->isObject() &&
-          curObj->inherits(&ArrayInstanceImp::info)) {
+          curObj->inherits(&ArrayInstance::info)) {
         unsigned int k = 0;
         // Older versions tried to optimize out getting the length of thisObj
         // by checking for n != 0, but that doesn't work if thisObj is an empty array.
-        length = curObj->get(exec,lengthPropertyName)->toUInt32(exec);
+        length = curObj->get(exec, exec->propertyNames().length)->toUInt32(exec);
         while (k < length) {
-          if (ValueImp *v = getProperty(exec, curObj, k))
+          if (JSValue *v = getProperty(exec, curObj, k))
             arr->put(exec, n, v);
           n++;
           k++;
@@ -517,20 +608,20 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
       if (it == args.end())
         break;
       curArg = *it;
-      curObj = static_cast<ObjectImp *>(it++); // may be 0
+      curObj = static_cast<JSObject *>(it++); // may be 0
     }
-    arr->put(exec,lengthPropertyName, Number(n), DontEnum | DontDelete);
+    arr->put(exec, exec->propertyNames().length, jsNumber(n), DontEnum | DontDelete);
 
     result = arr;
     break;
   }
   case Pop:{
     if (length == 0) {
-      thisObj->put(exec, lengthPropertyName, Number(length), DontEnum | DontDelete);
-      result = Undefined();
+      thisObj->put(exec, exec->propertyNames().length, jsNumber(length), DontEnum | DontDelete);
+      result = jsUndefined();
     } else {
       result = thisObj->get(exec, length - 1);
-      thisObj->put(exec, lengthPropertyName, Number(length - 1), DontEnum | DontDelete);
+      thisObj->put(exec, exec->propertyNames().length, jsNumber(length - 1), DontEnum | DontDelete);
     }
     break;
   }
@@ -538,8 +629,8 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
     for (int n = 0; n < args.size(); n++)
       thisObj->put(exec, length + n, args[n]);
     length += args.size();
-    thisObj->put(exec,lengthPropertyName, Number(length), DontEnum | DontDelete);
-    result = Number(length);
+    thisObj->put(exec, exec->propertyNames().length, jsNumber(length), DontEnum | DontDelete);
+    result = jsNumber(length);
     break;
   }
   case Reverse: {
@@ -548,8 +639,8 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
 
     for (unsigned int k = 0; k < middle; k++) {
       unsigned lk1 = length - k - 1;
-      ValueImp *obj2 = getProperty(exec, thisObj, lk1);
-      ValueImp *obj = getProperty(exec, thisObj, k);
+      JSValue *obj2 = getProperty(exec, thisObj, lk1);
+      JSValue *obj = getProperty(exec, thisObj, k);
 
       if (obj2) 
         thisObj->put(exec, k, obj2);
@@ -566,18 +657,18 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
   }
   case Shift: {
     if (length == 0) {
-      thisObj->put(exec, lengthPropertyName, Number(length), DontEnum | DontDelete);
-      result = Undefined();
+      thisObj->put(exec, exec->propertyNames().length, jsNumber(length), DontEnum | DontDelete);
+      result = jsUndefined();
     } else {
       result = thisObj->get(exec, 0);
       for(unsigned int k = 1; k < length; k++) {
-        if (ValueImp *obj = getProperty(exec, thisObj, k))
+        if (JSValue *obj = getProperty(exec, thisObj, k))
           thisObj->put(exec, k-1, obj);
         else
           thisObj->deleteProperty(exec, k-1);
       }
       thisObj->deleteProperty(exec, length - 1);
-      thisObj->put(exec, lengthPropertyName, Number(length - 1), DontEnum | DontDelete);
+      thisObj->put(exec, exec->propertyNames().length, jsNumber(length - 1), DontEnum | DontDelete);
     }
     break;
   }
@@ -585,7 +676,7 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
     // http://developer.netscape.com/docs/manuals/js/client/jsref/array.htm#1193713 or 15.4.4.10
 
     // We return a new array
-    ObjectImp *resObj = static_cast<ObjectImp *>(exec->lexicalInterpreter()->builtinArray()->construct(exec,List::empty()));
+    JSObject *resObj = static_cast<JSObject *>(exec->lexicalInterpreter()->builtinArray()->construct(exec,List::empty()));
     result = resObj;
     double begin = 0;
     if (!args[0]->isUndefined()) {
@@ -617,10 +708,10 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
     int b = static_cast<int>(begin);
     int e = static_cast<int>(end);
     for(int k = b; k < e; k++, n++) {
-      if (ValueImp *v = getProperty(exec, thisObj, k))
+      if (JSValue *v = getProperty(exec, thisObj, k))
         resObj->put(exec, n, v);
     }
-    resObj->put(exec, lengthPropertyName, Number(n), DontEnum | DontDelete);
+    resObj->put(exec, exec->propertyNames().length, jsNumber(n), DontEnum | DontDelete);
     break;
   }
   case Sort:{
@@ -629,7 +720,7 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
     for ( unsigned int i = 0 ; i<length ; ++i )
       printf("KJS Array::Sort: %d: %s\n", i, thisObj->get(exec, i)->toString(exec).ascii() );
 #endif
-    ObjectImp *sortFunction = NULL;
+    JSObject *sortFunction = NULL;
     if (!args[0]->isUndefined())
       {
         sortFunction = args[0]->toObject(exec);
@@ -637,17 +728,17 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
           sortFunction = NULL;
       }
     
-    if (thisObj->classInfo() == &ArrayInstanceImp::info) {
+    if (thisObj->classInfo() == &ArrayInstance::info) {
       if (sortFunction)
-        ((ArrayInstanceImp *)thisObj)->sort(exec, sortFunction);
+        ((ArrayInstance *)thisObj)->sort(exec, sortFunction);
       else
-        ((ArrayInstanceImp *)thisObj)->sort(exec);
+        ((ArrayInstance *)thisObj)->sort(exec);
       result = thisObj;
       break;
     }
 
     if (length == 0) {
-      thisObj->put(exec, lengthPropertyName, Number(0), DontEnum | DontDelete);
+      thisObj->put(exec, exec->propertyNames().length, jsNumber(0), DontEnum | DontDelete);
       result = thisObj;
       break;
     }
@@ -656,12 +747,12 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
     // or quicksort, and much less swapping than bubblesort/insertionsort.
     for ( unsigned int i = 0 ; i<length-1 ; ++i )
       {
-        ValueImp *iObj = thisObj->get(exec,i);
+        JSValue *iObj = thisObj->get(exec,i);
         unsigned int themin = i;
-        ValueImp *minObj = iObj;
+        JSValue *minObj = iObj;
         for ( unsigned int j = i+1 ; j<length ; ++j )
           {
-            ValueImp *jObj = thisObj->get(exec,j);
+            JSValue *jObj = thisObj->get(exec,j);
             double cmp;
             if (jObj->isUndefined()) {
               cmp = 1; // don't check minObj because there's no need to differentiate == (0) from > (1)
@@ -699,7 +790,7 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
   }
   case Splice: {
     // 15.4.4.12 - oh boy this is huge
-    ObjectImp *resObj = static_cast<ObjectImp *>(exec->lexicalInterpreter()->builtinArray()->construct(exec,List::empty()));
+    JSObject *resObj = static_cast<JSObject *>(exec->lexicalInterpreter()->builtinArray()->construct(exec,List::empty()));
     result = resObj;
     int begin = args[0]->toUInt32(exec);
     if ( begin < 0 )
@@ -710,10 +801,10 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
 
     //printf( "Splicing from %d, deleteCount=%d \n", begin, deleteCount );
     for(unsigned int k = 0; k < deleteCount; k++) {
-      if (ValueImp *v = getProperty(exec, thisObj, k+begin))
+      if (JSValue *v = getProperty(exec, thisObj, k+begin))
         resObj->put(exec, k, v);
     }
-    resObj->put(exec, lengthPropertyName, Number(deleteCount), DontEnum | DontDelete);
+    resObj->put(exec, exec->propertyNames().length, jsNumber(deleteCount), DontEnum | DontDelete);
 
     unsigned int additionalArgs = maxInt( args.size() - 2, 0 );
     if ( additionalArgs != deleteCount )
@@ -722,7 +813,7 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
       {
         for ( unsigned int k = begin; k < length - deleteCount; ++k )
         {
-          if (ValueImp *v = getProperty(exec, thisObj, k+deleteCount))
+          if (JSValue *v = getProperty(exec, thisObj, k+deleteCount))
             thisObj->put(exec, k+additionalArgs, v);
           else
             thisObj->deleteProperty(exec, k+additionalArgs);
@@ -734,7 +825,7 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
       {
         for ( unsigned int k = length - deleteCount; (int)k > begin; --k )
         {
-          if (ValueImp *obj = getProperty(exec, thisObj, k + deleteCount - 1))
+          if (JSValue *obj = getProperty(exec, thisObj, k + deleteCount - 1))
             thisObj->put(exec, k + additionalArgs - 1, obj);
           else
             thisObj->deleteProperty(exec, k+additionalArgs-1);
@@ -745,24 +836,67 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
     {
       thisObj->put(exec, k+begin, args[k+2]);
     }
-    thisObj->put(exec, lengthPropertyName, Number(length - deleteCount + additionalArgs), DontEnum | DontDelete);
+    thisObj->put(exec, exec->propertyNames().length, jsNumber(length - deleteCount + additionalArgs), DontEnum | DontDelete);
     break;
   }
   case UnShift: { // 15.4.4.13
     unsigned int nrArgs = args.size();
     for ( unsigned int k = length; k > 0; --k )
     {
-      if (ValueImp *v = getProperty(exec, thisObj, k - 1))
+      if (JSValue *v = getProperty(exec, thisObj, k - 1))
         thisObj->put(exec, k+nrArgs-1, v);
       else
         thisObj->deleteProperty(exec, k+nrArgs-1);
     }
     for ( unsigned int k = 0; k < nrArgs; ++k )
       thisObj->put(exec, k, args[k]);
-    result = Number(length + nrArgs);
-    thisObj->put(exec, lengthPropertyName, result, DontEnum | DontDelete);
+    result = jsNumber(length + nrArgs);
+    thisObj->put(exec, exec->propertyNames().length, result, DontEnum | DontDelete);
     break;
   }
+  case Filter:
+  case Map: {
+    JSObject *eachFunction = args[0]->toObject(exec);
+    
+    if (!eachFunction->implementsCall())
+      return throwError(exec, TypeError);
+    
+    JSObject *applyThis = args[1]->isUndefinedOrNull() ? exec->dynamicInterpreter()->globalObject() :  args[1]->toObject(exec);
+    JSObject *resultArray;
+    
+    if (id == Filter) 
+      resultArray = static_cast<JSObject *>(exec->lexicalInterpreter()->builtinArray()->construct(exec, List::empty()));
+    else {
+      List args;
+      args.append(jsNumber(length));
+      resultArray = static_cast<JSObject *>(exec->lexicalInterpreter()->builtinArray()->construct(exec, args));
+    }
+    
+    unsigned filterIndex = 0;
+    for (unsigned k = 0; k < length && !exec->hadException(); ++k) {
+      PropertySlot slot;
+
+      if (!thisObj->getPropertySlot(exec, k, slot))
+         continue;
+        
+      JSValue *v = slot.getValue(exec, thisObj, k);
+      
+      List eachArguments;
+      
+      eachArguments.append(v);
+      eachArguments.append(jsNumber(k));
+      eachArguments.append(thisObj);
+      
+      JSValue *result = eachFunction->call(exec, applyThis, eachArguments);
+      
+      if (id == Map)
+        resultArray->put(exec, k, result);
+      else if (result->toBoolean(exec)) 
+        resultArray->put(exec, filterIndex++, v);
+    }
+    
+    return resultArray;
+  }
   case Every:
   case ForEach:
   case Some: {
@@ -771,40 +905,96 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
     //http://developer-test.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Objects:Array:forEach
     //http://developer-test.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Objects:Array:some
     
-    ObjectImp *eachFunction = args[0]->toObject(exec);
+    JSObject *eachFunction = args[0]->toObject(exec);
     
     if (!eachFunction->implementsCall())
       return throwError(exec, TypeError);
     
-    ObjectImp *applyThis = args[1]->isUndefinedOrNull() ? exec->dynamicInterpreter()->globalObject() :  args[1]->toObject(exec);
+    JSObject *applyThis = args[1]->isUndefinedOrNull() ? exec->dynamicInterpreter()->globalObject() :  args[1]->toObject(exec);
     
     if (id == Some || id == Every)
-      result = Boolean(id == Every);
+      result = jsBoolean(id == Every);
     else
-      result = thisObj;
+      result = jsUndefined();
     
     for (unsigned k = 0; k < length && !exec->hadException(); ++k) {
+      PropertySlot slot;
+        
+      if (!thisObj->getPropertySlot(exec, k, slot))
+        continue;
       
       List eachArguments;
       
-      eachArguments.append(thisObj->get(exec, k));
-      eachArguments.append(Number(k));
+      eachArguments.append(slot.getValue(exec, thisObj, k));
+      eachArguments.append(jsNumber(k));
       eachArguments.append(thisObj);
       
       bool predicateResult = eachFunction->call(exec, applyThis, eachArguments)->toBoolean(exec);
       
       if (id == Every && !predicateResult) {
-        result = Boolean(false);
+        result = jsBoolean(false);
         break;
       }
       if (id == Some && predicateResult) {
-        result = Boolean(true);
+        result = jsBoolean(true);
         break;
       }
     }
     break;
   }
-    
+
+  case IndexOf: {
+    // JavaScript 1.5 Extension by Mozilla
+    // Documentation: http://developer.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Global_Objects:Array:indexOf
+
+    unsigned index = 0;
+    double d = args[1]->toInteger(exec);
+    if (d < 0)
+        d += length;
+    if (d > 0) {
+        if (d > length)
+            index = length;
+        else
+            index = static_cast<unsigned>(d);
+    }
+
+    JSValue* searchElement = args[0];
+    for (; index < length; ++index) {
+        JSValue* e = getProperty(exec, thisObj, index);
+        if (!e)
+            continue;
+        if (strictEqual(exec, searchElement, e))
+            return jsNumber(index);
+    }
+
+    return jsNumber(-1);
+  }
+  case LastIndexOf: {
+       // JavaScript 1.6 Extension by Mozilla
+      // Documentation: http://developer.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Global_Objects:Array:lastIndexOf 
+
+    int index = length - 1;
+    double d = args[1]->toInteger(exec);
+
+    if (d < 0) {
+        d += length;
+        if (d < 0) 
+            return jsNumber(-1);
+    }
+    if (d < length)
+        index = static_cast<int>(d);
+          
+    JSValue* searchElement = args[0];
+    for (; index >= 0; --index) {
+        JSValue* e = getProperty(exec, thisObj, index);
+        if (!e)
+            continue;
+        if (strictEqual(exec, searchElement, e))
+            return jsNumber(index);
+    }
+          
+    return jsNumber(-1);
+}
   default:
     assert(0);
     result = 0;
@@ -816,15 +1006,15 @@ ValueImp *ArrayProtoFuncImp::callAsFunction(ExecState *exec, ObjectImp *thisObj,
 // ------------------------------ ArrayObjectImp -------------------------------
 
 ArrayObjectImp::ArrayObjectImp(ExecState *exec,
-                               FunctionPrototypeImp *funcProto,
-                               ArrayPrototypeImp *arrayProto)
+                               FunctionPrototype *funcProto,
+                               ArrayPrototype *arrayProto)
   : InternalFunctionImp(funcProto)
 {
   // ECMA 15.4.3.1 Array.prototype
-  put(exec,prototypePropertyName, arrayProto, DontEnum|DontDelete|ReadOnly);
+  put(exec, exec->propertyNames().prototype, arrayProto, DontEnum|DontDelete|ReadOnly);
 
   // no. of arguments for constructor
-  put(exec,lengthPropertyName, Number(1), ReadOnly|DontDelete|DontEnum);
+  put(exec, exec->propertyNames().length, jsNumber(1), ReadOnly|DontDelete|DontEnum);
 }
 
 bool ArrayObjectImp::implementsConstruct() const
@@ -833,27 +1023,22 @@ bool ArrayObjectImp::implementsConstruct() const
 }
 
 // ECMA 15.4.2
-ObjectImp *ArrayObjectImp::construct(ExecState *exec, const List &args)
+JSObject *ArrayObjectImp::construct(ExecState *exec, const List &args)
 {
   // a single numeric argument denotes the array size (!)
   if (args.size() == 1 && args[0]->isNumber()) {
     uint32_t n = args[0]->toUInt32(exec);
     if (n != args[0]->toNumber(exec))
       return throwError(exec, RangeError, "Array size is not a small enough positive integer.");
-    return new ArrayInstanceImp(exec->lexicalInterpreter()->builtinArrayPrototype(), n);
+    return new ArrayInstance(exec->lexicalInterpreter()->builtinArrayPrototype(), n);
   }
 
   // otherwise the array is constructed with the arguments in it
-  return new ArrayInstanceImp(exec->lexicalInterpreter()->builtinArrayPrototype(), args);
-}
-
-bool ArrayObjectImp::implementsCall() const
-{
-  return true;
+  return new ArrayInstance(exec->lexicalInterpreter()->builtinArrayPrototype(), args);
 }
 
 // ECMA 15.6.1
-ValueImp *ArrayObjectImp::callAsFunction(ExecState *exec, ObjectImp */*thisObj*/, const List &args)
+JSValue *ArrayObjectImp::callAsFunction(ExecState *exec, JSObject *, const List &args)
 {
   // equivalent to 'new Array(....)'
   return construct(exec,args);