diff options
| author | Neal Norwitz <nnorwitz@gmail.com> | 2008-03-23 06:19:57 (GMT) |
|---|---|---|
| committer | Neal Norwitz <nnorwitz@gmail.com> | 2008-03-23 06:19:57 (GMT) |
| commit | ade57d048527f35a6086040a14a0e7184faa91ce (patch) | |
| tree | 42142a68d3a30bc793fd7706853ad292581061a4 /Python/pystrcmp.c | |
| parent | acdb6fb2a58bf7a4cbeb647c2a9201eb9009f68a (diff) | |
| download | cpython-ade57d048527f35a6086040a14a0e7184faa91ce.zip cpython-ade57d048527f35a6086040a14a0e7184faa91ce.tar.gz cpython-ade57d048527f35a6086040a14a0e7184faa91ce.tar.bz2 | |
Remove compiler warnings (on Alpha at least) about using chars as
array subscripts. Using chars are dangerous b/c they are signed
on some platforms and unsigned on others.
Diffstat (limited to 'Python/pystrcmp.c')
| -rw-r--r-- | Python/pystrcmp.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/Python/pystrcmp.c b/Python/pystrcmp.c index 0012ef3..84295e7 100644 --- a/Python/pystrcmp.c +++ b/Python/pystrcmp.c @@ -1,4 +1,4 @@ -/* Cross platform case insenstive string compare functions +/* Cross platform case insensitive string compare functions */ #include "Python.h" @@ -8,18 +8,19 @@ PyOS_mystrnicmp(const char *s1, const char *s2, Py_ssize_t size) { if (size == 0) return 0; - while ((--size > 0) && (tolower(*s1) == tolower(*s2))) { + while ((--size > 0) && + (tolower((unsigned)*s1) == tolower((unsigned)*s2))) { if (!*s1++ || !*s2++) break; } - return tolower(*s1) - tolower(*s2); + return tolower((unsigned)*s1) - tolower((unsigned)*s2); } int PyOS_mystricmp(const char *s1, const char *s2) { - while (*s1 && (tolower(*s1++) == tolower(*s2++))) { + while (*s1 && (tolower((unsigned)*s1++) == tolower((unsigned)*s2++))) { ; } - return (tolower(*s1) - tolower(*s2)); + return (tolower((unsigned)*s1) - tolower((unsigned)*s2)); } |