Split PruneFreezeParams initializers to one field per line

melanieplageman · melanieplageman · commit 1e14edcea5e1 · 2025-11-20T17:36:40.000-05:00
This conforms more closely with the style of other struct initializers in the code base. Initializing multiple fields on a single line is unpopular in part because pgindent won't permit a space after the comma before the next field's period. Author: Melanie Plageman <melanieplageman@gmail.com> Reported-by: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> Discussion: https://postgr.es/m/87see87fnq.fsf%40wibble.ilmari.org
diff --git a/src/backend/access/heap/pruneheap.c b/src/backend/access/heap/pruneheap.c
@@ -269,9 +269,13 @@ heap_page_prune_opt(Relation relation, Buffer buffer)
 			 * cannot safely determine that during on-access pruning with the
 			 * current implementation.
 			 */
-			PruneFreezeParams params = {.relation = relation,.buffer = buffer,
-				.reason = PRUNE_ON_ACCESS,.options = 0,
-				.vistest = vistest,.cutoffs = NULL
+			PruneFreezeParams params = {
+				.relation = relation,
+				.buffer = buffer,
+				.reason = PRUNE_ON_ACCESS,
+				.options = 0,
+				.vistest = vistest,
+				.cutoffs = NULL,
 			};
 
 			heap_page_prune_and_freeze(&params, &presult, &dummy_off_loc,
diff --git a/src/backend/access/heap/vacuumlazy.c b/src/backend/access/heap/vacuumlazy.c
@@ -1965,9 +1965,13 @@ lazy_scan_prune(LVRelState *vacrel,
 {
 	Relation	rel = vacrel->rel;
 	PruneFreezeResult presult;
-	PruneFreezeParams params = {.relation = rel,.buffer = buf,
-		.reason = PRUNE_VACUUM_SCAN,.options = HEAP_PAGE_PRUNE_FREEZE,
-		.vistest = vacrel->vistest,.cutoffs = &vacrel->cutoffs
+	PruneFreezeParams params = {
+		.relation = rel,
+		.buffer = buf,
+		.reason = PRUNE_VACUUM_SCAN,
+		.options = HEAP_PAGE_PRUNE_FREEZE,
+		.vistest = vacrel->vistest,
+		.cutoffs = &vacrel->cutoffs,
 	};
 
 	Assert(BufferGetBlockNumber(buf) == blkno);
