Re: Emacs vs pg_indent's weird indentation for function declarations
Piotr Stefaniak <postgres@piotr-stefaniak.me>
From: Piotr Stefaniak <postgres@piotr-stefaniak.me>
To: Tom Lane <tgl@sss.pgh.pa.us>, Thomas Munro <thomas.munro@gmail.com>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>, Michael Paquier <michael@paquier.xyz>
Date: 2019-05-19T17:25:05Z
Lists: pgsql-hackers
On 17/05/2019 16.48, Tom Lane wrote: > I wrote: >>>> A small problem with the "rejiggering" is that it now makes the wrong >>>> choice for K&R-style function definitions, causing them to be weirdly >>>> indented. For our purposes, that's a non-problem so I'm not excited >>>> about trying to make it smart enough to recognize those. We do have >>>> a couple of amazingly old and crufty K&R-style functions in src/port/, >>>> though, so probably we'd wish to fix those. > It doesn't really seem practical to me to make the lookahead function > smart enough to tell the difference between attributes and K&R-style > parameter declarations. What I'm thinking of doing to have an > upstreamable patch is to invent a new switch, perhaps '-kr'/'-nkr', > to indicate whether the user is more worried about K&R function > declarations than she is about function attributes. I think it's safe to assume that upstream can drop support for K&R-style parameters altogether.
Commits
-
Phase 2 pgindent run for v12.
- 8255c7a5eeba 12.0 landed
-
ANSI-ify a few straggler K&R-style function definitions.
- da71f98efba9 12.0 landed