Re: pgindent weirdness - Mailing list pgsql-hackers

From Tom Lane
Subject Re: pgindent weirdness
Date
Msg-id 28016.1303315456@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgindent weirdness  (Bruce Momjian <bruce@momjian.us>)
Responses Re: pgindent weirdness
Re: pgindent weirdness
List pgsql-hackers
Bruce Momjian <bruce@momjian.us> writes:
> Tom Lane wrote:
>> Now having said that, there seems to be a pgindent bug here too, in that
>> it's throwing a space into
>> 
>> Buffer
>> RelationGetBufferForTuple(Relation relation, Size len,
>> Buffer otherBuffer, int options,
>> struct BulkInsertStateData * bistate)
>> 
>> Whether BulkInsertStateData is flagged as a typedef or not, surely it
>> ought to understand that "struct BulkInsertStateData" is a type name.

> Uh, I think we have this listed as a known bug at the top of the
> pgindent script:

Hm.  I guess the third observation is that in the current state of the
code, there's no very good reason to be using "struct" in
RelationGetBufferForTuple's prototype anyway, since the typedef is
declared right above it.  Maybe we should just change that and not
risk fooling with pgindent.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: REINDEX vs broken HOT chains, redux
Next
From: David Fetter
Date:
Subject: Re: pgbench \for or similar loop