Fix under-parenthesized macro definitions
authorAlvaro Herrera
Fri, 13 Sep 2019 19:26:55 +0000 (16:26 -0300)
committerAlvaro Herrera
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 c9c72ab94a6d9e3c654a2a91ec6fea24ba2ff422..d6b943c898c72900a6bde8105ce4003d008f8285 100644 (file)
@@ -3300,8 +3300,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
 {