some != NULL checks

[wxWidgets.git] / src / common / encconv.cpp

diff --git a/src/common/encconv.cpp b/src/common/encconv.cpp
index 85acc2143a910b39bb6ef176393a4380377971b7..3a5ce7faba7cdc1f298f47ef5ce91cf875af8fc4 100644 (file)
--- a/src/common/encconv.cpp
+++ b/src/common/encconv.cpp
@@ -22,11 +22,15 @@
 
 #include <stdlib.h>
 
 
 #include <stdlib.h>
 

+// conversion tables, generated by scripts in $(WXWIN)/misc/unictabl:
+#ifdef __BORLANDC__
+#include "../common/unictabl.inc"
+#else

 #include "unictabl.inc" 
 #include "unictabl.inc" 

-// conversion tables, generated by scripts in $(WXWIN)/misc/unictabl
+#endif

 
 
 
 

-static wxUint16 *GetEncTable(wxFontEncoding enc)
+static wxUint16* LINKAGEMODE GetEncTable(wxFontEncoding enc)

 {
     for (int i = 0; encodings_list[i].table != NULL; i++)
     {
 {
     for (int i = 0; encodings_list[i].table != NULL; i++)
     {


@@ -43,13 +47,13 @@ typedef struct {
 
 
 
 
 
 

-static int CompareCharsetItems(const void *i1, const void *i2)
+static int LINKAGEMODE CompareCharsetItems(const void *i1, const void *i2)

 {
     return ( ((CharsetItem*)i1) -> u - ((CharsetItem*)i2) -> u );
 }
 
 
 {
     return ( ((CharsetItem*)i1) -> u - ((CharsetItem*)i2) -> u );
 }
 
 

-static CharsetItem* BuildReverseTable(wxUint16 *tbl)
+static CharsetItem* LINKAGEMODE BuildReverseTable(wxUint16 *tbl)

 {
     CharsetItem *rev = new CharsetItem[128];
     
 {
     CharsetItem *rev = new CharsetItem[128];
     


@@ -66,7 +70,7 @@ static CharsetItem* BuildReverseTable(wxUint16 *tbl)
 wxEncodingConverter::wxEncodingConverter()
 {
     m_Table = NULL;
 wxEncodingConverter::wxEncodingConverter()
 {
     m_Table = NULL;

-    m_UnicodeInput = FALSE;
+    m_UnicodeInput = m_UnicodeOutput = FALSE;

     m_JustCopy = FALSE;
 }
 
     m_JustCopy = FALSE;
 }
 


@@ -85,6 +89,7 @@ bool wxEncodingConverter::Init(wxFontEncoding input_enc, wxFontEncoding output_e
 
     if (input_enc == output_enc) {m_JustCopy = TRUE; return TRUE;}
     
 
     if (input_enc == output_enc) {m_JustCopy = TRUE; return TRUE;}
     

+    m_UnicodeOutput = (output_enc == wxFONTENCODING_UNICODE);

     m_JustCopy = FALSE;
     
     if (input_enc == wxFONTENCODING_UNICODE)
     m_JustCopy = FALSE;
     
     if (input_enc == wxFONTENCODING_UNICODE)


@@ -150,6 +155,7 @@ bool wxEncodingConverter::Init(wxFontEncoding input_enc, wxFontEncoding output_e
 }
 
 
 }
 
 

+

 void wxEncodingConverter::Convert(const wxChar* input, wxChar* output)
 {
     if (m_JustCopy)
 void wxEncodingConverter::Convert(const wxChar* input, wxChar* output)
 {
     if (m_JustCopy)


@@ -173,6 +179,83 @@ void wxEncodingConverter::Convert(const wxChar* input, wxChar* output)
 }
 
 
 }
 
 

+#if wxUSE_UNICODE // otherwise wxChar === char
+
+void wxEncodingConverter::Convert(const char* input, wxChar* output)
+{
+    wxASSERT_MSG(!m_UnicodeInput, wxT("You cannot convert from unicode if input is const char*!"));
+
+    const char *i;
+    wxChar *o;
+
+    if (m_JustCopy)
+    {
+        for (i = input, o = output; *i != 0;)
+            *(o++) = (wxChar)(*(i++));
+        *o = 0;
+        return;
+    }
+    
+    wxASSERT_MSG(m_Table != NULL, wxT("You must call wxEncodingConverter::Init() before actually converting!"));
+       
+    for (i = input, o = output; *i != 0;)
+        *(o++) = (wxChar)(m_Table[(wxUint8)*(i++)]);
+    *o = 0;
+}
+
+
+
+void wxEncodingConverter::Convert(const wxChar* input, char* output)
+{
+    wxASSERT_MSG(!m_UnicodeOutput, wxT("You cannot convert to unicode if output is const char*!"));
+
+    const wxChar *i;
+    char *o;
+
+    if (m_JustCopy)
+    {
+        for (i = input, o = output; *i != 0;)
+            *(o++) = (char)(*(i++));
+        *o = 0;
+        return;
+    }
+    
+    wxASSERT_MSG(m_Table != NULL, wxT("You must call wxEncodingConverter::Init() before actually converting!"));
+       
+    if (m_UnicodeInput)
+        for (i = input, o = output; *i != 0; i++, o++)
+            *o = (char)(m_Table[(wxUint16)*i]);
+    else
+        for (i = input, o = output; *i != 0; i++, o++)
+            *o = (char)(m_Table[(wxUint8)*i]);
+    *o = 0;
+}
+
+
+
+void wxEncodingConverter::Convert(const char* input, char* output)
+{
+    wxASSERT_MSG(!m_UnicodeOutput, wxT("You cannot convert to unicode if output is const char*!"));
+    wxASSERT_MSG(!m_UnicodeInput, wxT("You cannot convert from unicode if input is const char*!"));
+
+    const char *i;
+    char *o;
+
+    if (m_JustCopy)
+    {
+        strcpy(output, input);
+        return;
+    }
+    
+    wxASSERT_MSG(m_Table != NULL, wxT("You must call wxEncodingConverter::Init() before actually converting!"));
+       
+    for (i = input, o = output; *i != 0;)
+        *(o++) = (char)(m_Table[(wxUint8)*(i++)]);
+    *o = 0;
+}
+
+#endif // wxUSE_UNICODE
+

 
 wxString wxEncodingConverter::Convert(const wxString& input)
 {
 
 wxString wxEncodingConverter::Convert(const wxString& input)
 {


@@ -195,20 +278,25 @@ wxString wxEncodingConverter::Convert(const wxString& input)
 
 
 
 
 
 

+
+
+

 // Following tables describe classes of encoding equivalence.
 // 
 
 #define STOP wxFONTENCODING_SYSTEM
 
 #define NUM_OF_PLATFORMS  4 /*must conform to enum wxPLATFORM_XXXX !!!*/
 // Following tables describe classes of encoding equivalence.
 // 
 
 #define STOP wxFONTENCODING_SYSTEM
 
 #define NUM_OF_PLATFORMS  4 /*must conform to enum wxPLATFORM_XXXX !!!*/

-#define ENC_PER_PLATFORM  3
+#define ENC_PER_PLATFORM  5

            // max no. of encodings for one language used on one platform
            // max no. of encodings for one language used on one platform

-           // Anybody thinks 3 is not enough? ;-)
+           // Anybody thinks 5 is not enough? ;-)

 
 static wxFontEncoding 
     EquivalentEncodings[][NUM_OF_PLATFORMS][ENC_PER_PLATFORM+1] = {
 
 
 static wxFontEncoding 
     EquivalentEncodings[][NUM_OF_PLATFORMS][ENC_PER_PLATFORM+1] = {
 

-    // West European (Latin1)
+    // *** Please put more common encodings as first! ***
+
+    // West European

     {
         /* unix    */ {wxFONTENCODING_ISO8859_1, wxFONTENCODING_ISO8859_15, STOP},
         /* windows */ {wxFONTENCODING_CP1252, STOP},
     {
         /* unix    */ {wxFONTENCODING_ISO8859_1, wxFONTENCODING_ISO8859_15, STOP},
         /* windows */ {wxFONTENCODING_CP1252, STOP},


@@ -216,13 +304,70 @@ static wxFontEncoding
         /* mac     */ {STOP}   
     },
 
         /* mac     */ {STOP}   
     },
 

-    // Central European (Latin2)
+    // Central European

     {
         /* unix    */ {wxFONTENCODING_ISO8859_2, STOP},
         /* windows */ {wxFONTENCODING_CP1250, STOP},
         /* os2     */ {STOP},
         /* mac     */ {STOP}   
     },
     {
         /* unix    */ {wxFONTENCODING_ISO8859_2, STOP},
         /* windows */ {wxFONTENCODING_CP1250, STOP},
         /* os2     */ {STOP},
         /* mac     */ {STOP}   
     },

+    
+    // Baltic
+    {
+        /* unix    */ {wxFONTENCODING_ISO8859_13, STOP},
+        /* windows */ {wxFONTENCODING_CP1257, STOP},
+        /* os2     */ {STOP},
+        /* mac     */ {STOP}   
+    },
+
+    // Hebrew
+    {
+        /* unix    */ {wxFONTENCODING_ISO8859_8, STOP},
+        /* windows */ {wxFONTENCODING_CP1255, STOP},
+        /* os2     */ {STOP},
+        /* mac     */ {STOP}   
+    },
+
+    // Greek
+    {
+        /* unix    */ {wxFONTENCODING_ISO8859_7, STOP},
+        /* windows */ {wxFONTENCODING_CP1253, STOP},
+        /* os2     */ {STOP},
+        /* mac     */ {STOP}   
+    },
+
+    // Arabic
+    {
+        /* unix    */ {wxFONTENCODING_ISO8859_6, STOP},
+        /* windows */ {wxFONTENCODING_CP1256, STOP},
+        /* os2     */ {STOP},
+        /* mac     */ {STOP}   
+    },
+
+    // Turkish
+    {
+        /* unix    */ {wxFONTENCODING_ISO8859_9, STOP},
+        /* windows */ {wxFONTENCODING_CP1254, STOP},
+        /* os2     */ {STOP},
+        /* mac     */ {STOP}   
+    },
+
+    // Cyrillic
+    {
+        /* unix    */ {wxFONTENCODING_ISO8859_13, wxFONTENCODING_ISO8859_4,
+                       wxFONTENCODING_ISO8859_15, wxFONTENCODING_ISO8859_1, STOP},
+        /* windows */ {wxFONTENCODING_CP1257, wxFONTENCODING_CP1252, STOP},
+        /* os2     */ {STOP},
+        /* mac     */ {STOP}   
+    },
+
+    // Russia and other KOI-8 users:
+    {
+        /* unix    */ {wxFONTENCODING_KOI8, wxFONTENCODING_ISO8859_5, STOP},
+        /* windows */ {wxFONTENCODING_CP1251, STOP},
+        /* os2     */ {STOP},
+        /* mac     */ {STOP}   
+    },

 
     {{STOP},{STOP},{STOP},{STOP}} /* Terminator */
     /* no, _not_ Arnold! */
 
     {{STOP},{STOP},{STOP},{STOP}} /* Terminator */
     /* no, _not_ Arnold! */


@@ -257,7 +402,9 @@ wxFontEncodingArray wxEncodingConverter::GetPlatformEquivalents(wxFontEncoding e
             for (e = 0; EquivalentEncodings[clas][i][e] != STOP; e++)
                 if (EquivalentEncodings[clas][i][e] == enc)
                 {
             for (e = 0; EquivalentEncodings[clas][i][e] != STOP; e++)
                 if (EquivalentEncodings[clas][i][e] == enc)
                 {

-                    for (f = EquivalentEncodings[clas][platform]; *f != STOP; f++) 
+                    for (f = EquivalentEncodings[clas][platform]; *f != STOP; f++)
+                        if (*f == enc) arr.Add(enc);
+                    for (f = EquivalentEncodings[clas][platform]; *f != STOP; f++)

                         if (arr.Index(*f) == wxNOT_FOUND) arr.Add(*f);
                     i = NUM_OF_PLATFORMS/*hack*/; break; 
                 }
                         if (arr.Index(*f) == wxNOT_FOUND) arr.Add(*f);
                     i = NUM_OF_PLATFORMS/*hack*/; break; 
                 }


@@ -274,6 +421,8 @@ wxFontEncodingArray wxEncodingConverter::GetAllEquivalents(wxFontEncoding enc)
     int i, clas, e, j ;
     wxFontEncoding *f;
     wxFontEncodingArray arr;
     int i, clas, e, j ;
     wxFontEncoding *f;
     wxFontEncodingArray arr;

+    
+    arr = GetPlatformEquivalents(enc); // we want them to be first items in array

 
     clas = 0;
     while (EquivalentEncodings[clas][0][0] != STOP)
 
     clas = 0;
     while (EquivalentEncodings[clas][0][0] != STOP)