]> granicus.if.org Git - graphviz/commitdiff
remove unnecessary 'dtlink' casts
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 22 Jun 2022 03:39:44 +0000 (20:39 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 25 Jun 2022 16:52:40 +0000 (09:52 -0700)
cmd/tools/gvpack.cpp
lib/circogen/edgelist.c
lib/common/htmlparse.y
lib/common/htmltable.c
lib/common/pointset.c
lib/neatogen/constraint.c

index 9a8be95641edafc1ebb690fe9a00c02d92dfad6c..5d01991b63c907fea2826b1efd5ca7edab18f01c 100644 (file)
@@ -439,7 +439,7 @@ fillGraph(Agraph_t * g, Dt_t * d,
          Agsym_t *(*setf)(Agraph_t*, char*, const char*), size_t cnt) {
     attr_t *av;
     for (av = (attr_t *) dtflatten(d); av;
-        av = (attr_t *) dtlink(d, (Dtlink_t *) av)) {
+        av = (attr_t *)dtlink(d, av)) {
        if (cnt == av->cnt)
            setf(g, av->name, av->value);
        else
index a3ebf6a9e1dce220bbe3de450973192ce2f85cd7..be0d1e57f942fcaaf9c4b04fff7a023a97d4cb2e 100644 (file)
@@ -82,7 +82,7 @@ void print_edge(edgelist * list)
     Agedge_t *ep;
 
     for (temp = (edgelistitem *) dtflatten(list); temp;
-        temp = (edgelistitem *) dtlink(list, (Dtlink_t *) temp)) {
+        temp = (edgelistitem *)dtlink(list, temp)) {
        ep = temp->edge;
        fprintf(stderr, "%s--", agnameof(agtail(ep)));
        fprintf(stderr, "%s \n", agnameof(aghead(ep)));
index 4304979a1b0a2589e628a23031b1a48001f0d7e4..998e307122390a9c7ad9b45eb143e91713c1261a 100644 (file)
@@ -196,7 +196,7 @@ appendFLineList (int v)
        ln->lp.items = N_NEW(cnt, textspan_t);
 
        fi = (fitem*)dtflatten(ilist);
-       for (; fi; fi = (fitem*)dtlink(fitemList,(Dtlink_t*)fi)) {
+       for (; fi; fi = (fitem*)dtlink(fitemList, fi)) {
                /* NOTE: When fitemList is closed, it uses free_item, which only frees the container,
                 * not the contents, so this copy is safe.
                 */
index a444d7ef8fc859ca95eff3ccd89c7ac0d1808366..59ed11d235d0dc3589c3d49cbe92537234eb705c 100644 (file)
@@ -1218,12 +1218,12 @@ static int processTbl(graph_t * g, htmltbl_t * tbl, htmlenv_t * env)
        while (cp) {
            cellp = cp->u.cp;
            cnt++;
-           cp = (pitem *) dtlink(cdict, (Dtlink_t *) cp);
+           cp = (pitem *)dtlink(cdict, cp);
        }
        if (rp->ruled) {
            addIntSet(is, r + 1);
        }
-       rp = (pitem *) dtlink(rows, (Dtlink_t *) rp);
+       rp = (pitem *)dtlink(rows, rp);
        r++;
     }
 
@@ -1246,9 +1246,9 @@ static int processTbl(graph_t * g, htmltbl_t * tbl, htmlenv_t * env)
            n_rows = MAX(r + cellp->rspan, n_rows);
            if (inIntSet(is, r + cellp->rspan))
                cellp->ruled |= HTML_HRULE;
-           cp = (pitem *) dtlink(cdict, (Dtlink_t *) cp);
+           cp = (pitem *)dtlink(cdict, cp);
        }
-       rp = (pitem *) dtlink(rows, (Dtlink_t *) rp);
+       rp = (pitem *)dtlink(rows, rp);
        r++;
     }
     tbl->rc = n_rows;
index f7cafb21c7684c7b1279700181789c614cf7056a..09f1862ad361714d2daf65fc1aee5bb8b18551ad 100644 (file)
@@ -122,7 +122,7 @@ point *pointsOf(PointSet * ps)
     point *pp = pts;
 
     for (p = (pair *) dtflatten(ps); p;
-        p = (pair *) dtlink(ps, (Dtlink_t *) p)) {
+        p = (pair *)dtlink(ps, p)) {
        *pp++ = p->id;
     }
 
index ba5e301d263b9593dcee0667a08ae2f3b1c31550..a6ad8caa162d83d30cbfe4af3009698ed163522c 100644 (file)
@@ -216,8 +216,7 @@ static graph_t *mkNConstraintG(graph_t * g, Dt_t * list,
     graph_t *cg = agopen("cg", Agstrictdirected, NULL);
     agbindrec(cg, "Agraphinfo_t", sizeof(Agraphinfo_t), true);  // graph custom data
 
-    for (p = (nitem *) dtflatten(list); p;
-        p = (nitem *) dtlink(list, (Dtlink_t *) p)) {
+    for (p = (nitem *)dtflatten(list); p; p = (nitem *)dtlink(list, p)) {
        n = agnode(cg, agnameof(p->np), 1);      /* FIX */
        agbindrec(n, "Agnodeinfo_t", sizeof(Agnodeinfo_t), true); //node custom data
        ND_alg(n) = p;
@@ -231,10 +230,8 @@ static graph_t *mkNConstraintG(graph_t * g, Dt_t * list,
            lastn = GD_nlist(cg) = n;
        }
     }
-    for (p = (nitem *) dtflatten(list); p;
-        p = (nitem *) dtlink(list, (Dtlink_t *) p)) {
-       for (nxp = (nitem *) dtlink(link, (Dtlink_t *) p); nxp;
-            nxp = (nitem *) dtlink(list, (Dtlink_t *) nxp)) {
+    for (p = (nitem *)dtflatten(list); p; p = (nitem *)dtlink(list, p)) {
+       for (nxp = (nitem *)dtlink(link, p); nxp; nxp = (nitem *)dtlink(list, nxp)) {
            e = NULL;
            if (intersect(p, nxp)) {
                double delta = dist(&p->bb, &nxp->bb);
@@ -250,8 +247,7 @@ static graph_t *mkNConstraintG(graph_t * g, Dt_t * list,
        }
     }
    
-    for (p = (nitem *) dtflatten(list); p;
-        p = (nitem *) dtlink(list, (Dtlink_t *) p)) {
+    for (p = (nitem *)dtflatten(list); p; p = (nitem *)dtlink(list, p)) {
        n = p->cnode;
        for (e = agfstout(cg,n); e; e = agnxtout(cg,e)) {
            elist_append(e, ND_out(n));
@@ -287,8 +283,7 @@ static graph_t *mkConstraintG(graph_t * g, Dt_t * list,
 
     /* count distinct nodes */
     cnt = 0;
-    for (p = (nitem *) dtflatten(list); p;
-        p = (nitem *) dtlink(list, (Dtlink_t *) p)) {
+    for (p = (nitem *)dtflatten(list); p; p = (nitem *)dtlink(list, p)) {
        if (oldval != p->val) {
            oldval = p->val;
            cnt++;
@@ -298,8 +293,7 @@ static graph_t *mkConstraintG(graph_t * g, Dt_t * list,
     /* construct basic chain to enforce left to right order */
     oldval = -INT_MAX;
     lcnt = 0;
-    for (p = (nitem *) dtflatten(list); p;
-        p = (nitem *) dtlink(list, (Dtlink_t *) p)) {
+    for (p = (nitem *)dtflatten(list); p; p = (nitem *)dtlink(list, p)) {
        if (oldval != p->val) {
            oldval = p->val;
            /* n = newNode (cg); */
@@ -339,28 +333,25 @@ static graph_t *mkConstraintG(graph_t * g, Dt_t * list,
      * FIX: Incremental algorithm to construct trans. reduction?
      */
     vg = agopen("vg", Agstrictdirected, NULL);
-    for (p = (nitem *) dtflatten(list); p;
-        p = (nitem *) dtlink(list, (Dtlink_t *) p)) {
+    for (p = (nitem *)dtflatten(list); p; p = (nitem *)dtlink(list, p)) {
        n = agnode(vg, agnameof(p->np), 1);  /* FIX */
        agbindrec(n, "Agnodeinfo_t", sizeof(Agnodeinfo_t), true);  //node custom data
        p->vnode = n;
        ND_alg(n) = p;
     }
     oldval = -INT_MAX;
-    for (p = (nitem *) dtflatten(list); p;
-        p = (nitem *) dtlink(list, (Dtlink_t *) p)) {
+    for (p = (nitem *)dtflatten(list); p; p = (nitem *)dtlink(list, p)) {
        if (oldval != p->val) { /* new pos: reset nxt */
            oldval = p->val;
-           for (nxt = (nitem *) dtlink(link, (Dtlink_t *) p); nxt;
-                nxt = (nitem *) dtlink(list, (Dtlink_t *) nxt)) {
+           for (nxt = (nitem *)dtlink(link, p); nxt;
+                nxt = (nitem *)dtlink(list, nxt)) {
                if (nxt->val != oldval)
                    break;
            }
            if (!nxt)
                break;
        }
-       for (nxp = nxt; nxp;
-            nxp = (nitem *) dtlink(list, (Dtlink_t *) nxp)) {
+       for (nxp = nxt; nxp; nxp = (nitem *)dtlink(list, nxp)) {
            if (intersect(p, nxp))
                agedge(vg, p->vnode, nxp->vnode, NULL, 1);
        }