Changeset 4792 in webkit for trunk/JavaScriptCore/kjs/identifier.h

Timestamp:

Aug 8, 2003, 8:21:12 AM (22 years ago)

Author:

darin

Message:

Reviewed by John Sullivan.

• fixed 3365527 -- subscripting JavaScript strings does not work (leads to hang at www.newmagna.com.au)

The JavaScript specification says nothing about this, but other browsers seem to give
read-only access to the characters in a string as if the string was an array of characters.

• kjs/array_object.cpp: (ArrayInstanceImp::get): Update to use a public toArrayIndex function instead of our own getArrayIndex function, so we can share with string. (ArrayInstanceImp::put): Ditto. (ArrayInstanceImp::hasProperty): Ditto. (ArrayInstanceImp::setLength): Ditto.

• kjs/ustring.h: Add toArrayIndex.
• kjs/ustring.cpp: (UString::toArrayIndex): Added. Implements the rule from array.
• kjs/identifier.h: Add a forwarding function so we can use toArrayIndex.

• kjs/string_object.cpp: (StringInstanceImp::get): Return a single character string if the property name is an array index. (StringInstanceImp::hasProperty): Return true for property names that are suitable array indices.

• JavaScriptCore.pbproj/project.pbxproj: Let Xcode be Xcode.

File:

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

trunk/JavaScriptCore/kjs/identifier.h

@@ -54 +54 @@
         uint32_t toUInt32(bool *ok) const { return _ustring.toUInt32(ok); }
         uint32_t toStrictUInt32(bool *ok) const { return _ustring.toStrictUInt32(ok); }
+        unsigned toArrayIndex(bool *ok) const { return _ustring.toArrayIndex(ok); }
         double toDouble() const { return _ustring.toDouble(); }