Changeset 54789 in webkit for trunk/JavaScriptCore/bytecode

Timestamp:

Feb 15, 2010, 1:03:45 PM (15 years ago)

Author:

[email protected]

Message:

Bug 34952 - String lengths in UString should be unsigned.
This matches WebCore::StringImpl, and better unifies behaviour throughout JSC.

Reviewed by Geoff Garen.

JavaScriptCore:

• JavaScriptCore.exp:
• bytecode/EvalCodeCache.h:
• runtime/Identifier.cpp:

(JSC::Identifier::equal):

• runtime/Identifier.h:
• runtime/JSGlobalObjectFunctions.cpp:

(JSC::globalFuncEscape):

• runtime/JSONObject.cpp:

(JSC::gap):
(JSC::Stringifier::indent):

• runtime/NumberPrototype.cpp:

(JSC::numberProtoFuncToFixed):
(JSC::numberProtoFuncToPrecision):

• runtime/RegExp.cpp:

(JSC::RegExp::match):

• runtime/StringPrototype.cpp:

(JSC::substituteBackreferencesSlow):
(JSC::stringProtoFuncReplace):
(JSC::stringProtoFuncSplit):
(JSC::trimString):

• runtime/UString.cpp:

(JSC::UString::UString):
(JSC::UString::from):
(JSC::UString::getCString):
(JSC::UString::ascii):
(JSC::UString::operator[]):
(JSC::UString::toStrictUInt32):
(JSC::UString::find):
(JSC::UString::rfind):
(JSC::UString::substr):
(JSC::operator<):
(JSC::operator>):
(JSC::compare):
(JSC::equal):
(JSC::UString::UTF8String):

• runtime/UString.h:

(JSC::UString::size):
(JSC::operator==):

• runtime/UStringImpl.cpp:

(JSC::UStringImpl::create):

• runtime/UStringImpl.h:

(JSC::UStringImpl::create):
(JSC::UStringImpl::size):
(JSC::UStringImpl::computeHash):
(JSC::UStringImpl::UStringImpl):

WebCore:

• bindings/js/JSDOMWindowCustom.cpp:

(WebCore::JSDOMWindow::atob):
(WebCore::JSDOMWindow::btoa):

Location:

trunk/JavaScriptCore/bytecode

Files:

• CodeBlock.cpp (modified) (1 diff)
• EvalCodeCache.h (modified) (1 diff)

trunk/JavaScriptCore/bytecode/CodeBlock.cpp

@@ -50 +50 @@
 {
     UString result = str;
-    int pos = 0;
-    while ((pos = result.find('\"', pos)) >= 0) {
+    unsigned pos = 0;
+    while ((pos = result.find('\"', pos)) != UString::NotFound) {
         result = makeString(result.substr(0, pos), "\"\\\"\"", result.substr(pos + 1));
         pos += 4;

trunk/JavaScriptCore/bytecode/EvalCodeCache.h

@@ -66 +66 @@
 
     private:
-        static const int maxCacheableSourceLength = 256;
+        static const unsigned maxCacheableSourceLength = 256;
         static const int maxCacheEntries = 64;