]> granicus.if.org Git - graphviz/commitdiff
gvpack: take const values in various functions that call 'agattr'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 14 Apr 2022 00:19:15 +0000 (17:19 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Mon, 25 Apr 2022 02:12:48 +0000 (19:12 -0700)
This pre-emptively squashes several upcoming warnings when converting this file
to C++ which otherwise fail the CMake build:

  cmd/tools/gvpack.cpp:181:22: error: ISO C++11 does not allow conversion from
    string literal to 'char *' [-Werror,-Wwritable-strings]

Gitlab: #2154

cmd/tools/gvpack.c

index ecb351813a14bf0e8583fc9cfd12e8607a8fa9ca..c7d8241efd25824b99aa1a790a07f4c4b847370a 100644 (file)
@@ -155,18 +155,15 @@ static int setUInt(unsigned int *v, char *arg)
     return 0;
 }
 
-static Agsym_t *agraphattr(Agraph_t *g, char *name, char *value)
-{
+static Agsym_t *agraphattr(Agraph_t *g, char *name, const char *value) {
     return agattr(g, AGRAPH, name, value);
 }
 
-static Agsym_t *agnodeattr(Agraph_t *g, char *name, char *value)
-{
+static Agsym_t *agnodeattr(Agraph_t *g, char *name, const char *value) {
     return agattr(g, AGNODE, name, value);
 }
 
-static Agsym_t *agedgeattr(Agraph_t *g, char *name, char *value)
-{
+static Agsym_t *agedgeattr(Agraph_t *g, char *name, const char *value) {
     return agattr(g, AGEDGE, name, value);
 }
 
@@ -463,7 +460,7 @@ static void fillDict(Dt_t * newdict, Agraph_t* g, int kind)
  */
 static void
 fillGraph(Agraph_t * g, Dt_t * d,
-         Agsym_t * (*setf) (Agraph_t *, char *, char *), int cnt)
+         Agsym_t * (*setf) (Agraph_t *, char *, const char *), int cnt)
 {
     attr_t *av;
     for (av = (attr_t *) dtflatten(d); av;