Re: pgindent issue with EXEC_BACKEND-only typedefs

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgindent issue with EXEC_BACKEND-only typedefs
Date: 2007-12-21 14:25:43
Message-ID: 200712211425.lBLEPhO23830@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Alvaro Herrera wrote:
> I don't know how to make it output the symbol names like it seems to do
> for you.
>
> Having the typedef list in the script itself seems like a barrier for
> other people to contribute to this thing. I wonder if that can be
> changed so that the typedef is on a separate list.

Done and committed.

> (Why are we still distributing pgjindent anyway?)

Removed.

> I am also wondering if current GNU indent is now better suited to the
> task. Perhaps the bugs that it had on earlier versions have since been
> fixed? I remember checking the source code size a couple of years ago
> and it had grown by an order of magnitude or something like that.

Yes, we should give that another try. I will keep this as an 8.4 item.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2007-12-21 14:29:04 Re: pgindent issue with EXEC_BACKEND-only typedefs
Previous Message Merlin Moncure 2007-12-21 14:13:32 Re: function body actors (was: viewing source code)