Re: pgindent - Mailing list pgsql-hackers

From Robert Haas
Subject Re: pgindent
Date
Msg-id CA+Tgmob1DKeOb1dk3RjaJCHx67cvrMs6dvA8hVtT6w9BipNZWg@mail.gmail.com
Whole thread Raw
In response to Re: pgindent  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pgindent  (Bruce Momjian <bruce@momjian.us>)
Re: pgindent  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Wed, Apr 27, 2016 at 2:23 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
>> I think it's about time for us to run pgindent.  I did a trial run
>> today of pgindent today and came up with the attached patch for
>> typedefs.list, which I'd like to commit more or less immediately,
>> barring objections.
>
> Um, we normally take the buildfarm's list of typedefs, not anything
> manually created.

Well, we can still do that, but I don't see much advantage in it.  It
just churns the file to the extent that manual review of the changes
is impossible, and then when pgindent does the wrong thing it only
gets reported after the fact.  How is that better than making sure
that the contents of the file are such as to actually produce good
output from pgindent?

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: [COMMITTERS] pgsql: Modify the isolation tester so that multiple sessions can wait.
Next
From: Alvaro Herrera
Date:
Subject: Re: [COMMITTERS] pgsql: Modify the isolation tester so that multiple sessions can wait.