]> granicus.if.org Git - graphviz/commitdiff
core plugin: use const arrays for some strings
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 13 Aug 2022 15:34:45 +0000 (08:34 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 19 Aug 2022 00:46:06 +0000 (17:46 -0700)
This makes very little difference to the compiler, as it can see all uses of
these strings. But this makes it clearer to readers that these variables are not
modified. Using an array instead of a pointer slightly increases flexibility,
allowing a useful application of `sizeof` to these variables should we ever want
to do that.

plugin/core/gvrender_core_svg.c

index 4b2a6340181157b0e050d7cb30293f21b9ad7082..f86da09eb11998c7006658a1e7f555377e0220d2 100644 (file)
 typedef enum { FORMAT_SVG, FORMAT_SVGZ, } format_type;
 
 /* SVG dash array */
-static char *sdasharray = "5,2";
+static const char sdasharray[] = "5,2";
 /* SVG dot array */
-static char *sdotarray = "1,5";
+static const char sdotarray[] = "1,5";
 
-static char *transparent = "transparent";
-static char *none = "none";
-static char *black = "black";
+static const char transparent[] = "transparent";
+static const char none[] = "none";
+static const char black[] = "black";
 
 static void svg_bzptarray(GVJ_t * job, pointf * A, int n)
 {