diff options
author | Rich Felker <dalias@aerifal.cx> | 2011-06-07 11:26:42 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2011-06-07 11:26:42 -0400 |
commit | 0b6b43ed3fd26cf8cd926193be5c9fd831b534c4 (patch) | |
tree | 32c878f70b8b996ef69f6b0f7efbfdaf2a5bc74b | |
parent | 86f8c72bb1cc1fad05e1ed1b2a6f4433defc9cf7 (diff) | |
download | musl-0b6b43ed3fd26cf8cd926193be5c9fd831b534c4.tar.gz musl-0b6b43ed3fd26cf8cd926193be5c9fd831b534c4.tar.xz musl-0b6b43ed3fd26cf8cd926193be5c9fd831b534c4.zip |
use __WCHAR_TYPE__ on i386 if it is defined
unfortunately traditional i386 practice was to use "long" rather than "int" for wchar_t, despite the latter being much more natural and logical. we followed this practice, but it seems some compilers (clang and maybe certain gcc builds or others too..?) have switched to using int, resulting in spurious pointer type mismatches when L"..." wide strings are used. the best solution I could find is to use the compiler's definition of wchar_t if it exists, and otherwise fallback to the traditional definition. there's no point in duplicating this approach on 64-bit archs, as their only 32-bit type is int.
-rwxr-xr-x | arch/i386/bits/alltypes.h.sh | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/arch/i386/bits/alltypes.h.sh b/arch/i386/bits/alltypes.h.sh index 335c0957..672d6a45 100755 --- a/arch/i386/bits/alltypes.h.sh +++ b/arch/i386/bits/alltypes.h.sh @@ -26,7 +26,11 @@ TYPEDEF __builtin_va_list va_list; TYPEDEF struct __va_list * va_list; #endif +#ifdef __WCHAR_TYPE__ +TYPEDEF __WCHAR_TYPE__ wchar_t; +#else TYPEDEF long wchar_t; +#endif TYPEDEF long wint_t; TYPEDEF long wctrans_t; TYPEDEF long wctype_t; |