Re: run pgindent on a regular basis / scripted manner

On 2023-02-04 Sa 09:20, Andrew Dunstan wrote:
>
>
> On 2023-02-04 Sa 06:34, Andres Freund wrote:
>>
>> ISTM that we're closer to being able to enforce pgindent than
>> perltidy. At the same time, I think the issue of C code in HEAD not
>> being indented is more pressing - IME it's much more common to have to
>> touch a lot of C code than to have to touch a lot fo perl files. So
>> perhaps we should just start with being more stringent with C code, and
>> once we made perltidy less noisy, add that?
>>
>
> Sure, we don't have to tie them together.
>
> I'm currently experimenting with settings on the buildfarm code,
> trying to get it both stable and looking nice. Then I'll try on the
> Postgres core code and post some results.
>

So here's a diff made from running perltidy v20221112 with the
additional setting --valign-exclusion-list=", = => || && if unless"

Essentially this abandons those bits of vertical alignment that tend to
catch us out when additions are made to the code.

I think this will make the code much more maintainable and result in
much less perltidy churn. It would also mean that it's far more likely
that what we would naturally code can be undisturbed by perltidy.

cheers

andrew

--
Andrew Dunstan
EDB:https://www.enterprisedb.com