X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/b63b07a809f9a3d22596d4971ef5c8971153823a..ed7c11abc829a8f7becc92ee438866e36f2216fe:/src/common/wxchar.cpp diff --git a/src/common/wxchar.cpp b/src/common/wxchar.cpp index 655a9b58b2..e09e7e06b6 100644 --- a/src/common/wxchar.cpp +++ b/src/common/wxchar.cpp @@ -5,7 +5,7 @@ // Modified by: Ron Lee // Created: 09/04/99 // RCS-ID: $Id$ -// Copyright: (c) wxWindows copyright +// Copyright: (c) wxWidgets copyright // Licence: wxWindows licence ///////////////////////////////////////////////////////////////////////////// @@ -130,11 +130,11 @@ bool WXDLLEXPORT wxOKlibc() (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; + return false; } } #endif - return TRUE; + return true; } // ============================================================================ @@ -180,10 +180,10 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, { static char s_szFlags[256] = "%"; size_t flagofs = 1; - bool adj_left = FALSE, - in_prec = FALSE, - prec_dot = FALSE, - done = FALSE; + bool adj_left = false, + in_prec = false, + prec_dot = false, + done = false; int ilen = 0; size_t min_width = 0, max_width = wxSTRING_MAXLEN; @@ -194,14 +194,16 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, if (in_prec && !prec_dot) \ { \ s_szFlags[flagofs++] = '.'; \ - prec_dot = TRUE; \ + prec_dot = true; \ } #define APPEND_CH(ch) \ - if ( lenCur == lenMax ) \ - return -1; \ - \ - buf[lenCur++] = ch + { \ + if ( lenCur == lenMax ) \ + return -1; \ + \ + buf[lenCur++] = ch; \ + } #define APPEND_STR(s) \ { \ @@ -218,12 +220,12 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, case wxT('\0'): APPEND_CH(_T('\0')); - done = TRUE; + done = true; break; case wxT('%'): APPEND_CH(_T('%')); - done = TRUE; + done = true; break; case wxT('#'): @@ -237,14 +239,14 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, case wxT('-'): CHECK_PREC - adj_left = TRUE; + adj_left = true; s_szFlags[flagofs++] = ch; break; case wxT('.'): CHECK_PREC - in_prec = TRUE; - prec_dot = FALSE; + in_prec = true; + prec_dot = false; max_width = 0; // dot will be auto-added to s_szFlags if non-negative // number follows @@ -370,7 +372,7 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, APPEND_STR(tmp); } - done = TRUE; + done = true; break; case wxT('e'): @@ -398,7 +400,7 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, APPEND_STR(tmp); } - done = TRUE; + done = true; break; case wxT('p'): @@ -413,52 +415,77 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, wxConvLibc.cMB2WX(szScratch); APPEND_STR(tmp); - done = TRUE; + done = true; } break; case wxT('c'): { - wxChar val = va_arg(argptr, int); - // we don't need to honor padding here, do we? + int val = va_arg(argptr, int); +#if wxUSE_UNICODE + if (ilen == -1) + { + const char buf[2] = { val, 0 }; + val = wxString(buf, wxConvLibc)[0u]; + } +#elif wxUSE_WCHAR_T + if (ilen == 1) + { + const wchar_t buf[2] = { val, 0 }; + val = wxString(buf, wxConvLibc)[0u]; + } +#endif + size_t i; + + if (!adj_left) + for (i = 1; i < min_width; i++) + APPEND_CH(_T(' ')); + APPEND_CH(val); - done = TRUE; + if (adj_left) + for (i = 1; i < min_width; i++) + APPEND_CH(_T(' ')); + + 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 *); + const wxChar *val = NULL; #if wxUSE_UNICODE - // ASCII->Unicode constructor handles max_width - // right - wxString s(val, wxConvLibc, max_width); -#else - size_t len = wxSTRING_MAXLEN; - if (val) + wxString s; + + if (ilen == -1) { - for ( len = 0; - val[len] && (len < max_width); - len++ ) - ; + // wx extension: we'll let %hs mean non-Unicode + // strings + char *v = va_arg(argptr, char *); + + if (v) + val = s = wxString(v, wxConvLibc); + } + else +#elif wxUSE_WCHAR_T + wxString s; + + if (ilen == 1) + { + // %ls means Unicode strings + wchar_t *v = va_arg(argptr, wchar_t *); + + if (v) + val = s = wxString(v, wxConvLibc); } else - val = wxT("(null)"); - wxString s(val, len); #endif - if (s.Len() < min_width) - s.Pad(min_width - s.Len(), wxT(' '), adj_left); + { + val = va_arg(argptr, wxChar *); + } + + size_t len = 0; - APPEND_STR(s); - } - else - { - wxChar *val = va_arg(argptr, wxChar *); - size_t len = wxSTRING_MAXLEN; if (val) { for ( len = 0; @@ -466,16 +493,32 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, len++ ) ; } - else + else if (max_width >= 6) + { val = wxT("(null)"); + len = 6; + } + else + { + val = wxT(""); + len = 0; + } + + size_t i; + + if (!adj_left) + for (i = len; i < min_width; i++) + APPEND_CH(_T(' ')); - wxString s(val, len); - if (s.Len() < min_width) - s.Pad(min_width - s.Len(), wxT(' '), adj_left); + for (i = 0; i < len; i++) + APPEND_CH(val[i]); - APPEND_STR(s); + if (adj_left) + for (i = len; i < min_width; i++) + APPEND_CH(_T(' ')); + + done = true; } - done = TRUE; break; case wxT('n'): @@ -494,14 +537,14 @@ int WXDLLEXPORT wxVsnprintf_(wxChar *buf, size_t lenMax, long int *val = va_arg(argptr, long int *); *val = lenCur; } - done = TRUE; + done = true; break; default: // bad format, leave unchanged APPEND_CH(_T('%')); APPEND_CH(ch); - done = TRUE; + done = true; break; } } @@ -561,23 +604,23 @@ int WXDLLEXPORT wxSnprintf_(wxChar *buf, size_t len, const wxChar *format, ...) // implement the standard IO functions for wide char if libc doesn't have them // ---------------------------------------------------------------------------- -#ifdef wxNEED_FPUTWC - +#ifdef wxNEED_FPUTS 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; } +#endif // wxNEED_FPUTS +#ifdef wxNEED_PUTC int /* not wint_t */ wxPutc(wchar_t wc, FILE *stream) { wchar_t ws[2] = { wc, L'\0' }; return wxFputs(ws, stream); } - -#endif // wxNEED_FPUTWC +#endif // wxNEED_PUTC // NB: we only implement va_list functions here, the ones taking ... are // defined below for wxNEED_PRINTF_CONVERSION case anyhow and we reuse @@ -661,7 +704,7 @@ int vwprintf(const wxChar *format, va_list argptr) 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 + wxWidgets specifier POSIX specifier ---------------------------------------- %hc, %C, %hC %c @@ -731,10 +774,10 @@ private: ch == _T('0') || ch == _T(' ') || ch == _T('#'); } - void SkipDigits(const wxChar **ppc) + void SkipDigits(const wxChar **ptpc) { - while ( **ppc >= _T('0') && **ppc <= _T('9') ) - CopyFmtChar(*(*ppc)++); + while ( **ptpc >= _T('0') && **ptpc <= _T('9') ) + CopyFmtChar(*(*ptpc)++); } // the translated format @@ -919,23 +962,14 @@ int wxSprintf( wxChar *str, const wxChar *format, ... ) va_list argptr; va_start(argptr, format); - // callers of wxSprintf() deserve what they get - //int ret = vswprintf( str, UINT_MAX, wxFormatConverter(format), argptr ); - - // ... true, but if we are going to implement it, they probably still - // deserve something a little better than absolutely guaranteed silent - // failure. For some (very mysterious) reason, this call fails under glibc - // 2.3.2 if str was allocated on the heap and maxsize is larger than this. - // Even more mysterious is that it does still succeed if str was allocated - // on the stack. This should still be plenty large enough for people who - // want to overflow a buffer. The bug was first noticed in unicode builds - // of tex2rtf, but I'm going to fix that to not use this unsafe function - // instead of wasting time diagnosing this further right now. - int ret = vswprintf( str, INT_MAX / 4, wxFormatConverter(format), argptr ); + // note that wxString::FormatV() uses wxVsnprintf(), not wxSprintf(), so + // it's safe to implement this one in terms of it + wxString s(wxString::FormatV(format, argptr)); + wxStrcpy(str, s); va_end(argptr); - return ret; + return s.length(); } int wxFprintf( FILE *stream, const wxChar *format, ... ) @@ -1295,36 +1329,16 @@ long WXDLLEXPORT wxAtol(const wxChar *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) - { - // nope, retrieve it, #if wxUSE_UNICODE - wxCharBuffer buffer = wxConvLocal.cWX2MB(name); - // printf( "buffer %s\n", (const char*) buffer ); - const char *val = getenv( (const char *)buffer ); + // NB: buffer returned by getenv() is allowed to be overwritten next + // time getenv() is called, so it is OK to use static string + // buffer to hold the data. + static wxWCharBuffer value((wxChar*)NULL); + value = wxConvLocal.cMB2WX(getenv(wxConvLocal.cWX2MB(name))); + return value.data(); #else - const char *val = getenv( name ); + return getenv(name); #endif - - if (!val) return (wxChar *)NULL; - // printf( "home %s\n", val ); - - // convert it, -#if wxUSE_UNICODE - data = (wxObject *)new wxString(val, wxConvLocal); -#else - data = (wxObject *)new wxString(val); -#endif - - // and store it - env.Put(name, data); - } - // return converted env var - return (wxChar *)((wxString *)data)->c_str(); } int WXDLLEXPORT wxSystem(const wxChar *psz)