Re: pgindent weirdness

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: pgindent weirdness
Дата
Msg-id 28016.1303315456@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: pgindent weirdness  (Bruce Momjian <bruce@momjian.us>)
Ответы Re: pgindent weirdness  (Bruce Momjian <bruce@momjian.us>)
Re: pgindent weirdness  (Bruce Momjian <bruce@momjian.us>)
Список 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 по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: REINDEX vs broken HOT chains, redux
Следующее
От: David Fetter
Дата:
Сообщение: Re: pgbench \for or similar loop