Re: Harmonizing pg_bsd_indent parameter names - Mailing list pgsql-hackers

From Nathan Bossart
Subject Re: Harmonizing pg_bsd_indent parameter names
Date
Msg-id ZmoUBm95xoizU08L@nathan
Whole thread Raw
In response to Harmonizing pg_bsd_indent parameter names  (Peter Geoghegan <pg@bowt.ie>)
Responses Re: Harmonizing pg_bsd_indent parameter names
List pgsql-hackers
On Wed, Jun 12, 2024 at 05:14:44PM -0400, Peter Geoghegan wrote:
> I have been putting this off because I wasn't sure that the policy
> should be the same for pg_bsd_indent. Is there any reason to think
> that this will create more work down the line? It seems like it might,
> due to some kind of need to keep pg_bsd_indent's consistent with
> upstream BSD indent. But probably not. The patch is pretty small, in
> any case.

I would be surprised if this 2-line patch created anything approaching a
significant amount of work down the road.  FWIW commit 10d34fe already
changed one line in indent.c.

> I'd like to push this patch now. It's generally easier to be strict
> about these inconsistencies. My clang-tidy workflow doesn't
> automatically filter out various special cases requiring subjective
> judgement, so leaving stuff like this unfixed creates more work down
> the road.

Are these the only remaining inconsistencies?

-- 
nathan



pgsql-hackers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Harmonizing pg_bsd_indent parameter names
Next
From: Tom Lane
Date:
Subject: Re: Harmonizing pg_bsd_indent parameter names