]> granicus.if.org Git - graphviz/commitdiff
gvmap plot_dot_polygons: use a 'size_t' for 'maxlen'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 20 Jan 2023 16:25:15 +0000 (08:25 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 21 Jan 2023 21:44:04 +0000 (13:44 -0800)
Squashes some -Wsign-conversion warnings.

cmd/gvmap/make_map.c

index 220a4f091230eaf668691a8d427baa8584ea3044..50e92025073869afa8b245b367182a5e72a2fa3c 100644 (file)
@@ -277,13 +277,19 @@ static void plot_dot_polygons(agxbuf *sbuff, double line_width,
                               double *x_poly, int *polys_groups, float *r,
                               float *g, float *b, const char *opacity) {
   int i, j, *ia = polys->ia, *ja = polys->ja, *a = polys->a, npolys = polys->m, nverts = polys->n, ipoly,first;
-  int np = 0, maxlen = 0;
+  int np = 0;
   float *xp, *yp;
   int fill = -1;
   char cstring[] = "#aaaaaaff";
   int use_line = (line_width >= 0);
   
-  for (i = 0; i < npolys; i++) maxlen = MAX(maxlen, ia[i+1]-ia[i]);
+  size_t maxlen = 0;
+  for (i = 0; i < npolys; i++) {
+    int len = ia[i + 1] - ia[i];
+    if (len > 0 && (size_t)len > maxlen) {
+      maxlen = (size_t)len;
+    }
+  }
 
   xp = gv_calloc(maxlen, sizeof(float));
   yp = gv_calloc(maxlen, sizeof(float));