]> granicus.if.org Git - graphviz/commitdiff
use C99 bools to set 'onstack'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 16 Jan 2022 19:30:36 +0000 (11:30 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 20 Jan 2022 02:15:00 +0000 (18:15 -0800)
cmd/tools/acyclic.c
lib/common/ns.c
lib/dotgen/acyclic.c
lib/dotgen/mincross.c
lib/dotgen/rank.c
lib/neatogen/neatoinit.c

index 8ad095bba8adfd04f3560453d07a416d6c0c2a38..903804bf9a421d238efdc270f4f2070b9b3f872a 100644 (file)
@@ -19,6 +19,7 @@
 #ifdef HAVE_UNISTD_H
 #include <unistd.h>
 #endif
+#include <stdbool.h>
 #include <stdio.h>
 
 #include <stdlib.h>
@@ -72,7 +73,7 @@ static int dfs(Agraph_t * g, Agnode_t * t, int hasCycle)
     Agnode_t *h;
 
     ND_mark(t) = 1;
-    ND_onstack(t) = 1;
+    ND_onstack(t) = true;
     for (e = agfstout(g, t); e; e = f) {
        f = agnxtout(g, e);
        if (agtail(e) == aghead(e))
@@ -92,7 +93,7 @@ static int dfs(Agraph_t * g, Agnode_t * t, int hasCycle)
        } else if (ND_mark(h) == 0)
            hasCycle |= dfs(g, h, hasCycle);
     }
-    ND_onstack(t) = 0;
+    ND_onstack(t) = false;
     return hasCycle;
 }
 
index 430da327a36e9a4e33cb24c98e40337e1ca43701..3c2244419f9bb16df01ed38cdb67e78e9934304c 100644 (file)
@@ -14,6 +14,7 @@
  */
 
 #include <common/render.h>
+#include <stdbool.h>
 
 static void dfs_cutval(node_t * v, edge_t * par);
 static int dfs_range(node_t * v, edge_t * par, int low);
@@ -1146,7 +1147,7 @@ static node_t *checkdfs(graph_t* g, node_t * n)
     if (ND_mark(n))
        return 0;
     ND_mark(n) = TRUE;
-    ND_onstack(n) = TRUE;
+    ND_onstack(n) = true;
     for (i = 0; (e = ND_out(n).list[i]); i++) {
        w = aghead(e);
        if (ND_onstack(w)) {
@@ -1173,15 +1174,17 @@ static node_t *checkdfs(graph_t* g, node_t * n)
            }
        }
     }
-    ND_onstack(n) = FALSE;
+    ND_onstack(n) = false;
     return 0;
 }
 
 void check_cycles(graph_t * g)
 {
     node_t *n;
-    for (n = GD_nlist(g); n; n = ND_next(n))
-       ND_mark(n) = ND_onstack(n) = FALSE;
+    for (n = GD_nlist(g); n; n = ND_next(n)) {
+       ND_mark(n) = FALSE;
+       ND_onstack(n) = false;
+    }
     for (n = GD_nlist(g); n; n = ND_next(n))
        checkdfs(g, n);
 }
index 1cd0bffc8b271c1a1921fd6f61a02ce478ad0c68..f98ed93adb08b0ece67adee215441e2a2d938faa 100644 (file)
@@ -14,6 +14,7 @@
  */
 
 #include <dotgen/dot.h>
+#include <stdbool.h>
 
 void reverse_edge(edge_t * e)
 {
@@ -36,7 +37,7 @@ dfs(node_t * n)
     if (ND_mark(n))
        return;
     ND_mark(n) = TRUE;
-    ND_onstack(n) = TRUE;
+    ND_onstack(n) = true;
     for (i = 0; (e = ND_out(n).list[i]); i++) {
        w = aghead(e);
        if (ND_onstack(w)) {
@@ -47,7 +48,7 @@ dfs(node_t * n)
                dfs(w);
        }
     }
-    ND_onstack(n) = FALSE;
+    ND_onstack(n) = false;
 }
 
 
index 43e3d4c6d4560e4ecf6243fbcd1af6dde8a160aa..32609344cf9e5edccbc7c03fe13eee98cfa390e4 100644 (file)
@@ -1214,7 +1214,7 @@ static void flat_search(graph_t * g, node_t * v)
     adjmatrix_t *M = GD_rank(g)[ND_rank(v)].flat;
 
     ND_mark(v) = TRUE;
-    ND_onstack(v) = TRUE;
+    ND_onstack(v) = true;
     hascl = GD_n_cluster(dot_root(g)) > 0;
     if (ND_flat_out(v).list)
        for (i = 0; (e = ND_flat_out(v).list[i]); i++) {
@@ -1239,7 +1239,7 @@ static void flat_search(graph_t * g, node_t * v)
                    flat_search(g, aghead(e));
            }
        }
-    ND_onstack(v) = FALSE;
+    ND_onstack(v) = false;
 }
 
 static void flat_breakcycles(graph_t * g)
@@ -1251,7 +1251,8 @@ static void flat_breakcycles(graph_t * g)
        flat = 0;
        for (i = 0; i < GD_rank(g)[r].n; i++) {
            v = GD_rank(g)[r].v[i];
-           ND_mark(v) = ND_onstack(v) = FALSE;
+           ND_mark(v) = FALSE;
+           ND_onstack(v) = false;
            flatindex(v) = i;
            if (ND_flat_out(v).size > 0 && flat == 0) {
                GD_rank(g)[r].flat =
index e420d144aaa695f84ff3cf24a758ed7c097b9aee..f0a9f71ca0a2e0927480a1944701e4c6fc420842 100644 (file)
@@ -938,7 +938,7 @@ static void dfs(graph_t * g, node_t * v)
     if (ND_mark(v))
        return;
     ND_mark(v) = TRUE;
-    ND_onstack(v) = TRUE;
+    ND_onstack(v) = true;
     for (e = agfstout(g, v); e; e = f) {
        f = agnxtout(g, e);
        w = aghead(e);
@@ -949,15 +949,17 @@ static void dfs(graph_t * g, node_t * v)
                dfs(g, w);
        }
     }
-    ND_onstack(v) = FALSE;
+    ND_onstack(v) = false;
 }
 
 static void break_cycles(graph_t * g)
 {
     node_t *n;
 
-    for (n = agfstnode(g); n; n = agnxtnode(g, n))
-       ND_mark(n) = ND_onstack(n) = FALSE;
+    for (n = agfstnode(g); n; n = agnxtnode(g, n)) {
+       ND_mark(n) = FALSE;
+       ND_onstack(n) = false;
+    }
     for (n = agfstnode(g); n; n = agnxtnode(g, n))
        dfs(g, n);
 }
index d1d9b0386f88607ca00665b685407c9a6980516c..b27539f86c8f10852ae884a13da88cc1343484f2 100644 (file)
@@ -709,7 +709,7 @@ dfsCycle (vtx_data* graph, int i,int mode, node_t* nodes[])
 
     np = nodes[i];
     ND_mark(np) = TRUE;
-    ND_onstack(np) = TRUE;
+    ND_onstack(np) = true;
     for (e = 1; e < graph[i].nedges; e++) {
        if (graph[i].edists[e] == 1.0) continue;  /* in edge */
        j = graph[i].edges[e];
@@ -723,7 +723,7 @@ dfsCycle (vtx_data* graph, int i,int mode, node_t* nodes[])
        else if (!ND_mark(hp)) dfsCycle(graph, j, mode, nodes);
 
     }
-    ND_onstack(np) = FALSE;
+    ND_onstack(np) = false;
 }
 
 /* acyclic:
@@ -738,7 +738,7 @@ acyclic (vtx_data* graph, int nv, int mode, node_t* nodes[])
     for (i = 0; i < nv; i++) {
        np = nodes[i];
        ND_mark(np) = FALSE;
-       ND_onstack(np) = FALSE;
+       ND_onstack(np) = false;
     }
     for (i = 0; i < nv; i++) {
        if (ND_mark(nodes[i])) continue;