On Mon, Oct 26, 2020 at 11:39:33AM -0300, Alvaro Herrera wrote: > Please remember that in the macro definition, the arguments should be > enclosed in parens. No bug here at present, but it seems better to be > cautious.
Indeed, I can see similar changes in the history of the tree. Do you think that doing something like the attached is sufficient? -- Michael
diff --git a/src/include/storage/bufpage.h b/src/include/storage/bufpage.h
index d0a52f8e08..82c974dfb3 100644
--- a/src/include/storage/bufpage.h
+++ b/src/include/storage/bufpage.h
@@ -420,7 +420,7 @@ do { \
#define PageIsVerified(page, blkno) \
PageIsVerifiedExtended(page, blkno, \
- PIV_LOG_WARNING | PIV_REPORT_STAT)
+ (PIV_LOG_WARNING | PIV_REPORT_STAT))
/*
* Check that BLCKSZ is a multiple of sizeof(size_t). In
signature.asc
Description: PGP signature
