]> granicus.if.org Git - graphviz/commitdiff
graphml2gv: remove 'nameMap'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 22 Oct 2022 20:43:38 +0000 (13:43 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 2 Nov 2022 03:26:06 +0000 (20:26 -0700)
This dictionary was only ever inserted to by code that was removed in the
previous commit as unreachable. So we know it is always empty and lookups into
it would have always returned null.

Gitlab: #2300

cmd/tools/graphml2gv.c

index 60176356812c1d524fbe7cd74f4173abdd0ca45c..90f265561a2e1ad47081a44ba4e5854b410a41e5 100644 (file)
@@ -89,7 +89,6 @@ typedef struct {
     gv_stack_t elements;
     int closedElementType;
     int edgeinverted;
-    Dt_t *nameMap;
 } userdata_t;
 
 static Agraph_t *root;         /* root graph */
@@ -100,34 +99,6 @@ static Agedge_t *E;         /* Set if Current_class == TAG_EDGE */
 
 static gv_stack_t Gstack;
 
-typedef struct {
-    Dtlink_t link;
-    char *name;
-    char *unique_name;
-} namev_t;
-
-static namev_t *make_nitem(Dt_t * d, namev_t * objp, Dtdisc_t * disc)
-{
-    namev_t *np = gv_alloc(sizeof(*np));
-    np->name = objp->name;
-    np->unique_name = 0;
-    return np;
-}
-
-static void free_nitem(Dt_t * d, namev_t * np, Dtdisc_t * disc)
-{
-    free(np->unique_name);
-    free(np);
-}
-
-static Dtdisc_t nameDisc = {
-    .key = offsetof(namev_t, name),
-    .size = -1,
-    .link = offsetof(namev_t, link),
-    .makef = (Dtmake_f)make_nitem,
-    .freef = (Dtfree_f)free_nitem,
-};
-
 static userdata_t *genUserdata(char* dfltname)
 {
     userdata_t *user = gv_alloc(sizeof(*user));
@@ -135,25 +106,15 @@ static userdata_t *genUserdata(char* dfltname)
     user->closedElementType = TAG_NONE;
     user->edgeinverted = FALSE;
     user->gname = dfltname;
-    user->nameMap = dtopen(&nameDisc, Dtoset);
     return user;
 }
 
 static void freeUserdata(userdata_t * ud)
 {
-    dtclose(ud->nameMap);
     freeString(&ud->elements);
     free(ud);
 }
 
-static char *mapLookup(Dt_t *nm, const char *name) {
-    namev_t *objp = dtmatch(nm, name);
-    if (objp)
-       return objp->unique_name;
-    else
-       return 0;
-}
-
 static int isAnonGraph(const char *name) {
     if (*name++ != '%')
        return 0;
@@ -343,14 +304,6 @@ startElementHandler(void *userData, const char *name, const char **atts)
        if (pos > 0)
            head = atts[pos];
 
-       tname = mapLookup(ud->nameMap, tail);
-       if (tname)
-           tail = tname;
-
-       tname = mapLookup(ud->nameMap, head);
-       if (tname)
-           head = tname;
-
         if (G == 0)
             fprintf(stderr,"edge source %s target %s outside graph, ignored\n",tail,head);
         else {