Re: [PATCH] pgindent truncates last line of files missing a trailing newline
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Akshay Joshi <akshay.joshi@enterprisedb.com>,
Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>,
pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2026-03-27T19:28:16Z
Lists: pgsql-hackers
Andrew Dunstan <andrew@dunslane.net> writes: > On 2026-03-26 Th 12:29 PM, Tom Lane wrote: >> I'm not sure I want to expend the brain cells to figure out whether >> this is a correct/complete patch. If pg_bsd_indent were less of an >> undercommented spaghetti-code nightmare, maybe fixing it here would >> be reasonable. But as things stand, why don't we just fix this in >> the perl wrapper, as attached? > Well, I thought we were trying to reduce the fixups we did in pgindent. Yeah, other things being equal I too would prefer to fix it within pg_bsd_indent. But other things are not equal: the complexity and reviewability of the proposed v2 patch are not comparable to a one-liner fix in the wrapper. > However, I take your point about the ugly nature of the pg_bsd_indent > code. So I'm ok with this fix, which is quite straightforward. I'll make it so. regards, tom lane
Commits
-
pgindent: ensure all C files end with a newline.
- 79ac82125ef6 19 (unreleased) landed