]> granicus.if.org Git - graphviz/commitdiff
Resolve 7 warnings in colxlate.c
authorErwin Janssen <erwinjanssen@outlook.com>
Thu, 1 Sep 2016 19:18:16 +0000 (21:18 +0200)
committerErwin Janssen <erwinjanssen@outlook.com>
Thu, 1 Sep 2016 19:21:33 +0000 (21:21 +0200)
In three function, the type of two local variables could safely be changed to `size_t`.

lib/common/colxlate.c

index 73d9ba10550b6b6c9c8e19f10494038e02c94092..8d2a7e42718651aae861927009d27e4b1c265048 100644 (file)
@@ -145,9 +145,9 @@ static int colorcmpf(const void *p0, const void *p1)
 char *canontoken(char *str)
 {
     static unsigned char *canon;
-    static int allocated;
+    static size_t allocated;
     unsigned char c, *p, *q;
-    int len;
+    size_t len;
 
     p = (unsigned char *) str;
     len = strlen(str);
@@ -162,7 +162,7 @@ char *canontoken(char *str)
        /* if (isalnum(c) == FALSE) */
            /* continue; */
        if (isupper(c))
-           c = tolower(c);
+           c = (unsigned char) tolower(c);
        *q++ = c;
     }
     *q = '\0';
@@ -175,8 +175,8 @@ char *canontoken(char *str)
 static char* fullColor (char* prefix, char* str)
 {
     static char *fulls;
-    static int allocated;
-    int len = strlen (prefix) + strlen (str) + 3;
+    static size_t allocated;
+    size_t len = strlen(prefix) + strlen(str) + 3;
 
     if (len >= allocated) {
        allocated = len + 10;
@@ -255,14 +255,15 @@ int colorxlate(char *str, gvcolor_t * color, color_type_t target_type)
 {
     static hsvrgbacolor_t *last;
     static unsigned char *canon;
-    static int allocated;
+    static size_t allocated;
     unsigned char *p, *q;
     hsvrgbacolor_t fake;
     unsigned char c;
     double H, S, V, A, R, G, B;
     double C, M, Y, K;
     unsigned int r, g, b, a;
-    int len, rc;
+    size_t len;
+    int rc;
 
     color->type = target_type;