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

Timestamp:

Apr 21, 2010, 12:13:50 PM (15 years ago)

Author:

[email protected]

Message:

JavaScriptCore: Bug 37906 - Remove JSC::UStringImpl; unify with StringImpl.

Reviewed by Oliver Hunt, Darin Adler.

JSC::UStringImpl and WebCore::StringImpl (soon to be renamed to
WTF::StringImpl) are almost identical. Remove duplication of code by unifying
the two, move missing features from UStringImpl into StringImpl & delete the
class UStringImpl.

• API/JSClassRef.cpp:
• API/JSContextRef.cpp:
• GNUmakefile.am:
• JavaScriptCore.exp:
• JavaScriptCore.pro:
• JavaScriptCore.xcodeproj/project.pbxproj:
• bytecode/EvalCodeCache.h:
• bytecode/JumpTable.cpp:
• profiler/ProfileNode.cpp:
• runtime/Identifier.cpp:

(JSC::Identifier::add):

• runtime/Identifier.h:

(JSC::Identifier::equal):

• runtime/UString.cpp:
• runtime/UString.h:

(WTF::):

• runtime/UStringImpl.cpp: Removed.
• runtime/UStringImpl.h:
• wtf/text/StringHash.h:

(WebCore::StringHash::equal):
(WebCore::CaseFoldingHash::equal):

• wtf/text/StringImpl.cpp:

(WebCore::StringImpl::~StringImpl):
(WebCore::StringImpl::empty):
(WebCore::StringImpl::sharedBuffer):
(WebCore::equal):

• wtf/text/StringImpl.h:

(WebCore::StringImpl::StringImpl):
(WebCore::StringImpl::create):
(WebCore::StringImpl::tryCreateUninitialized):
(WebCore::StringImpl::cost):
(WebCore::StringImpl::isIdentifier):
(WebCore::StringImpl::setIsIdentifier):
(WebCore::StringImpl::computeHash):
(WebCore::StringImpl::copyChars):
(WebCore::StringImpl::):

JavaScriptGlue: Bug 37906 - Remove JSC::UStringImpl; unify with StringImpl.
Add forwarding header.

Reviewed by Oliver Hunt, Darin Adler.

• ForwardingHeaders/wtf/ASCIICType.h: Added.
• ForwardingHeaders/wtf/text/StringImpl.h: Added.

WebCore: Bug 37906 - Remove JSC::UStringImpl; unify with StringImpl.
Add include for StringHash.h.

Reviewed by Oliver Hunt, Darin Adler.

• WebCore.xcodeproj/project.pbxproj:
• bridge/c/c_class.cpp:

File:

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

trunk/JavaScriptCore/runtime/UString.cpp

@@ -588 +588 @@
 }
 
-bool equal(const UString::Rep* r, const UString::Rep* b)
-{
-    unsigned length = r->length();
-    if (length != b->length())
-        return false;
-    const UChar* d = r->characters();
-    const UChar* s = b->characters();
-    for (unsigned i = 0; i != length; ++i) {
-        if (d[i] != s[i])
-            return false;
-    }
-    return true;
-}
-
 CString UString::UTF8String(bool strict) const
 {