v4-0001-Make-pgindent-add-a-space-between-comma-and-perio.patch
text/x-patch
Filename: v4-0001-Make-pgindent-add-a-space-between-comma-and-perio.patch
Type: text/x-patch
Part: 0
Patch
Same data as JSON:
GET /api/v1/attachments/:id/patch
the parsed metadata as JSON — format, series position, per-file stats; never the diff bytes.
API reference →
Format: format-patch
Series: patch v4-0001
Subject: Make pgindent add a space between comma and period
| File | + | − |
|---|---|---|
| src/tools/pg_bsd_indent/indent.c | 2 | 0 |
| src/tools/pg_bsd_indent/tests/struct.0 | 2 | 0 |
| src/tools/pg_bsd_indent/tests/struct.0.stdout | 2 | 0 |
From 86453d75405549f29d7f2811f15b58588de17c15 Mon Sep 17 00:00:00 2001
From: Andreas Karlsson <andreas@proxel.se>
Date: Tue, 17 Feb 2026 09:04:01 +0100
Subject: [PATCH v4 1/2] Make pgindent add a space between comma and period
Formatting of varidic functions and struct literals with named field
used to be ugly due to pgindent treaing period as always being binary
operator for struct access, but after comma it has a different meaning.
Also adds a new test case which tests the formatting of struct literals
with named fields.
---
src/tools/pg_bsd_indent/indent.c | 2 ++
src/tools/pg_bsd_indent/tests/struct.0 | 2 ++
src/tools/pg_bsd_indent/tests/struct.0.stdout | 2 ++
3 files changed, 6 insertions(+)
diff --git a/src/tools/pg_bsd_indent/indent.c b/src/tools/pg_bsd_indent/indent.c
index 421592db928..58fa04293d9 100644
--- a/src/tools/pg_bsd_indent/indent.c
+++ b/src/tools/pg_bsd_indent/indent.c
@@ -1013,6 +1013,8 @@ check_type:
case period: /* treat a period kind of like a binary
* operation */
+ if (ps.want_blank && ps.last_token == comma)
+ *e_code++ = ' ';
*e_code++ = '.'; /* move the period into line */
ps.want_blank = false; /* dont put a blank after a period */
break;
diff --git a/src/tools/pg_bsd_indent/tests/struct.0 b/src/tools/pg_bsd_indent/tests/struct.0
index 83142bfb197..6f65e461331 100644
--- a/src/tools/pg_bsd_indent/tests/struct.0
+++ b/src/tools/pg_bsd_indent/tests/struct.0
@@ -19,3 +19,5 @@ void u(struct x a) {
int b;
struct y c = (struct y *)&a;
}
+
+static struct foo f = { .a = 1,.b=2 };
diff --git a/src/tools/pg_bsd_indent/tests/struct.0.stdout b/src/tools/pg_bsd_indent/tests/struct.0.stdout
index 38613128654..69e321ca05f 100644
--- a/src/tools/pg_bsd_indent/tests/struct.0.stdout
+++ b/src/tools/pg_bsd_indent/tests/struct.0.stdout
@@ -21,3 +21,5 @@ u(struct x a)
int b;
struct y c = (struct y *)&a;
}
+
+static struct foo f = {.a = 1, .b = 2};
--
2.47.3