]> granicus.if.org Git - graphviz/commitdiff
smyrna gvpr_select: use cgraph wrapper for allocation
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 25 Sep 2022 17:43:10 +0000 (10:43 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 25 Sep 2022 20:25:40 +0000 (13:25 -0700)
The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

The commit series ending in 78a0004fd0d1350f2c34e3a770d93032882706f7 previously
removed all uses of the lib/common/memory.h wrappers. But reverting
656a15f855a31acc6eeb162284b89ad3367253b9 brought another usage back in.

cmd/smyrna/gui/frmobjectui.c

index cc09dfeef23ff7a244c1475dcd39262892472ad9..a7acf4fdcbfd36bdc4df6b5cfd069f1330e4e6dc 100644 (file)
@@ -25,7 +25,6 @@
 #include <cgraph/alloc.h>
 #include <cgraph/strcasecmp.h>
 #include <cgraph/strview.h>
-#include <common/memory.h>
 #include <string.h>
 
 static int sel_node;
@@ -722,7 +721,7 @@ static void gvpr_select(char *attrname, char *regex_str, int objType) {
     argc = 1;
     if (*bf2 != '\0')
        argc++;
-    argv = N_NEW(argc + 1, char *);
+    argv = gv_calloc(argc + 1, sizeof(char*));
     size_t j = 0;
     argv[j++] = "smyrna";
     argv[j++] = bf2;