From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Chris Browne <cbbrowne(at)acm(dot)org>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Lessons from commit fest |
Date: | 2008-04-17 09:16:17 |
Message-ID: | 20080417111617.45ab18f3@mha-laptop |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Bruce Momjian wrote:
> Bruce Momjian wrote:
> > Tom Lane wrote:
> > > Bruce Momjian <bruce(at)momjian(dot)us> writes:
> > > > I am reviewing the psql wrap patch and just used pgindent today
> > > > to clean it up. (pgindent did not add any extra spacing
> > > > changes.) Patch reviewers should probably be able to run
> > > > pgindent.
> > >
> > > Well, that means nobody in the world can review except you,
> > > because nobody else has ever reported success in duplicating your
> > > pgindent results. I know I haven't been able to.
> > >
> > > If you really believe the above then you need to try a bit harder
> > > to find a portable version of indent that we all can use.
> >
> > The source code of pgindent I use is on our ftp site. find_typedefs
> > should now work on Linux as well as BSD. Not sure what else would
> > be a problem.
>
> Also I can put up a web page where you can upload or email your C file
> and get a formatted version back.
IIRC, last time I tried it, the failure was because I couldn't get it
to build the proper typedefs. Any chance you could just put a regularly
updated typedefs file somewhere that I could wget down?
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | PFC | 2008-04-17 09:20:11 | Re: count(*) performance improvement ideas |
Previous Message | Brendan Jurd | 2008-04-17 08:48:42 | Re: printTable API (was: Show INHERIT in \du) |