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 /Objects | |
| 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 'Objects')
| -rw-r--r-- | Objects/unicodeobject.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Objects/unicodeobject.c b/Objects/unicodeobject.c index dd56e11..5878f96 100644 --- a/Objects/unicodeobject.c +++ b/Objects/unicodeobject.c @@ -624,9 +624,9 @@ PyUnicode_FromFormatV(const char *format, va_list vargs) if (*f == '%') { const char* p = f; width = 0; - while (isdigit(*f)) + while (isdigit((unsigned)*f)) width = (width*10) + *f++ - '0'; - while (*++f && *f != '%' && !isalpha(*f)) + while (*++f && *f != '%' && !isalpha((unsigned)*f)) ; /* skip the 'l' or 'z' in {%ld, %zd, %lu, %zu} since @@ -787,12 +787,12 @@ PyUnicode_FromFormatV(const char *format, va_list vargs) zeropad = (*f == '0'); /* parse the width.precision part */ width = 0; - while (isdigit(*f)) + while (isdigit((unsigned)*f)) width = (width*10) + *f++ - '0'; precision = 0; if (*f == '.') { f++; - while (isdigit(*f)) + while (isdigit((unsigned)*f)) precision = (precision*10) + *f++ - '0'; } /* handle the long flag, but only for %ld and %lu. |