]> granicus.if.org Git - graphviz/commitdiff
gvpr colorxlate: use a char for 'c'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 29 Jan 2023 01:39:17 +0000 (17:39 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 29 Jan 2023 02:02:17 +0000 (18:02 -0800)
Similar to the previous commit, this allows dropping some casts.

lib/gvpr/actions.c

index 227490ff914c5ab1f7ec9e154bf0b09890ecaa5b..1e4bc361f9ccaa4e577976248aa0184beb9dd2ca 100644 (file)
@@ -1048,7 +1048,6 @@ int colorxlate(char *str, gvcolor_t * color, color_type_t target_type)
 {
     static hsvrgbacolor_t *last;
     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;
@@ -1113,11 +1112,12 @@ int colorxlate(char *str, gvcolor_t * color, color_type_t target_type)
     }
 
     /* test for hsv value such as: ".6,.5,.3" */
-    if ((c = *p) == '.' || isdigit(c)) {
+    char c;
+    if ((c = *p) == '.' || isdigit((int)c)) {
        int cnt;
        agxbuf canon = {0};
        while ((c = *p++)) {
-           agxbputc(&canon, c == ',' ? ' ' : (char)c);
+           agxbputc(&canon, c == ',' ? ' ' : c);
        }
 
        if ((cnt = sscanf(agxbuse(&canon), "%lf%lf%lf%lf", &H, &S, &V, &A)) >= 3) {