fixed memory leak in wxXmlResource introduced when fixing wxVector<> usage (patch...

[wxWidgets.git] / include / wx / stringops.h

diff --git a/include/wx/stringops.h b/include/wx/stringops.h
index f3046794beece784609001df31127e394d6639be..50aa1376ac30215f5c204395122c5b1d440e0416 100644 (file)
--- a/include/wx/stringops.h
+++ b/include/wx/stringops.h
@@ -36,17 +36,17 @@ struct WXDLLIMPEXP_BASE wxStringOperationsWchar
     static void DecIter(wxStringImpl::const_iterator& i) { --i; }
 
     // moves the iterator by n Unicode characters
     static void DecIter(wxStringImpl::const_iterator& i) { --i; }
 
     // moves the iterator by n Unicode characters

-    static wxStringImpl::iterator AddToIter(const wxStringImpl::iterator& i, int n)
+    static wxStringImpl::iterator AddToIter(const wxStringImpl::iterator& i, ptrdiff_t n)

         { return i + n; }
         { return i + n; }

-    static wxStringImpl::const_iterator AddToIter(const wxStringImpl::const_iterator& i, int n)
+    static wxStringImpl::const_iterator AddToIter(const wxStringImpl::const_iterator& i, ptrdiff_t n)

         { return i + n; }
 
     // returns distance of the two iterators in Unicode characters
         { return i + n; }
 
     // returns distance of the two iterators in Unicode characters

-    static int DiffIters(const wxStringImpl::iterator& i1,
-                         const wxStringImpl::iterator& i2)
+    static ptrdiff_t DiffIters(const wxStringImpl::iterator& i1,
+                               const wxStringImpl::iterator& i2)

         { return i1 - i2; }
         { return i1 - i2; }

-    static int DiffIters(const wxStringImpl::const_iterator& i1,
-                         const wxStringImpl::const_iterator& i2)
+    static ptrdiff_t DiffIters(const wxStringImpl::const_iterator& i1,
+                               const wxStringImpl::const_iterator& i2)

         { return i1 - i2; }
 
     // encodes the character to a form used to represent it in internal
         { return i1 - i2; }
 
     // encodes the character to a form used to represent it in internal


@@ -70,7 +70,7 @@ struct WXDLLIMPEXP_BASE wxStringOperationsUtf8
 #endif
 
     // table of offsets to skip forward when iterating over UTF-8 sequence
 #endif
 
     // table of offsets to skip forward when iterating over UTF-8 sequence

-    static unsigned char ms_utf8IterTable[256];
+    static const unsigned char ms_utf8IterTable[256];

 
 
     template<typename Iterator>
 
 
     template<typename Iterator>


@@ -97,18 +97,18 @@ struct WXDLLIMPEXP_BASE wxStringOperationsUtf8
     }
 
     template<typename Iterator>
     }
 
     template<typename Iterator>

-    static Iterator AddToIter(const Iterator& i, int n)
+    static Iterator AddToIter(const Iterator& i, ptrdiff_t n)

     {
         Iterator out(i);
 
         if ( n > 0 )
         {
     {
         Iterator out(i);
 
         if ( n > 0 )
         {

-            for ( int j = 0; j < n; ++j )
+            for ( ptrdiff_t j = 0; j < n; ++j )

                 IncIter(out);
         }
         else if ( n < 0 )
         {
                 IncIter(out);
         }
         else if ( n < 0 )
         {

-            for ( int j = 0; j > n; --j )
+            for ( ptrdiff_t j = 0; j > n; --j )

                 DecIter(out);
         }
 
                 DecIter(out);
         }
 


@@ -116,9 +116,9 @@ struct WXDLLIMPEXP_BASE wxStringOperationsUtf8
     }
 
     template<typename Iterator>
     }
 
     template<typename Iterator>

-    static int DiffIters(Iterator i1, Iterator i2)
+    static ptrdiff_t DiffIters(Iterator i1, Iterator i2)

     {
     {

-        int dist = 0;
+        ptrdiff_t dist = 0;

 
         if ( i1 < i2 )
         {
 
         if ( i1 < i2 )
         {


@@ -140,15 +140,10 @@ struct WXDLLIMPEXP_BASE wxStringOperationsUtf8
         return dist;
     }
 
         return dist;
     }
 

-    // buffer for single UTF-8 character
-    struct Utf8CharBuffer
-    {
-        char data[5];
-        operator const char*() const { return data; }
-    };
-

     // encodes the character as UTF-8:
     // encodes the character as UTF-8:

-    static Utf8CharBuffer EncodeChar(const wxUniChar& ch);
+    typedef wxUniChar::Utf8CharBuffer Utf8CharBuffer;
+    static Utf8CharBuffer EncodeChar(const wxUniChar& ch)
+        { return ch.AsUTF8(); }

 
     // returns n copies of ch encoded in UTF-8 string
     static wxCharBuffer EncodeNChars(size_t n, const wxUniChar& ch);
 
     // returns n copies of ch encoded in UTF-8 string
     static wxCharBuffer EncodeNChars(size_t n, const wxUniChar& ch);


@@ -161,7 +156,15 @@ struct WXDLLIMPEXP_BASE wxStringOperationsUtf8
     }
 
     // decodes single UTF-8 character from UTF-8 string
     }
 
     // decodes single UTF-8 character from UTF-8 string

-    static wxUniChar DecodeChar(wxStringImpl::const_iterator i);
+    static wxUniChar DecodeChar(wxStringImpl::const_iterator i)
+    {
+        if ( (unsigned char)*i < 0x80 )
+            return (int)*i;
+        return DecodeNonAsciiChar(i);
+    }
+
+private:
+    static wxUniChar DecodeNonAsciiChar(wxStringImpl::const_iterator i);

 };
 #endif // wxUSE_UNICODE_UTF8
 
 };
 #endif // wxUSE_UNICODE_UTF8