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

Timestamp:

Apr 30, 2009, 4:36:38 PM (16 years ago)

Author:

[email protected]

Message:

2009-04-30 Maciej Stachowiak <​[email protected]>

Reviewed by Gavin Barraclough.

• Concatenate final three strings in simple replace case at one go

~0.2% SunSpider speedup

• runtime/StringPrototype.cpp: (JSC::stringProtoFuncReplace): Use new replaceRange helper instead of taking substrings and concatenating three strings.
• runtime/UString.cpp: (JSC::UString::replaceRange): New helper function.
• runtime/UString.h:

File:

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

trunk/JavaScriptCore/runtime/UString.cpp

@@ -982 +982 @@
 }
 
+UString UString::replaceRange(int rangeStart, int rangeLength, const UString& replacement) const
+{
+    m_rep->checkConsistency();
+
+    int replacementLength = replacement.size();
+    int totalLength = size() - rangeLength + replacementLength;
+    if (totalLength == 0)
+        return "";
+
+    UChar* buffer = allocChars(totalLength);
+    if (!buffer)
+        return null();
+
+    copyChars(buffer, data(), rangeStart);
+    copyChars(buffer + rangeStart, replacement.data(), replacementLength);
+    int rangeEnd = rangeStart + rangeLength;
+    copyChars(buffer + rangeStart + replacementLength, data() + rangeEnd, size() - rangeEnd);
+
+    return UString::Rep::create(buffer, totalLength);
+}
+
+
 UString& UString::append(const UString &t)
 {