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 4a91d7fb28232df4f235613b20d3b4ab18f72f29..62d3683be5e7de509762cb650c25e08e68332706 100644 (file)
@@ -3271,7 +3271,7 @@ typedef struct ConstraintsSetStmt
  */
 
 /* Reindex options */
-#define REINDEXOPT_VERBOSE 1 << 0  /* print progress info */
+#define REINDEXOPT_VERBOSE (1 << 0)    /* print progress info */
 
 typedef enum ReindexObjectType
 {