Changeset 65344 in webkit for trunk/JavaScriptCore/wtf/text/WTFString.cpp

Timestamp:

Aug 13, 2010, 6:28:27 PM (15 years ago)

Author:

[email protected]

Message:

Switch String::/UString::ascii() to return a CString.

Rubber stamped by Sam Weinig.

JavaScriptCore:

• JavaScriptCore.exp:
• JavaScriptCore.xcodeproj/project.pbxproj:
• bytecode/CodeBlock.cpp:

(JSC::CodeBlock::dump):

• bytecode/SamplingTool.cpp:

(JSC::SamplingTool::dump):

• interpreter/CallFrame.cpp:

(JSC::CallFrame::dumpCaller):

• jsc.cpp:

(runWithScripts):
(runInteractive):

• runtime/Identifier.h:

(JSC::Identifier::ascii):

• runtime/ScopeChain.cpp:

(JSC::ScopeChainNode::print):

• runtime/UString.cpp:

(JSC::UString::ascii):
(JSC::UString::latin1):

• runtime/UString.h:
• wtf/text/StringImpl.cpp:

(WTF::StringImpl::asciiOLD):

• wtf/text/StringImpl.h:
• wtf/text/WTFString.cpp:

(WTF::String::ascii):
(WTF::String::latin1):

• wtf/text/WTFString.h:

WebCore:

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

(JSC::Bindings::CClass::methodsNamed):
(JSC::Bindings::CClass::fieldNamed):

• bridge/jni/jsc/JNIBridgeJSC.cpp:

(JavaField::valueFromInstance):
(JavaField::setValueToInstance):

• bridge/jni/jsc/JavaInstanceJSC.cpp:

(JavaInstance::invokeMethod):

• bridge/objc/objc_class.mm:

(JSC::Bindings::ObjcClass::methodsNamed):
(JSC::Bindings::ObjcClass::fieldNamed):

• bridge/objc/objc_instance.mm:

(ObjcInstance::setValueOfUndefinedField):
(ObjcInstance::getValueOfUndefinedField):

• loader/icon/IconRecord.cpp:
• platform/sql/SQLiteDatabase.cpp:
• platform/sql/SQLiteStatement.cpp:
• storage/SQLStatement.cpp:

WebKit:

• WebKit.xcodeproj/project.pbxproj:

WebKit/mac:

• Plugins/Hosted/ProxyInstance.mm:

(WebKit::ProxyInstance::methodsNamed):
(WebKit::ProxyInstance::fieldNamed):

File:

• trunk/JavaScriptCore/wtf/text/WTFString.cpp (modified) (2 diffs)

trunk/JavaScriptCore/wtf/text/WTFString.cpp

@@ -613 +613 @@
 }
 
-Vector<char> String::ascii() const
-{
-    if (m_impl) 
-        return m_impl->ascii();
-    
-    const char* nullMsg = "(null impl)";
-    Vector<char, 2048> buffer;
-    for (int i = 0; nullMsg[i]; ++i)
-        buffer.append(nullMsg[i]);
-    
-    buffer.append('\0');
-    return buffer;
-}
-
-CString String::latin1() const
+CString String::ascii() const
 {
     // Basic Latin1 (ISO) encoding - Unicode characters 0..255 are
@@ -640 +626 @@
     for (unsigned i = 0; i < length; ++i) {
         UChar ch = characters[i];
-        characterBuffer[i] = ch > 255 ? '?' : ch;
+        characterBuffer[i] = ch && (ch < 0x20 || ch > 0x7f) ? '?' : ch;
+    }
+
+    return result;
+}
+
+CString String::latin1() const
+{
+    // Basic Latin1 (ISO) encoding - Unicode characters 0..255 are
+    // preserved, characters outside of this range are converted to '?'.
+
+    unsigned length = this->length();
+    const UChar* characters = this->characters();
+
+    char* characterBuffer;
+    CString result = CString::newUninitialized(length, characterBuffer);
+
+    for (unsigned i = 0; i < length; ++i) {
+        UChar ch = characters[i];
+        characterBuffer[i] = ch > 0xff ? '?' : ch;
     }