canontoken: use char pointers instead of unsigned char pointers

It is not clear to me why this code was using unsigned char pointers.

diff --git a/lib/common/colxlate.c b/lib/common/colxlate.c
index fd8ecfa6fe0772bb02846291e78a6b6f326cf812..e65c1de08e8eeccd36df1194163b24e2b0dc25d4 100644 (file)
--- a/lib/common/colxlate.c
+++ b/lib/common/colxlate.c
@@ -131,12 +131,12 @@ static int colorcmpf(const void *p0, const void *p1)
 
 char *canontoken(char *str)
 {
-    static unsigned char *canon;
+    static char *canon;
     static size_t allocated;
-    unsigned char c, *p, *q;
+    char c, *p, *q;
     size_t len;
 
-    p = (unsigned char *) str;
+    p = str;
     len = strlen(str);
     if (len >= allocated) {
        allocated = len + 1 + 10;
@@ -145,11 +145,11 @@ char *canontoken(char *str)
     q = canon;
     while ((c = *p++)) {
        if (isupper(c))
-           c = (unsigned char) tolower(c);
+           c = (char)tolower(c);
        *q++ = c;
     }
     *q = '\0';
-    return (char*)canon;
+    return canon;
 }
 
 /* fullColor: