On Sun, Apr 10, 2011 at 11:55 AM, Greg Stark <gsstark@mit.edu> wrote:
> On Sun, Apr 10, 2011 at 4:42 PM, Bruce Momjian <bruce@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