Changeset 10180 in webkit for trunk/JavaScriptCore/kjs/object.h

Timestamp:

Aug 14, 2005, 9:27:13 AM (20 years ago)

Author:

darin

Message:

Reviewed by Maciej.

• fixed ​http://bugzilla.opendarwin.org/show_bug.cgi?id=4417 speed up JavaScript with some small changes to the property map code

my measurements show an improvement of 2% on iBench JavaScript

• kjs/property_map.h: (KJS::PropertyMap::PropertyMap): Made the default constructor inline.
• kjs/property_map.cpp: (KJS::PropertyMap::~PropertyMap): Changed loop to exit early once we know we've processed all the hash table entries, based on the count. (KJS::PropertyMap::mark): Ditto.

• kjs/object.h: Made an arbitrary change here to force recompiling so we pick up changes to property_map.h. Works around what seems to be an Xcode header dependency bug.

File:

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

trunk/JavaScriptCore/kjs/object.h

@@ -617 +617 @@
   };
 
-    inline bool AllocatedValueImp::isObject(const ClassInfo *info) const
-        { return isObject() && static_cast<const ObjectImp *>(this)->inherits(info); }
+inline bool AllocatedValueImp::isObject(const ClassInfo *info) const
+{
+    return isObject() && static_cast<const ObjectImp *>(this)->inherits(info);
+}
 
 inline ObjectImp::ObjectImp(ObjectImp *proto)