On Sun, Apr 10, 2011 at 11:55 AM, Greg Stark <gsst...@mit.edu> wrote: > On Sun, Apr 10, 2011 at 4:42 PM, Bruce Momjian <br...@momjian.us> wrote: >> 27 hours later, done. I ran all the tests outlined in the pgindent >> README. >> > > What's with things like: > > -void _PG_init(void); > +void _PG_init(void); > > - Datum diff = DirectFunctionCall2(date_mi, > + Datum diff = DirectFunctionCall2(date_mi, > > const TimeADT *aa = (const TimeADT *) a; > const TimeADT *bb = (const TimeADT *) b; > - Interval *i; > + Interval *i; > > > Note that in the last one someone carefully made the variable names > line up and pgindent is changing the spacing to an arbitrary amount.
Well, it's the same arbitrary amount that we use throughout our code, presumably. I am not sure whether pgident is the best tool for the job, but at least it makes the code relatively consistent throughout, which is mostly a good thing. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
