On Sun, May 24, 2015 at 10:44:10AM -0400, Tom Lane wrote: > Andrew Dunstan <and...@dunslane.net> writes: > > On 05/23/2015 11:37 PM, Tom Lane wrote: > >> No, pgindent has *always* been wonky about lines that contain a typedef > >> name but are not variable declarations. > > > Well, that sounds like something we should try to patch, doesn't it? > > (No, I'm not volunteering.) > > In the past, the main argument against changing pgindent has been that > it would cause reformatting of settled code. For example, when Bruce > twiddled its right margin limit for comments around 8.1, that caused > literally years worth of back-patching pain. If we can get to an > agreement on re-indenting back branches at the same time as master, > then this problem would go away, and I for one would get a lot more > enthusiastic about trying to improve pgindent rather than just working > around its oddities. > > Not that I'm volunteering either right now.
That is something we will need to fix in the BSD indent code. -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + Everyone has their own god. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
