]> git.saurik.com Git - wxWidgets.git/blobdiff - src/common/wxchar.cpp
Fixed MinGW/ANSI (and probably any other C++ compiler, excluding, of course,
[wxWidgets.git] / src / common / wxchar.cpp
index 7e3c343b5e6dfd4bb6b3c19506349c1ec5648171..f36473d75c57e0607af80e4e23f51daacbaadfd0 100644 (file)
@@ -98,8 +98,8 @@ bool WXDLLEXPORT wxOKlibc()
     // this is UTF-8 allright, check whether that's what we want
     char *cur_locale = setlocale(LC_CTYPE, NULL);
     if ((strlen(cur_locale) < 4) ||
-       (strcasecmp(cur_locale + strlen(cur_locale) - 4, "utf8")) ||
-       (strcasecmp(cur_locale + strlen(cur_locale) - 5, "utf-8"))) {
+            (strcasecmp(cur_locale + strlen(cur_locale) - 4, "utf8")) ||
+            (strcasecmp(cur_locale + strlen(cur_locale) - 5, "utf-8"))) {
       // nope, don't use libc conversion
       return FALSE;
     }
@@ -117,6 +117,756 @@ size_t   WXDLLEXPORT wcslen(const wchar_t *s)
 }
 #endif
 
+// ============================================================================
+// printf() functions business
+// ============================================================================
+
+// ----------------------------------------------------------------------------
+// implement [v]snprintf() if the system doesn't provide a safe one
+// ----------------------------------------------------------------------------
+
+#if !defined(wxVsnprintf_)
+int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax,
+                             const wxChar *format, va_list argptr)
+{
+    // buffer to avoid dynamic memory allocation each time for small strings
+    char szScratch[1024];
+
+    // number of characters in the buffer so far, must be less than lenMax
+    size_t lenCur = 0;
+
+    for (size_t n = 0; format[n]; n++)
+    {
+        if (format[n] == wxT('%')) {
+            static char s_szFlags[256] = "%";
+            size_t flagofs = 1;
+            bool adj_left = FALSE,
+                 in_prec = FALSE,
+                 prec_dot = FALSE,
+                 done = FALSE;
+            int ilen = 0;
+            size_t min_width = 0,
+                   max_width = wxSTRING_MAXLEN;
+            do {
+
+#define CHECK_PREC \
+                if (in_prec && !prec_dot) \
+                { \
+                    s_szFlags[flagofs++] = '.'; \
+                    prec_dot = TRUE; \
+                }
+
+#define APPEND_CH(ch) \
+                if ( lenCur == lenMax ) \
+                    return -1; \
+                \
+                buf[lenCur++] = ch
+
+#define APPEND_STR(s) \
+                { \
+                    for ( const wxChar *p = s; *p; p++ ) \
+                    { \
+                        APPEND_CH(*p); \
+                    } \
+                }
+
+                switch (format[++n]) {
+                    case wxT('\0'):
+                        APPEND_CH(_T('\0'));
+
+                        done = TRUE;
+                        break;
+
+                    case wxT('%'):
+                        APPEND_CH(_T('%'));
+                        done = TRUE;
+                        break;
+
+                    case wxT('#'):
+                    case wxT('0'):
+                    case wxT(' '):
+                    case wxT('+'):
+                    case wxT('\''):
+                        CHECK_PREC
+                        s_szFlags[flagofs++] = format[n];
+                        break;
+
+                    case wxT('-'):
+                        CHECK_PREC
+                        adj_left = TRUE;
+                        s_szFlags[flagofs++] = format[n];
+                        break;
+
+                    case wxT('.'):
+                        CHECK_PREC
+                            in_prec = TRUE;
+                        prec_dot = FALSE;
+                        max_width = 0;
+                        // dot will be auto-added to s_szFlags if non-negative number follows
+                        break;
+
+                    case wxT('h'):
+                        ilen = -1;
+                        CHECK_PREC
+                        s_szFlags[flagofs++] = format[n];
+                        break;
+
+                    case wxT('l'):
+                        ilen = 1;
+                        CHECK_PREC
+                        s_szFlags[flagofs++] = format[n];
+                        break;
+
+                    case wxT('q'):
+                    case wxT('L'):
+                        ilen = 2;
+                        CHECK_PREC
+                        s_szFlags[flagofs++] = format[n];
+                        break;
+
+                    case wxT('Z'):
+                        ilen = 3;
+                        CHECK_PREC
+                        s_szFlags[flagofs++] = format[n];
+                        break;
+
+                    case wxT('*'):
+                        {
+                            int len = va_arg(argptr, int);
+                            if (in_prec) {
+                                if (len<0) break;
+                                CHECK_PREC
+                                    max_width = len;
+                            } else {
+                                if (len<0) {
+                                    adj_left = !adj_left;
+                                    s_szFlags[flagofs++] = '-';
+                                    len = -len;
+                                }
+                                min_width = len;
+                            }
+                            flagofs += ::sprintf(s_szFlags+flagofs,"%d",len);
+                        }
+                        break;
+
+                    case wxT('1'): case wxT('2'): case wxT('3'):
+                    case wxT('4'): case wxT('5'): case wxT('6'):
+                    case wxT('7'): case wxT('8'): case wxT('9'):
+                        {
+                            int len = 0;
+                            CHECK_PREC
+                                while ((format[n]>=wxT('0')) && (format[n]<=wxT('9'))) {
+                                    s_szFlags[flagofs++] = format[n];
+                                    len = len*10 + (format[n] - wxT('0'));
+                                    n++;
+                                }
+                            if (in_prec) max_width = len;
+                            else min_width = len;
+                            n--; // the main loop pre-increments n again
+                        }
+                        break;
+
+                    case wxT('d'):
+                    case wxT('i'):
+                    case wxT('o'):
+                    case wxT('u'):
+                    case wxT('x'):
+                    case wxT('X'):
+                        CHECK_PREC
+                        s_szFlags[flagofs++] = format[n];
+                        s_szFlags[flagofs] = '\0';
+                        if (ilen == 0 ) {
+                            int val = va_arg(argptr, int);
+                            ::sprintf(szScratch, s_szFlags, val);
+                        }
+                        else if (ilen == -1) {
+                            // NB: 'short int' value passed through '...'
+                            //      is promoted to 'int'
+                            short int val = (short int) va_arg(argptr, int);
+                            ::sprintf(szScratch, s_szFlags, val);
+                        }
+                        else if (ilen == 1) {
+                            long int val = va_arg(argptr, long int);
+                            ::sprintf(szScratch, s_szFlags, val);
+                        }
+                        else if (ilen == 2) {
+#if SIZEOF_LONG_LONG
+                            long long int val = va_arg(argptr, long long int);
+                            ::sprintf(szScratch, s_szFlags, val);
+#else
+                            long int val = va_arg(argptr, long int);
+                            ::sprintf(szScratch, s_szFlags, val);
+#endif
+                        }
+                        else if (ilen == 3) {
+                            size_t val = va_arg(argptr, size_t);
+                            ::sprintf(szScratch, s_szFlags, val);
+                        }
+                        {
+                            const wxMB2WXbuf tmp =
+                                wxConvLibc.cMB2WX(szScratch);
+                            APPEND_STR(tmp);
+                        }
+
+                        done = TRUE;
+                        break;
+
+                    case wxT('e'):
+                    case wxT('E'):
+                    case wxT('f'):
+                    case wxT('g'):
+                    case wxT('G'):
+                        CHECK_PREC
+                        s_szFlags[flagofs++] = format[n];
+                        s_szFlags[flagofs] = '\0';
+                        if (ilen == 2) {
+                            long double val = va_arg(argptr, long double);
+                            ::sprintf(szScratch, s_szFlags, val);
+                        } else {
+                            double val = va_arg(argptr, double);
+                            ::sprintf(szScratch, s_szFlags, val);
+                        }
+                        {
+                            const wxMB2WXbuf tmp =
+                                wxConvLibc.cMB2WX(szScratch);
+                            APPEND_STR(tmp);
+                        }
+
+                        done = TRUE;
+                        break;
+
+                    case wxT('p'):
+                        {
+                            void *val = va_arg(argptr, void *);
+                            CHECK_PREC
+                                s_szFlags[flagofs++] = format[n];
+                            s_szFlags[flagofs] = '\0';
+                            ::sprintf(szScratch, s_szFlags, val);
+
+                            const wxMB2WXbuf tmp =
+                                wxConvLibc.cMB2WX(szScratch);
+                            APPEND_STR(tmp);
+
+                            done = TRUE;
+                        }
+                        break;
+
+                    case wxT('c'):
+                        {
+                            wxChar val = va_arg(argptr, int);
+                            // we don't need to honor padding here, do we?
+                            APPEND_CH(val);
+
+                            done = TRUE;
+                        }
+                        break;
+
+                    case wxT('s'):
+                        if (ilen == -1) {
+                            // wx extension: we'll let %hs mean non-Unicode strings
+                            char *val = va_arg(argptr, char *);
+#if wxUSE_UNICODE
+                            // ASCII->Unicode constructor handles max_width right
+                            wxString s(val, wxConvLibc, max_width);
+#else
+                            size_t len = wxSTRING_MAXLEN;
+                            if (val) {
+                                for (len = 0; val[len] && (len<max_width); len++);
+                            } else val = wxT("(null)");
+                            wxString s(val, len);
+#endif
+                            if (s.Len() < min_width)
+                                s.Pad(min_width - s.Len(), wxT(' '), adj_left);
+
+                            APPEND_STR(s);
+                        } else {
+                            wxChar *val = va_arg(argptr, wxChar *);
+                            size_t len = wxSTRING_MAXLEN;
+                            if (val) {
+                                for (len = 0; val[len] && (len<max_width); len++);
+                            } else val = wxT("(null)");
+                            wxString s(val, len);
+                            if (s.Len() < min_width)
+                                s.Pad(min_width - s.Len(), wxT(' '), adj_left);
+
+                            APPEND_STR(s);
+                        }
+                        done = TRUE;
+                        break;
+
+                    case wxT('n'):
+                        if (ilen == 0) {
+                            int *val = va_arg(argptr, int *);
+                            *val = lenCur;
+                        }
+                        else if (ilen == -1) {
+                            short int *val = va_arg(argptr, short int *);
+                            *val = lenCur;
+                        }
+                        else if (ilen >= 1) {
+                            long int *val = va_arg(argptr, long int *);
+                            *val = lenCur;
+                        }
+                        done = TRUE;
+                        break;
+
+                    default:
+                        if (wxIsalpha(format[n]))
+                            // probably some flag not taken care of here yet
+                            s_szFlags[flagofs++] = format[n];
+                        else {
+                            // bad format
+                            APPEND_CH(_T('%')); // just to pass the glibc tst-printf.c
+                            n--;
+                            done = TRUE;
+                        }
+                        break;
+                }
+            } while (!done);
+        }
+        else
+        {
+            APPEND_CH(format[n]);
+        }
+    }
+
+    return lenCur;
+}
+
+#undef APPEND_CH
+#undef APPEND_STR
+#undef CHECK_PREC
+
+#endif // !wxVsnprintfA
+
+#if !defined(wxSnprintf_)
+int WXDLLEXPORT wxSnprintf_(wxChar *buf, size_t len, const wxChar *format, ...)
+{
+    va_list argptr;
+    va_start(argptr, format);
+
+    int iLen = wxVsnprintf_(buf, len, format, argptr);
+
+    va_end(argptr);
+
+    return iLen;
+}
+#endif // wxSnprintf_
+
+// ----------------------------------------------------------------------------
+// implement the standard IO functions for wide char if libc doesn't have them
+// ----------------------------------------------------------------------------
+
+#ifdef wxNEED_FPUTWC
+
+int wxFputs(const wchar_t *ws, FILE *stream)
+{
+    // counting the number of wide characters written isn't worth the trouble,
+    // simply distinguish between ok and error
+    return fputs(wxConvLibc.cWC2MB(ws), stream) == -1 ? -1 : 0;
+}
+
+int /* not wint_t */ wxPutc(wchar_t wc, FILE *stream)
+{
+    wchar_t ws[2] = { wc, L'\0' };
+
+    return wxFputs(ws, stream);
+}
+
+#endif // wxNEED_FPUTWC
+
+// NB: we only implement va_list functions here, the ones taking ... are
+//     defined below for wxNEED_PRINTF_CONVERSION case anyhow and we reuse
+//     the definitions there to avoid duplicating them here
+#ifdef wxNEED_WPRINTF
+
+// TODO: implement the scanf() functions
+int vwscanf(const wchar_t *format, va_list argptr)
+{
+    wxFAIL_MSG( _T("TODO") );
+
+    return -1;
+}
+
+int vswscanf(const wchar_t *ws, const wchar_t *format, va_list argptr)
+{
+    wxFAIL_MSG( _T("TODO") );
+
+    return -1;
+}
+
+int vfwscanf(FILE *stream, const wchar_t *format, va_list argptr)
+{
+    wxFAIL_MSG( _T("TODO") );
+
+    return -1;
+}
+
+#define vswprintf wxVsnprintf_
+
+int vfwprintf(FILE *stream, const wchar_t *format, va_list argptr)
+{
+    wxString s;
+    int rc = s.PrintfV(format, argptr);
+
+    if ( rc != -1 )
+    {
+        // we can't do much better without Unicode support in libc...
+        if ( fprintf(stream, s.mb_str()) == -1 )
+            return -1;
+    }
+
+    return rc;
+}
+
+int vwprintf(const wchar_t *format, va_list argptr)
+{
+    return wxVfprintf(stdout, format, argptr);
+}
+
+#endif // wxNEED_WPRINTF
+
+#ifdef wxNEED_PRINTF_CONVERSION
+
+// ----------------------------------------------------------------------------
+// wxFormatConverter: class doing the "%s" -> "%ls" conversion
+// ----------------------------------------------------------------------------
+
+/*
+   Here are the gory details. We want to follow the Windows/MS conventions,
+   that is to have
+
+   In ANSI mode:
+
+   format specifier         results in
+   -----------------------------------
+   %c, %hc, %hC             char
+   %lc, %C, %lC             wchar_t
+
+   In Unicode mode:
+
+   format specifier         results in
+   -----------------------------------
+   %hc, %C, %hC             char
+   %c, %lc, %lC             wchar_t
+
+
+   while on POSIX systems we have %C identical to %lc and %c always means char
+   (in any mode) while %lc always means wchar_t,
+
+   So to use native functions in order to get our semantics we must do the
+   following translations in Unicode mode (nothing to do in ANSI mode):
+
+   wxWindows specifier      POSIX specifier
+   ----------------------------------------
+
+   %hc, %C, %hC             %c
+   %c                       %lc
+
+
+   And, of course, the same should be done for %s as well.
+*/
+
+class wxFormatConverter
+{
+public:
+    wxFormatConverter(const wxChar *format);
+
+    // notice that we only translated the string if m_fmtOrig == NULL (as set
+    // by CopyAllBefore()), otherwise we should simply use the original format
+    operator const wxChar *() const
+        { return m_fmtOrig ? m_fmtOrig : m_fmt.c_str(); }
+
+private:
+    // copy another character to the translated format: this function does the
+    // copy if we are translating but doesn't do anything at all if we don't,
+    // so we don't create the translated format string at all unless we really
+    // need to (i.e. InsertFmtChar() is called)
+    wxChar CopyFmtChar(wxChar ch)
+    {
+        if ( !m_fmtOrig )
+        {
+            // we're translating, do copy
+            m_fmt += ch;
+        }
+        else
+        {
+            // simply increase the count which should be copied by
+            // CopyAllBefore() later if needed
+            m_nCopied++;
+        }
+
+        return ch;
+    }
+
+    // insert an extra character
+    void InsertFmtChar(wxChar ch)
+    {
+        if ( m_fmtOrig )
+        {
+            // so far we haven't translated anything yet
+            CopyAllBefore();
+        }
+
+        m_fmt += ch;
+    }
+
+    void CopyAllBefore()
+    {
+        wxASSERT_MSG( m_fmtOrig && m_fmt.empty(), _T("logic error") );
+
+        m_fmt = wxString(m_fmtOrig, m_nCopied);
+
+        // we won't need it any longer
+        m_fmtOrig = NULL;
+    }
+
+    static bool IsFlagChar(wxChar ch)
+    {
+        return ch == _T('-') || ch == _T('+') ||
+               ch == _T('0') || ch == _T(' ') || ch == _T('#');
+    }
+
+    void SkipDigits(const wxChar **ppc)
+    {
+        while ( **ppc >= _T('0') && **ppc <= _T('9') )
+            CopyFmtChar(*(*ppc)++);
+    }
+
+    // the translated format
+    wxString m_fmt;
+
+    // the original format
+    const wxChar *m_fmtOrig;
+
+    // the number of characters already copied
+    size_t m_nCopied;
+};
+
+wxFormatConverter::wxFormatConverter(const wxChar *format)
+{
+    m_fmtOrig = format;
+    m_nCopied = 0;
+
+    while ( *format )
+    {
+        if ( CopyFmtChar(*format++) == _T('%') )
+        {
+            // skip any flags
+            while ( IsFlagChar(*format) )
+                CopyFmtChar(*format++);
+
+            // and possible width
+            if ( *format == _T('*') )
+                CopyFmtChar(*format++);
+            else
+                SkipDigits(&format);
+
+            // precision?
+            if ( *format == _T('.') )
+            {
+                SkipDigits(&format);
+            }
+
+            // next we can have a size modifier
+            enum
+            {
+                Default,
+                Short,
+                Long
+            } size;
+
+            switch ( *format )
+            {
+                case _T('h'):
+                    size = Short;
+                    format++;
+                    break;
+
+                case _T('l'):
+                    // "ll" has a different meaning!
+                    if ( format[1] != _T('l') )
+                    {
+                        size = Long;
+                        format++;
+                        break;
+                    }
+                    //else: fall through
+
+                default:
+                    size = Default;
+            }
+
+            // and finally we should have the type
+            switch ( *format )
+            {
+                case _T('C'):
+                case _T('S'):
+                    // %C and %hC -> %c and %lC -> %lc
+                    if ( size == Long )
+                        CopyFmtChar(_T('l'));
+
+                    InsertFmtChar(*format++ == _T('C') ? _T('c') : _T('s'));
+                    break;
+
+                case _T('c'):
+                case _T('s'):
+                    // %c -> %lc but %hc stays %hc and %lc is still %lc
+                    switch ( size )
+                    {
+                        case Default:
+                            InsertFmtChar(_T('l'));
+                            break;
+
+                        case Short:
+                            CopyFmtChar(_T('h'));
+                            break;
+
+                        case Long:
+                            ;
+                    }
+                    // fall through
+
+                default:
+                    // nothing special to do
+                    CopyFmtChar(*format++);
+            }
+        }
+    }
+}
+
+#else // !wxNEED_PRINTF_CONVERSION
+    // no conversion necessary
+    #define wxFormatConverter(x) (x)
+#endif // wxNEED_PRINTF_CONVERSION/!wxNEED_PRINTF_CONVERSION
+
+// ----------------------------------------------------------------------------
+// wxPrintf(), wxScanf() and relatives
+// ----------------------------------------------------------------------------
+
+#if defined(wxNEED_PRINTF_CONVERSION) || defined(wxNEED_WPRINTF)
+
+int wxScanf( const wxChar *format, ... ) ATTRIBUTE_PRINTF_2
+{
+    va_list argptr;
+    va_start(argptr, format);
+
+    int ret = vwscanf(wxFormatConverter(format), argptr );
+
+    va_end(argptr);
+
+    return ret;
+}
+
+int wxSscanf( const wxChar *str, const wxChar *format, ... ) ATTRIBUTE_PRINTF_3
+{
+    va_list argptr;
+    va_start(argptr, format);
+
+    int ret = vswscanf( str, wxFormatConverter(format), argptr );
+
+    va_end(argptr);
+
+    return ret;
+}
+
+int wxFscanf( FILE *stream, const wxChar *format, ... ) ATTRIBUTE_PRINTF_3
+{
+    va_list argptr;
+    va_start(argptr, format);
+
+    int ret = vfwscanf(stream, wxFormatConverter(format), argptr);
+
+    va_end(argptr);
+
+    return ret;
+}
+
+int wxPrintf( const wxChar *format, ... ) ATTRIBUTE_PRINTF_2
+{
+    va_list argptr;
+    va_start(argptr, format);
+    
+    int ret = vwprintf( wxFormatConverter(format), argptr );
+
+    va_end(argptr);
+
+    return ret;
+}
+
+#ifndef wxSnprintf
+int wxSnprintf( wxChar *str, size_t size, const wxChar *format, ... ) ATTRIBUTE_PRINTF_4
+{
+    va_list argptr;
+    va_start(argptr, format);
+
+    int ret = vswprintf( str, size, wxFormatConverter(format), argptr );
+
+    va_end(argptr);
+
+    return ret;
+}
+#endif // wxSnprintf
+
+int wxSprintf( wxChar *str, const wxChar *format, ... ) ATTRIBUTE_PRINTF_3
+{
+    va_list argptr;
+    va_start(argptr, format);
+
+    // callers of wxSprintf() deserve what they get
+    int ret = vswprintf( str, UINT_MAX, wxFormatConverter(format), argptr );
+
+    va_end(argptr);
+
+    return ret;
+}
+
+int wxFprintf( FILE *stream, const wxChar *format, ... ) ATTRIBUTE_PRINTF_3
+{
+    va_list argptr;
+    va_start( argptr, format );
+
+    int ret = vfwprintf( stream, wxFormatConverter(format), argptr );
+
+    va_end(argptr);
+
+    return ret;
+}
+
+int wxVsscanf( const wxChar *str, const wxChar *format, va_list argptr )
+{
+    return vswscanf( str, wxFormatConverter(format), argptr );
+}
+
+int wxVfprintf( FILE *stream, const wxChar *format, va_list argptr )
+{
+    return vfwprintf( stream, wxFormatConverter(format), argptr );
+}
+
+int wxVprintf( const wxChar *format, va_list argptr )
+{
+    return vwprintf( wxFormatConverter(format), argptr );
+}
+
+#ifndef wxVsnprintf
+int wxVsnprintf( wxChar *str, size_t size, const wxChar *format, va_list argptr )
+{
+    return vswprintf( str, size, wxFormatConverter(format), argptr );
+}
+#endif // wxVsnprintf
+
+int wxVsprintf( wxChar *str, const wxChar *format, va_list argptr )
+{
+    // same as for wxSprintf()
+    return vswprintf(str, UINT_MAX, wxFormatConverter(format), argptr);
+}
+
+#endif // wxNEED_PRINTF_CONVERSION
+
+// ----------------------------------------------------------------------------
+// ctype.h stuff (currently unused)
+// ----------------------------------------------------------------------------
+
 #if defined(__WIN32__) && defined(wxNEED_WX_CTYPE_H)
 inline WORD wxMSW_ctype(wxChar ch)
 {
@@ -178,32 +928,49 @@ int WXDLLEXPORT wxStrnicmp(const wxChar *s1, const wxChar *s2, size_t n)
 #ifndef wxStrtok
 WXDLLEXPORT wxChar * wxStrtok(wxChar *psz, const wxChar *delim, wxChar **save_ptr)
 {
-  if (!psz) psz = *save_ptr;
-  psz += wxStrspn(psz, delim);
-  if (!*psz) {
-    *save_ptr = (wxChar *)NULL;
-    return (wxChar *)NULL;
-  }
-  wxChar *ret = psz;
-  psz = wxStrpbrk(psz, delim);
-  if (!psz) *save_ptr = (wxChar*)NULL;
-  else {
-    *psz = wxT('\0');
-    *save_ptr = psz + 1;
-  }
-  return ret;
+    if (!psz)
+    {
+        psz = *save_ptr;
+        if ( !psz )
+            return NULL;
+    }
+
+    psz += wxStrspn(psz, delim);
+    if (!*psz)
+    {
+        *save_ptr = (wxChar *)NULL;
+        return (wxChar *)NULL;
+    }
+
+    wxChar *ret = psz;
+    psz = wxStrpbrk(psz, delim);
+    if (!psz)
+    {
+        *save_ptr = (wxChar*)NULL;
+    }
+    else
+    {
+        *psz = wxT('\0');
+        *save_ptr = psz + 1;
+    }
+
+    return ret;
 }
-#endif
+#endif // wxStrtok
 
 #ifndef wxSetlocale
 WXDLLEXPORT wxWCharBuffer wxSetlocale(int category, const wxChar *locale)
 {
-    char *localeOld = setlocale(category, wxConvLibc.cWX2MB(locale));
+    char *localeOld = setlocale(category, wxConvLocal.cWX2MB(locale));
 
-    return wxWCharBuffer(wxConvLibc.cMB2WC(localeOld));
+    return wxWCharBuffer(wxConvLocal.cMB2WC(localeOld));
 }
 #endif
 
+// ----------------------------------------------------------------------------
+// string.h functions
+// ----------------------------------------------------------------------------
+
 #ifdef wxNEED_WX_STRING_H
 WXDLLEXPORT wxChar * wxStrcat(wxChar *dest, const wxChar *src)
 {
@@ -213,10 +980,16 @@ WXDLLEXPORT wxChar * wxStrcat(wxChar *dest, const wxChar *src)
   return ret;
 }
 
-WXDLLEXPORT wxChar * wxStrchr(const wxChar *s, wxChar c)
+WXDLLEXPORT const wxChar * wxStrchr(const wxChar *s, wxChar c)
 {
-  while (*s && *s != c) s++;
-  return (*s) ? (wxChar *)s : (wxChar *)NULL;
+    // be careful here as the terminating NUL makes part of the string
+    while ( *s != c )
+    {
+        if ( !*s++ )
+            return NULL;
+    }
+
+    return s;
 }
 
 WXDLLEXPORT int wxStrcmp(const wxChar *s1, const wxChar *s2)
@@ -260,20 +1033,26 @@ WXDLLEXPORT wxChar * wxStrncpy(wxChar *dest, const wxChar *src, size_t n)
   return ret;
 }
 
-WXDLLEXPORT wxChar * wxStrpbrk(const wxChar *s, const wxChar *accept)
+WXDLLEXPORT const wxChar * wxStrpbrk(const wxChar *s, const wxChar *accept)
 {
-  while (*s && !wxStrchr(accept, *s)) s++;
-  return (*s) ? (wxChar *)s : (wxChar *)NULL;
+  while (*s && !wxStrchr(accept, *s))
+      s++;
+
+  return *s ? s : NULL;
 }
 
-WXDLLEXPORT wxChar * wxStrrchr(const wxChar *s, wxChar c)
+WXDLLEXPORT const wxChar * wxStrrchr(const wxChar *s, wxChar c)
 {
-  wxChar *ret = (wxChar *)NULL;
-  while (*s) {
-    if (*s == c) ret = (wxChar *)s;
-    s++;
-  }
-  return ret;
+    const wxChar *ret = NULL;
+    do
+    {
+        if ( *s == c )
+            ret = s;
+        s++;
+    }
+    while ( *s );
+
+    return ret;
 }
 
 WXDLLEXPORT size_t wxStrspn(const wxChar *s, const wxChar *accept)
@@ -283,14 +1062,23 @@ WXDLLEXPORT size_t wxStrspn(const wxChar *s, const wxChar *accept)
   return len;
 }
 
-WXDLLEXPORT wxChar * wxStrstr(const wxChar *haystack, const wxChar *needle)
+WXDLLEXPORT const wxChar *wxStrstr(const wxChar *haystack, const wxChar *needle)
 {
-  wxChar *fnd;
-  while ((fnd = wxStrchr(haystack, *needle))) {
-    if (!wxStrcmp(fnd, needle)) return fnd;
-    haystack = fnd + 1;
-  }
-  return (wxChar *)NULL;
+    wxCHECK_RET( needle, NULL, _T("NULL argument in wxStrstr") );
+
+    // VZ: this is not exactly the most efficient string search algorithm...
+
+    const size_t len = wxStrlen(needle);
+
+    while ( const wxChar *fnd = wxStrchr(haystack, *needle) )
+    {
+        if ( !wxStrncmp(fnd, needle, len) )
+            return fnd;
+
+        haystack = fnd + 1;
+    }
+
+    return NULL;
 }
 
 WXDLLEXPORT double wxStrtod(const wxChar *nptr, wxChar **endptr)
@@ -312,7 +1100,7 @@ WXDLLEXPORT double wxStrtod(const wxChar *nptr, wxChar **endptr)
   }
 
   wxString data(nptr, nptr-start);
-  wxWX2MBbuf dat = data.mb_str(wxConvLibc);
+  wxWX2MBbuf dat = data.mb_str(wxConvLocal);
   char *rdat = wxMBSTRINGCAST dat;
   double ret = strtod(dat, &rdat);
 
@@ -340,7 +1128,7 @@ WXDLLEXPORT long int wxStrtol(const wxChar *nptr, wxChar **endptr, int base)
          (wxIsalpha(*nptr) && (wxToupper(*nptr) - wxT('A') + 10 < base))) nptr++;
 
   wxString data(nptr, nptr-start);
-  wxWX2MBbuf dat = data.mb_str(wxConvLibc);
+  wxWX2MBbuf dat = data.mb_str(wxConvLocal);
   char *rdat = wxMBSTRINGCAST dat;
   long int ret = strtol(dat, &rdat, base);
 
@@ -348,138 +1136,83 @@ WXDLLEXPORT long int wxStrtol(const wxChar *nptr, wxChar **endptr, int base)
 
   return ret;
 }
-#endif
+#endif // wxNEED_WX_STRING_H
 
 #ifdef wxNEED_WX_STDIO_H
 WXDLLEXPORT FILE * wxFopen(const wxChar *path, const wxChar *mode)
 {
-  return fopen(wxConvFile.cWX2MB(path), wxConvLibc.cWX2MB(mode));
+    char mode_buffer[10];
+    for (size_t i = 0; i < wxStrlen(mode)+1; i++)
+       mode_buffer[i] = (char) mode[i];
+
+    return fopen( wxConvFile.cWX2MB(path), mode_buffer );
 }
 
 WXDLLEXPORT FILE * wxFreopen(const wxChar *path, const wxChar *mode, FILE *stream)
 {
-  return freopen(wxConvFile.cWX2MB(path), wxConvLibc.cWX2MB(mode), stream);
+    char mode_buffer[10];
+    for (size_t i = 0; i < wxStrlen(mode)+1; i++)
+       mode_buffer[i] = (char) mode[i];
+
+    return freopen( wxConvFile.cWX2MB(path), mode_buffer, stream );
 }
 
 WXDLLEXPORT int wxRemove(const wxChar *path)
 {
-  return remove(wxConvFile.cWX2MB(path));
+    return remove( wxConvFile.cWX2MB(path) );
 }
 
 WXDLLEXPORT int wxRename(const wxChar *oldpath, const wxChar *newpath)
 {
-  return rename(wxConvFile.cWX2MB(oldpath), wxConvFile.cWX2MB(newpath));
-}
-
-int WXDLLEXPORT wxPrintf(const wxChar *fmt, ...)
-{
-  va_list argptr;
-  int ret;
-
-  va_start(argptr, fmt);
-  ret = wxVprintf(fmt, argptr);
-  va_end(argptr);
-  return ret;
-}
-
-int WXDLLEXPORT wxVprintf(const wxChar *fmt, va_list argptr)
-{
-  wxString str;
-  str.PrintfV(fmt,argptr);
-  printf("%s", (const char*)str.mb_str());
-  return str.Len();
-}
-
-int WXDLLEXPORT wxFprintf(FILE *stream, const wxChar *fmt, ...)
-{
-  va_list argptr;
-  int ret;
-
-  va_start(argptr, fmt);
-  ret = wxVfprintf(stream, fmt, argptr);
-  va_end(argptr);
-  return ret;
-}
-
-int WXDLLEXPORT wxVfprintf(FILE *stream, const wxChar *fmt, va_list argptr)
-{
-  wxString str;
-  str.PrintfV(fmt,argptr);
-  fprintf(stream, "%s", (const char*)str.mb_str());
-  return str.Len();
-}
-
-int WXDLLEXPORT wxSprintf(wxChar *buf, const wxChar *fmt, ...)
-{
-  va_list argptr;
-  int ret;
-
-  va_start(argptr, fmt);
-  ret = wxVsprintf(buf, fmt, argptr);
-  va_end(argptr);
-  return ret;
-}
-
-int WXDLLEXPORT wxVsprintf(wxChar *buf, const wxChar *fmt, va_list argptr)
-{
-  // this might be sort of inefficient, but it doesn't matter since
-  // we'd prefer people to use wxString::Printf directly instead anyway
-  wxString str;
-  str.PrintfV(fmt,argptr);
-  wxStrcpy(buf,str.c_str());
-  return str.Len();
-}
-
-int WXDLLEXPORT wxSscanf(const wxChar *buf, const wxChar *fmt, ...)
-{
-  va_list argptr;
-  int ret;
-
-  va_start(argptr, fmt);
-  ret = wxVsscanf(buf, fmt, argptr);
-  va_end(argptr);
-  return ret;
-}
-
-int WXDLLEXPORT wxVsscanf(const wxChar *buf, const wxChar *fmt, va_list argptr)
-{
-  int ret;
-  // this will work only for numeric conversion! Strings will not be converted correctly
-  // hopefully this is all we'll need
-  ret = vsscanf(wxConvLibc.cWX2MB(buf), wxConvLibc.cWX2MB(fmt), argptr);
-  return ret;
+    return rename( wxConvFile.cWX2MB(oldpath), wxConvFile.cWX2MB(newpath) );
 }
 #endif
 
 #ifndef wxAtof
 double   WXDLLEXPORT wxAtof(const wxChar *psz)
 {
-  return atof(wxConvLibc.cWX2MB(psz));
+  return atof(wxConvLocal.cWX2MB(psz));
 }
 #endif
 
 #ifdef wxNEED_WX_STDLIB_H
 int      WXDLLEXPORT wxAtoi(const wxChar *psz)
 {
-  return atoi(wxConvLibc.cWX2MB(psz));
+  return atoi(wxConvLocal.cWX2MB(psz));
 }
 
 long     WXDLLEXPORT wxAtol(const wxChar *psz)
 {
-  return atol(wxConvLibc.cWX2MB(psz));
+  return atol(wxConvLocal.cWX2MB(psz));
 }
 
 wxChar * WXDLLEXPORT wxGetenv(const wxChar *name)
 {
   static wxHashTable env;
+
   // check if we already have stored the converted env var
   wxObject *data = env.Get(name);
-  if (!data) {
+  if (!data)
+  {
     // nope, retrieve it,
-    const char *val = getenv(wxConvLibc.cWX2MB(name));
+#if wxUSE_UNICODE
+    wxCharBuffer buffer = wxConvLocal.cWX2MB(name);
+    // printf( "buffer %s\n", (const char*) buffer );
+    const char *val = getenv( (const char *)buffer );
+#else
+    const char *val = getenv( name );
+#endif
+
     if (!val) return (wxChar *)NULL;
+    // printf( "home %s\n", val );
+
     // convert it,
+#ifdef wxUSE_UNICODE
+    data = (wxObject *)new wxString(val, wxConvLocal);
+#else
     data = (wxObject *)new wxString(val);
+#endif
+
     // and store it
     env.Put(name, data);
   }
@@ -487,9 +1220,9 @@ wxChar * WXDLLEXPORT wxGetenv(const wxChar *name)
   return (wxChar *)((wxString *)data)->c_str();
 }
 
-int      WXDLLEXPORT wxSystem(const wxChar *psz)
+int WXDLLEXPORT wxSystem(const wxChar *psz)
 {
-  return system(wxConvLibc.cWX2MB(psz));
+    return system(wxConvLocal.cWX2MB(psz));
 }
 
 #endif
@@ -497,17 +1230,21 @@ int      WXDLLEXPORT wxSystem(const wxChar *psz)
 #ifdef wxNEED_WX_TIME_H
 WXDLLEXPORT size_t   wxStrftime(wxChar *s, size_t max, const wxChar *fmt, const struct tm *tm)
 {
-  if (!max) return 0;
-  char *buf = (char *)malloc(max);
-  size_t ret = strftime(buf, max, wxConvLibc.cWX2MB(fmt), tm);
-  if (ret) {
-    wxStrcpy(s, wxConvLibc.cMB2WX(buf));
-    free(buf);
-    return wxStrlen(s);
-  } else {
-    free(buf);
-    *s = 0;
-    return 0;
+    if (!max) return 0;
+
+    char *buf = (char *)malloc(max);
+    size_t ret = strftime(buf, max, wxConvLocal.cWX2MB(fmt), tm);
+    if (ret)
+    {
+        wxStrcpy(s, wxConvLocal.cMB2WX(buf));
+        free(buf);
+        return wxStrlen(s);
+    }
+    else
+    {
+        free(buf);
+        *s = 0;
+        return 0;
   }
 }
 #endif