Fix under-parenthesized macro definitions
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Fri, 13 Sep 2019 19:26:55 +0000 (16:26 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Fri, 13 Sep 2019 19:26:55 +0000 (16:26 -0300)
Lack of parens in the definitions could cause a statement using these
macros to have unexpected semantics.  In current code no bug is
apparent, but best to fix the definitions to avoid problems down the
line.

Reported-by: Tom Lane
Discussion: https://postgr.es/m/19795.1568400476@sss.pgh.pa.us

src/include/nodes/parsenodes.h

index bb931b2cacf5fed64f95bc5b3ffd33799a595c3e..d93a79a55470055b1c4b66117774ccace4897119 100644 (file)
@@ -3312,8 +3312,8 @@ typedef struct ConstraintsSetStmt
  */
 
 /* Reindex options */
-#define REINDEXOPT_VERBOSE 1 << 0  /* print progress info */
-#define REINDEXOPT_REPORT_PROGRESS 1 << 1  /* report pgstat progress */
+#define REINDEXOPT_VERBOSE (1 << 0)    /* print progress info */
+#define REINDEXOPT_REPORT_PROGRESS (1 << 1)    /* report pgstat progress */
 
 typedef enum ReindexObjectType
 {