Address warnings issued by the MSVC++ compiler

gvanrossum · gvanrossum · commit 39b0f8976cae · 1998-04-10T21:52:06.000Z
diff --git a/Modules/pcremodule.c b/Modules/pcremodule.c
@@ -193,7 +193,6 @@ PyPcre_compile(self, args)
 	PyObject *dictionary;
 	char *pattern;
 	const char *error;
-	int num_zeros, i, j;
 	
 	int options, erroroffset;
 	if (!PyArg_ParseTuple(args, "siO!", &pattern, &options,
diff --git a/Modules/pypcre.c b/Modules/pypcre.c
@@ -3044,8 +3044,8 @@ if (md->off_num)    free(md->off_num);
 if (md->offset_top) free(md->offset_top); 
 if (md->r1)         free(md->r1); 
 if (md->r2)         free(md->r2); 
-if (md->eptr)       free(md->eptr); 
-if (md->ecode)      free(md->ecode);
+if (md->eptr)       free((char *)md->eptr); 
+if (md->ecode)      free((char *)md->ecode);
 return 0;
 }
 
diff --git a/Python/import.c b/Python/import.c
@@ -858,7 +858,7 @@ find_module(name, path, buf, buflen, p_fp)
 	}
 	else if (PyString_Check(path)) {
 		/* Submodule of frozen package */
-		if (PyString_Size(path) + 1 + strlen(name) >= buflen) {
+		if (PyString_Size(path) + 1 + strlen(name) >= (unsigned int)buflen) {
 			PyErr_SetString(PyExc_ImportError,
 					"full frozen module name too long");
 			return NULL;
diff --git a/Python/mystrtoul.c b/Python/mystrtoul.c
@@ -129,7 +129,7 @@ int		base;
 	result = result * base + c;
 #ifndef MPW
 	if(base == 10) {
-		if(((long)(result - c) / base != temp))	/* overflow */
+		if(((long)(result - c) / base != (long)temp))	/* overflow */
 			ovf = 1;
 	}
 	else {

Original file line number	Diff line number	Diff line change
`@@ -858,7 +858,7 @@ find_module(name, path, buf, buflen, p_fp)`
`858`	`858`	`}`
`859`	`859`	`else if (PyString_Check(path)) {`
`860`	`860`	`/* Submodule of frozen package */`
`861`		`- if (PyString_Size(path) + 1 + strlen(name) >= buflen) {`
	`861`	`+ if (PyString_Size(path) + 1 + strlen(name) >= (unsigned int)buflen) {`
`862`	`862`	`PyErr_SetString(PyExc_ImportError,`
`863`	`863`	`"full frozen module name too long");`
`864`	`864`	`return NULL;`
Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ int base;`
`129`	`129`	`result = result * base + c;`
`130`	`130`	`#ifndef MPW`
`131`	`131`	`if(base == 10) {`
`132`		`- if(((long)(result - c) / base != temp)) /* overflow */`
	`132`	`+ if(((long)(result - c) / base != (long)temp)) /* overflow */`
`133`	`133`	`ovf = 1;`
`134`	`134`	`}`
`135`	`135`	`else {`