]> granicus.if.org Git - graphviz/commitdiff
take a const pointer in agstrfree
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 4 Jul 2021 04:15:22 +0000 (21:15 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 18 Jul 2021 00:31:35 +0000 (17:31 -0700)
This function does not modify its string argument, so it is more accurate to
qualify it as const. Related to #634.

lib/cgraph/cgraph.h
lib/cgraph/refstr.c

index d1687e9bb912b11aafc32ac1e38ccced8ee0fede..536bf42099e98d4326e7bcd6a3360eeca8a76d4a 100644 (file)
@@ -306,7 +306,7 @@ CGRAPH_API char *agstrdup(Agraph_t *, const char *);
 CGRAPH_API char *agstrdup_html(Agraph_t *, const char *);
 CGRAPH_API int aghtmlstr(const char *);
 CGRAPH_API char *agstrbind(Agraph_t * g, const char *);
-CGRAPH_API int agstrfree(Agraph_t *, char *);
+CGRAPH_API int agstrfree(Agraph_t *, const char *);
 CGRAPH_API char *agcanon(char *, int);
 CGRAPH_API char *agstrcanon(char *, char *);
 CGRAPH_API char *agcanonStr(char *str);  /* manages its own buf */
index db14202bf782c203e7428238454e8b772c8ae68d..1f7e3974f022a1cc6be0406b5ab92cf94ff5bd8d 100644 (file)
@@ -148,7 +148,7 @@ char *agstrdup_html(Agraph_t * g, const char *s)
     return r->s;
 }
 
-int agstrfree(Agraph_t * g, char *s)
+int agstrfree(Agraph_t * g, const char *s)
 {
     refstr_t *r;
     Dict_t *strdict;