diff Objects/unicodeobject.c @ 46026:f1ecb3e978db legacy-trunk

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.
author Neal Norwitz <nnorwitz@gmail.com>
date Sun, 23 Mar 2008 06:19:57 +0000
parents d33cadb595f6
children 8710ba7fba2b
line wrap: on
line diff
--- a/Objects/unicodeobject.c	Sun Mar 23 06:16:04 2008 +0000
+++ b/Objects/unicodeobject.c	Sun Mar 23 06:19:57 2008 +0000
@@ -624,9 +624,9 @@
 		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 @@
 			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.