From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Piotr Stefaniak <postgres(at)piotr-stefaniak(dot)me> |
Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pgindent (was Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run.) |
Date: | 2017-06-13 21:18:01 |
Message-ID: | 31422.1497388681@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Piotr Stefaniak <postgres(at)piotr-stefaniak(dot)me> writes:
> On 2017-06-13 22:23, Tom Lane wrote:
>> I could not find any places where reverting this change made the
>> results worse, so I'm unclear on why you made it.
> I must admit I'm a bit confused about why it's not fixed yet, but I'll
> have to analyze that later this week. But the idea was to convince
> indent that the following is not a declaration and therefore it
> shouldn't be formatted as such:
> typedef void (*voidptr) (int *);
Hm. But that's just a function pointer typedef, and we like the
formatting we're getting for those from this new version --- with or
without that change. What do you think needs to be done differently?
I note btw that this is not the first time we've discussed that
particular bit of code in this thread. I proposed a couple of
different possible changes to it before ...
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-06-14 03:46:49 | pgsql: Disallow set-returning functions inside CASE or COALESCE. |
Previous Message | Piotr Stefaniak | 2017-06-13 21:13:12 | Re: pgindent (was Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run.) |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-06-13 21:28:40 | Re: A bug in mapping attributes in ATExecAttachPartition() |
Previous Message | Jeevan Ladhe | 2017-06-13 21:13:18 | Re: fix possible optimizations in ATExecAttachPartition() |