Changeset 40975 in webkit for trunk/JavaScriptCore/runtime/UString.cpp

Timestamp:

Feb 13, 2009, 8:16:50 AM (16 years ago)

Author:

Darin Adler

Message:

2009-02-12 Darin Adler <Darin Adler>

Reviewed by Oliver Hunt and Alexey Proskuryakov.

Speed up a couple string functions.

• runtime/StringPrototype.cpp: (JSC::stringProtoFuncIndexOf): Added a fast path for cases where the second argument is either missing or an integer. (JSC::stringProtoFuncBig): Use jsNontrivialString since the string is guaranteed to be 2 or more characters long. (JSC::stringProtoFuncSmall): Ditto. (JSC::stringProtoFuncBlink): Ditto. (JSC::stringProtoFuncBold): Ditto. (JSC::stringProtoFuncItalics): Ditto. (JSC::stringProtoFuncStrike): Ditto. (JSC::stringProtoFuncSub): Ditto. (JSC::stringProtoFuncSup): Ditto. (JSC::stringProtoFuncFontcolor): Ditto. (JSC::stringProtoFuncFontsize): Make the fast path Sam recently added even faster by avoiding all but the minimum memory allocation. (JSC::stringProtoFuncAnchor): Use jsNontrivialString. (JSC::stringProtoFuncLink): Added a fast path.

• runtime/UString.cpp: (JSC::UString::find): Added a fast path for single-character search strings.

File:

• trunk/JavaScriptCore/runtime/UString.cpp (modified) (1 diff)

trunk/JavaScriptCore/runtime/UString.cpp

@@ -1420 +1420 @@
 int UString::find(const UString& f, int pos) const
 {
+    int fsz = f.size();
+
+    if (pos < 0)
+        pos = 0;
+
+    if (fsz == 1) {
+        UChar ch = f[0];
+        const UChar* end = data() + size();
+        for (const UChar* c = data() + pos; c < end; c++) {
+            if (*c == ch)
+                return static_cast<int>(c - data());
+        }
+        return -1;
+    }
+
     int sz = size();
-    int fsz = f.size();
     if (sz < fsz)
         return -1;
-    if (pos < 0)
-        pos = 0;
     if (fsz == 0)
         return pos;