From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Sat, 24 Oct 2020 16:53:57 +0000 (-0700)
Subject: fix missing bracketing in a macro expansion
X-Git-Tag: 2.46.0~20^2^2~4^2~7
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4c71ebb37dc935cb3063bfc96d8ee6f96b579987;p=graphviz

fix missing bracketing in a macro expansion

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().
---

diff --git a/lib/cgraph/write.c b/lib/cgraph/write.c
index e3d45cf3c..153832ec8 100644
--- a/lib/cgraph/write.c
+++ b/lib/cgraph/write.c
@@ -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;}