diff options
| author | Peter Stephenson <pws@users.sourceforge.net> | 2006-08-04 13:38:26 +0000 |
|---|---|---|
| committer | Peter Stephenson <pws@users.sourceforge.net> | 2006-08-04 13:38:26 +0000 |
| commit | 596d38fb078aaa075bde47920b25c1b913056ea8 (patch) | |
| tree | 25c1fed73ed959bb8d69250d0c1018cb617dbcc9 /Src/utils.c | |
| parent | 99fa8b90b24b17536d95285c0b64d9b336570ff1 (diff) | |
| download | zsh-596d38fb078aaa075bde47920b25c1b913056ea8.tar.gz zsh-596d38fb078aaa075bde47920b25c1b913056ea8.zip | |
22586: fix warnings spotted on Cygwin
Diffstat (limited to 'Src/utils.c')
| -rw-r--r-- | Src/utils.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/Src/utils.c b/Src/utils.c index 0dcf4e59b..8fdf2c0ab 100644 --- a/Src/utils.c +++ b/Src/utils.c @@ -79,10 +79,15 @@ set_widearray(char *mb_array, Widechar_array wca) if (!mblen) break; /* No good unless all characters are convertible */ - if (*wcptr == WEOF) + if (wci == WEOF) return; *wcptr++ = (wchar_t)wci; #ifdef DEBUG + /* + * This generates a warning from the compiler (and is + * indeed useless) if chars are unsigned. It's + * extreme paranoia anyway. + */ if (wcptr[-1] < 0) fprintf(stderr, "BUG: Bad cast to wchar_t\n"); #endif @@ -501,6 +506,9 @@ wcs_nicechar(wchar_t c, size_t *widthp, char **swidep) /* * Can't or don't want to convert character: use UCS-2 or * UCS-4 code in print escape format. + * + * This comparison fails and generates a compiler warning + * if wchar_t is 16 bits, but the code is still correct. */ if (c >= 0x10000) { sprintf(buf, "\\U%.8x", (unsigned int)c); |