postgres.git / summary / log / commit / refs

commit    0fbcb9da181452e236b5aca6bf00c20ebcd7ec6b
Author:   Nathan Bossart <nathan@postgresql.org>
Date:     Mon Dec 09 22:26:23 2024 +0000
First released in: 13.19

    Fix unused-but-set-variable compiler warning in reorderbuffer.c.
    
    On v13, this variable is only used for an assertion, so adding
    PG_USED_FOR_ASSERTS_ONLY is sufficient to suppress this warning on
    builds with assertions disabled.  Older versions are unsupported,
    and newer versions use the variable for more than the assertion, so
    this patch only needs to be applied to REL_13_STABLE.
    
    Reviewed-by: Tom Lane
    Discussion: https://postgr.es/m/Z1dCFnzrP24O8WNR%40nathan


 src/backend/replication/logical/reorderbuffer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.c
index c7f8fa6216c..56c25e3a6da 100644
--- a/src/backend/replication/logical/reorderbuffer.c
+++ b/src/backend/replication/logical/reorderbuffer.c
@@ -2468,7 +2468,7 @@ ReorderBufferSerializeTXN(ReorderBuffer *rb, ReorderBufferTXN *txn)
 	dlist_mutable_iter change_i;
 	int			fd = -1;
 	XLogSegNo	curOpenSegNo = 0;
-	Size		spilled = 0;
+	Size		spilled PG_USED_FOR_ASSERTS_ONLY = 0;
 
 	elog(DEBUG2, "spill %u changes in XID %u to disk",
 		 (uint32) txn->nentries_mem, txn->xid);


[parent: a51ec088dbe5]