Fix compiler warning for MSVC in libpq_pipeline.c

david-rowley · david-rowley · commit 3b82d990ab78 · 2021-04-07T09:51:33.000+12:00
DEBUG was already defined by the MSVC toolchain for "Debug" builds. On these systems the unconditional #define DEBUG was causing a 'DEBUG': macro redefinition warning. Here we rename DEBUG to DEBUG_OUPUT and also get rid of the #define which defined this constant. This appears to have been left in the code by mistake. Discussion: https://postgr.es/m/CAApHDvqTTgDm38s4HRj03nhzhzQ1oMOj-RXFUB1pE6Bj07jyuQ@mail.gmail.com
diff --git a/src/test/modules/libpq_pipeline/libpq_pipeline.c b/src/test/modules/libpq_pipeline/libpq_pipeline.c
@@ -35,8 +35,7 @@ const char *const progname = "libpq_pipeline";
 char	   *tracefile = NULL;	/* path to PQtrace() file */
 
 
-#define DEBUG
-#ifdef DEBUG
+#ifdef DEBUG_OUTPUT
 #define	pg_debug(...)  do { fprintf(stderr, __VA_ARGS__); } while (0)
 #else
 #define pg_debug(...)
