Re: pgindent run? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: pgindent run?
Date
Msg-id 10248.1511905092@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgindent run?  (Thomas Munro <thomas.munro@enterprisedb.com>)
Responses Re: pgindent run?
List pgsql-hackers
Thomas Munro <thomas.munro@enterprisedb.com> writes:
> On Wed, Nov 29, 2017 at 9:47 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> I think that'd be taking it too far, especially given that the dependency
>> on a typedefs list means that the git hook might have a different idea
>> of what's correctly indented than the committer does.  It'd be very hard
>> to debug such discrepancies and figure out what would satisfy the hook.

> Is there no way to get reasonable indentation that doesn't depend on
> that typedefs list?

Perhaps, but not with the tool we've got.

It's well known that C is unparseable without knowing which identifiers
are typedefs, so it doesn't exactly surprise me that it might not be
sanely indentable without knowing that.  But I've not thought hard about
it, nor looked for alternate tools.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Feike Steenbergen
Date:
Subject: Skip index cleanup if autovacuum did not do any work
Next
From: Robert Haas
Date:
Subject: Re: [HACKERS] PG10 partitioning - odd behavior/possible bug