]> granicus.if.org Git - graphviz/commitdiff
fix missing bracketing in a macro expansion
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 24 Oct 2020 16:53:57 +0000 (09:53 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 31 Oct 2020 01:46:51 +0000 (18:46 -0700)
None of the calls to this macro were passing in multiple tokens, so this issue
was latent. This refactor is for the safety of any future introduced calls to
EMPTY().

lib/cgraph/write.c

index e3d45cf3c9b686a59859defcf82af23eed051da0..153832ec8d28a36e38d92b61259ba1405f1dd6ab 100644 (file)
@@ -16,7 +16,7 @@
 #include <cgraph/cghdr.h>
 #include <cgraph/strcasecmp.h>
 
-#define EMPTY(s)               ((s == 0) || (s)[0] == '\0')
+#define EMPTY(s)               (((s) == 0) || (s)[0] == '\0')
 #define MAX(a,b)     ((a)>(b)?(a):(b))
 #define CHKRV(v)     {if ((v) == EOF) return EOF;}