Re: pgindent timing (was Re: [COMMITTERS] pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY)

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: pgindent timing (was Re: [COMMITTERS] pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY)
Дата
Msg-id 20090812132629.GB5721@alvh.no-ip.org
обсуждение исходный текст
Ответ на Re: pgindent timing (was Re: [COMMITTERS] pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY)  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: pgindent timing (was Re: [COMMITTERS] pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY)  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Tom Lane escribió:
> Alvaro Herrera <alvherre@commandprompt.com> writes:
> > The reason this is like this is that the indent binary modifies the
> > prototype exactly like the function definition, and then the awk program
> > that's used in the pipeline "pulls up" the second line:
> 
> > #  Move prototype names to the same line as return type.  Useful for ctags. 
> > #  Indent should do this, but it does not.  It formats prototypes just
> > #  like real functions.
> 
> > In this day and age there's probably no reason to do this.
> 
> Um, sorry, no reason to do which?

No reason not to leave prototypes alone in the AWK code.  Isn't the
style emitted by indent good enough already?  The comment that ctags
needs it is probably outdated (I know my ctags, the Exuberant one,
doesn't).

-- 
Alvaro Herrera                                http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Joshua Tolley
Дата:
Сообщение: Re: "Hot standby"?
Следующее
От: Robert Haas
Дата:
Сообщение: Re: pgindent timing (was Re: [COMMITTERS] pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY)