From: Tom Lane <tgl@sss.pgh.pa.us>
To: Michael Paquier <michael@paquier.xyz>
cc: Justin Pryzby <pryzby@telsasoft.com>,
        Amit Kapila <amit.kapila16@gmail.com>,
        pgsql-hackers@lists.postgresql.org
Subject: Re: typos
In-reply-to: <Y+SIhEEVYkZf9P2f@paquier.xyz>
References: <20221230231257.GI1153@telsasoft.com>
 <Y7PZHbDrRMx5O9Hp@paquier.xyz> <20230103213922.GR1153@telsasoft.com>
 <20230110045722.GD9837@telsasoft.com>
 <CAA4eK1+OPcKAQneTu=2byxiRS07-NVmN0QwUFEHHw2N5osKoWw@mail.gmail.com>
 <Y70YN3RpQNBUJDeS@paquier.xyz>
 <CAA4eK1KgRPZr-2_LxAgeNCxmwnvN2nGVazoNNV1B_4SY9f=uSQ@mail.gmail.com>
 <Y75Wgpu0GLU3ArGv@paquier.xyz> <20230208155644.GM1653@telsasoft.com>
 <Y+SIhEEVYkZf9P2f@paquier.xyz>
Comments: In-reply-to Michael Paquier <michael@paquier.xyz>
	message dated "Thu, 09 Feb 2023 14:45:40 +0900"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <3841394.1675924608.1@sss.pgh.pa.us>
Date: Thu, 09 Feb 2023 01:36:48 -0500
Message-ID: <3841395.1675924608@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/3841395.1675924608%40sss.pgh.pa.us>
Precedence: bulk

Michael Paquier <michael@paquier.xyz> writes:
> On Wed, Feb 08, 2023 at 09:56:44AM -0600, Justin Pryzby wrote:
>> Some more accumulated/new typos.

> 0005 can indeed fix a lot of confusion around the spaces after an
> "else if" block.  Is that something that could be automated with the
> indentation, though?  Same remark for 0009 and 0010.

I see your point about 0005, but I've never seen pgindent remove
vertical whitespace once it's been added.  Not sure what it'd take
to teach it to do so, or whether we'd like the results.

I'd reject 0009 and 0010 altogether --- they don't add any readability
that's worth the potential increase in back-patch problems.

			regards, tom lane