Use int for the sign rather than a char. char can be signed or unsigned.

It's system dependent.  This might fix the problem with test_rfind failing.

diff --git a/Modules/mmapmodule.c b/Modules/mmapmodule.c
index defef4111120c6cb4445ce414915e3c13cf246fe..49fe7f7c72c563461f0e35e3af2e86db8a73a3b9 100644 (file)
--- a/Modules/mmapmodule.c
+++ b/Modules/mmapmodule.c
@@ -263,7 +263,7 @@ mmap_gfind(mmap_object *self,
                return NULL;
        } else {
                const char *p, *start_p, *end_p;
-               char sign = reverse ? -1 : 1;
+               int sign = reverse ? -1 : 1;
 
                 if (start < 0)
                        start += self->size;