Reviewed by Maciej.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Aug 2004 16:12:00 +0000 (16:12 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Aug 2004 16:12:00 +0000 (16:12 +0000)
        - added an ascii() member function to DOMString and DOMStringImpl to help debugging

        * khtml/dom/dom_string.h: Add ascii member function for debugging.
        * khtml/dom/dom_string.cpp: (DOM::DOMString::ascii): Added. Calls through to DOMStringImpl::ascii.
        * khtml/xml/dom_stringimpl.h: Add ascii member function for debugging.
        * khtml/xml/dom_stringimpl.cpp: (DOM::DOMStringImpl::ascii): Added. Makes a new buffer and puts
        a simple ASCII version in it. Maybe make it better about characters outside the 0x20-0x7E range
        some day, but for now this is way better than what we had before.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@7321 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/khtml/dom/dom_string.cpp
WebCore/khtml/dom/dom_string.h
WebCore/khtml/xml/dom_stringimpl.cpp
WebCore/khtml/xml/dom_stringimpl.h

index 88e9e4d6a8ce943cbe3285cc25699e10325af165..419ccf2af8f6fb9ba83b6808dcf9acffdd6870e9 100644 (file)
@@ -1,3 +1,16 @@
+2004-08-20  Darin Adler  <darin@apple.com>
+
+        Reviewed by Maciej.
+
+        - added an ascii() member function to DOMString and DOMStringImpl to help debugging
+
+        * khtml/dom/dom_string.h: Add ascii member function for debugging.
+        * khtml/dom/dom_string.cpp: (DOM::DOMString::ascii): Added. Calls through to DOMStringImpl::ascii.
+        * khtml/xml/dom_stringimpl.h: Add ascii member function for debugging.
+        * khtml/xml/dom_stringimpl.cpp: (DOM::DOMStringImpl::ascii): Added. Makes a new buffer and puts
+        a simple ASCII version in it. Maybe make it better about characters outside the 0x20-0x7E range
+        some day, but for now this is way better than what we had before.
+
 2004-08-20  David Hyatt  <hyatt@apple.com>
 
        Divorce the notion of a marquee being stopped from JS from the notion of being suspended by the back/forward cache.
@@ -10,7 +23,6 @@
         (Marquee::stop):
         (Marquee::updateMarqueePosition):
         * khtml/rendering/render_layer.h:
-        (khtml::):
 
 2004-08-20  Richard Williamson   <rjw@apple.com>
 
index ea804b477c52e9713b8101fff5d091b1a3d3843c..8ed0a81eab0a8d02e79d0a6df614945f2498cfbb 100644 (file)
@@ -270,6 +270,11 @@ khtml::Length* DOMString::toLengthArray(int& len) const
     return impl ? impl->toLengthArray(len) : 0;
 }
 
+const char *DOMString::ascii() const
+{
+    return impl ? impl->ascii() : "(null impl)";
+}
+
 //-----------------------------------------------------------------------------
 
 bool DOM::operator==( const DOMString &a, const DOMString &b )
@@ -316,5 +321,4 @@ bool DOM::operator==( const DOMString &a, const char *b )
     return *b == 0;
 }
 
-
 }
index 88e9855c4e87c67168a2c5fc7d0febb599ac426c..00da8a2d21ba6f6f568564efc1125a327060350d 100644 (file)
@@ -120,6 +120,10 @@ public:
     operator NSString *() const;
 #endif
 
+private:
+    // For debugging only, leaks memory.
+    const char *ascii() const;
+
 protected:
     DOMStringImpl *impl;
 };
index c465a45f4025f69ef26c03aa3be780b7d9e1f80a..3c1e0c01d62043eef02cbded4875c373b661127f 100644 (file)
@@ -485,4 +485,18 @@ unsigned DOMStringImpl::computeHash(const char *s)
     return h;
 }
 
+const char *DOMStringImpl::ascii() const
+{
+    char *buffer = new char[l];
+    char *p = buffer;
+    for (unsigned i = 0; i != l; ++i) {
+        unsigned short c = s[i].unicode();
+        if (c >= 0x20 && c < 0x7F)
+            *p++ = c;
+        else
+            *p++ = '?';
+    }
+    return buffer;
+}
+
 } // namespace DOM
index ed9cfde8df0dec97094d235bf6043fa0cb54c508..28c07d307b96060ca79e23f7fe69fa9ade9f4970 100644 (file)
@@ -85,6 +85,9 @@ public:
 
     static DOMStringImpl* empty();
 
+    // For debugging only, leaks memory.
+    const char *ascii() const;
+
     unsigned int l;
     QChar *s;
     mutable unsigned _hash;