Changeset 15385 in webkit for trunk/JavaScriptCore/API/testapi.c

Timestamp:

Jul 12, 2006, 3:01:06 AM (19 years ago)

Author:

ggaren

Message:

Reviewed by Maciej.

• Removed context and exception parameters from JSObjectGetPropertyEnumerator, removing the spurious use of ExecState inside JavaScriptCore that made us think this was necessary in the first place.

(StringInstance::getPropertyList): Use getString instead of toString because
we know we're dealing with a string -- we put it there in the first place.
While we're at it, store the string's size instead of retrieving it each time
through the loop, to avoid the unnecessary killing of puppies.

• kjs/string_object.h:

File:

• trunk/JavaScriptCore/API/testapi.c (modified) (3 diffs)

trunk/JavaScriptCore/API/testapi.c

@@ -163 +163 @@
 }
 
-static void MyObject_getPropertyList(JSContextRef context, JSObjectRef object, JSPropertyListRef propertyList, JSValueRef* exception)
+static void MyObject_addPropertiesToList(JSObjectRef object, JSPropertyListRef propertyList)
 {
     UNUSED_PARAM(context);
@@ -246 +246 @@
     MyObject_setProperty,
     MyObject_deleteProperty,
-    MyObject_getPropertyList,
+    MyObject_addPropertiesToList,
     MyObject_callAsFunction,
     MyObject_callAsConstructor,
@@ -538 +538 @@
     JSObjectSetProperty(context, o, jsOneIString, JSValueMakeNumber(1), kJSPropertyAttributeNone);
     JSObjectSetProperty(context, o, jsCFIString,  JSValueMakeNumber(1), kJSPropertyAttributeDontEnum);
-    JSPropertyEnumeratorRef enumerator = JSObjectCreatePropertyEnumerator(context, o);
+    JSPropertyEnumeratorRef enumerator = JSObjectCreatePropertyEnumerator(o);
     int count = 0;
     while (JSPropertyEnumeratorGetNextName(enumerator))