]> granicus.if.org Git - graphviz/commitdiff
gv2gml: remove 'outFile' parameter to 'emitInt'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 12 May 2022 04:42:35 +0000 (21:42 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 20 May 2022 05:00:24 +0000 (22:00 -0700)
The only calls to this function pass `outFile`, an already available global that
this parameter was shadowing. This removes a -Wshadow warning.

cmd/tools/gv2gml.c

index aef0ef489f99feb1d9aa55186059f4af295cb6d9..f4bb6710e778a2735aa6f847aaa0eaf0c2faeba8 100644 (file)
@@ -131,9 +131,7 @@ parseStyle (char* s)
     return flags;
 }
 
-static void 
-emitInt (char* name, int value, FILE* outFile, int ix)
-{
+static void emitInt(char *name, int value, int ix) {
     indent (ix, outFile);
     fprintf (outFile, "%s %d\n", name, value);
 }
@@ -372,10 +370,10 @@ static void  emitNodeAttrs(Agraph_t *G, Agnode_t *np, int ix) {
            emitReal("H", attrs.h, ix+1);
        }
        if (attrs.flags & INVIS) {
-           emitInt ("visible", 0, outFile, ix+1);
+           emitInt("visible", 0, ix+1);
        }
        if (attrs.flags & FILL) {
-           emitInt ("hasFill", 1, outFile, ix+1);
+           emitInt("hasFill", 1, ix+1);
        }
        if (attrs.type) {
            emitAttr("type", attrs.type, ix+1);
@@ -541,7 +539,7 @@ static void emitEdgeAttrs(Agraph_t *G, Agedge_t *ep, int ix) {
            emitSpline(attrs.pos, ix+1);
        }
        if (attrs.flags & INVIS) {
-           emitInt ("visible", 0, outFile, ix+1);
+           emitInt("visible", 0, ix+1);
        }
        if (attrs.fill) {
            emitAttr("fill", attrs.fill, ix+1);