Message114350
mbcs codec functions are surrounded by:
#if defined(MS_WINDOWS) && defined(HAVE_USABLE_WCHAR_T)
(especially in unicodeobject.c and _codecsmodule.c)
or
#ifdef MS_WIN32
(in unicodeobject.h)
or
#if defined(MS_WINDOWS) && !defined(__BORLANDC__)
(in timemodule.c)
I think that all of these tests are wrong. We should just check that we are compiling under Windows: mbcs functions don't use the wchar_t type. And it's better to use the same test in all tests (MS_WIN32 vs MS_WINDOWS).
Attached patch replaces all #ifdef (except the one in timemodule.c because I don't know what to do with the BORLAND check, does anyone use this compiler?).
I suppose that my patch doesn't change anything in pratice because mbcs is used in many places and noboby complained that mbcs encoding was missing on Windows. |
|
Date |
User |
Action |
Args |
2010-08-19 12:02:28 | vstinner | set | recipients:
+ vstinner, tim.golden, brian.curtin |
2010-08-19 12:02:28 | vstinner | set | messageid: <1282219348.43.0.267719038373.issue9642@psf.upfronthosting.co.za> |
2010-08-19 12:02:25 | vstinner | link | issue9642 messages |
2010-08-19 12:02:25 | vstinner | create | |
|