From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Chuck McDevitt <cmcdevitt(at)greenplum(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Neil Conway <neilc(at)samurai(dot)com>, Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Another pgindent gripe |
Date: | 2005-11-10 18:33:08 |
Message-ID: | 200511101833.jAAIX8201489@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
FYI, I am looking into all the reports of pgindent failures and will
have those fixed and a sample re-run diff posted in a few days.
---------------------------------------------------------------------------
Chuck McDevitt wrote:
> Pgindent adds spaces after the stars if it doesn't recognize the thing
> before the star as a typedef... Could it be that somehow the list of
> typedefs included in pgindent got corrupted?
>
> > -----Original Message-----
> > From: pgsql-hackers-owner(at)postgresql(dot)org [mailto:pgsql-hackers-
> > owner(at)postgresql(dot)org] On Behalf Of Tom Lane
> > Sent: Monday, November 07, 2005 8:19 AM
> > To: Neil Conway
> > Cc: Bruce Momjian; Hackers
> > Subject: Re: [HACKERS] Another pgindent gripe
> >
> > Neil Conway <neilc(at)samurai(dot)com> writes:
> > > On a related note, most of these changes are completely bogus:
> >
> > >
> >
> http://developer.postgresql.org/cvsweb.cgi/pgsql/src/pl/plpgsql/src/pl_e
> xe
> > c.c.diff?r1=1.152;r2=1.153
> >
> > Oy vey! Why did it insert spaces after the stars in all those
> function
> > declarations? That's certainly not in conformance with project style
> > ... and I don't see it having happened elsewhere.
> >
> > Seems like pgindent has suffered some significant regressions since
> the
> > 8.0 run. I thought it had not been changed much at all, but evidently
> > that's wrong.
> >
> > regards, tom lane
> >
> > ---------------------------(end of
> broadcast)---------------------------
> > TIP 5: don't forget to increase your free space map settings
>
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings
>
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
From | Date | Subject | |
---|---|---|---|
Next Message | Darcy Buskermolen | 2005-11-10 19:00:42 | Re: Obtaining a source tree from CVS |
Previous Message | Neil Conway | 2005-11-10 18:32:50 | Re: Obtaining a source tree from CVS |