]> granicus.if.org Git - graphviz/commitdiff
make_regular_edge: use a proper boolean for checking edge type is spline
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 7 Oct 2021 03:48:36 +0000 (20:48 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 10 Oct 2021 18:15:11 +0000 (11:15 -0700)
This change also removes a -Wshadow compiler warning.

lib/dotgen/dotsplines.c

index b85686e9776778aaa5d5275cb41d5aa5b249a40d..4245552e76d2bea17a609ff78a30c3c68acb6d82 100644 (file)
@@ -18,6 +18,7 @@
 #include <dotgen/dot.h>
 #include <limits.h>
 #include <math.h>
+#include <stdbool.h>
 #include <stdlib.h>
 #include <string.h>
 
@@ -1844,7 +1845,7 @@ make_regular_edge(graph_t* g, spline_info_t* sp, path * P, edge_t ** edges, int
     if ((et == ET_LINE) && (pointn = makeLineEdge (g, fe, pointfs, &hn))) {
     }
     else {
-       int splines = et == ET_SPLINE;
+       bool is_spline = et == ET_SPLINE;
        boxes_t boxes = {0};
        pointn = 0;
        segfirst = e;
@@ -1887,7 +1888,7 @@ make_regular_edge(graph_t* g, spline_info_t* sp, path * P, edge_t ** edges, int
            assert(boxes.size <= (size_t)INT_MAX && "integer overflow");
            completeregularpath(P, segfirst, e, &tend, &hend, boxes.data,
                                (int)boxes.size, 1);
-           if (splines) ps = routesplines(P, &pn);
+           if (is_spline) ps = routesplines(P, &pn);
            else {
                ps = routepolylines (P, &pn);
                if ((et == ET_LINE) && (pn > 4)) {
@@ -1939,7 +1940,7 @@ make_regular_edge(graph_t* g, spline_info_t* sp, path * P, edge_t ** edges, int
        completeregularpath(P, segfirst, e, &tend, &hend, boxes.data, (int)boxes.size,
                            longedge);
        boxes_free(&boxes);
-       if (splines) ps = routesplines(P, &pn);
+       if (is_spline) ps = routesplines(P, &pn);
        else ps = routepolylines (P, &pn);
        if (et == ET_LINE && pn > 4) {
            /* Here we have used the polyline case to handle