Re: pgindent run?

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: pgindent run?
Дата
Msg-id 200103230451.XAA07904@candle.pha.pa.us
обсуждение исходный текст
Ответ на Re: pgindent run?  (Hiroshi Inoue <Inoue@tpf.co.jp>)
Список pgsql-hackers
> Bruce Momjian wrote:
> > 
> > You don't notice the value of pgindent until you have some code that
> > hasn't been run through it.  For example, ODBC was not run through until
> > this release, and I had a terrible time trying to understand the code
> > because it didn't _look_ like the rest of the code.  Now that pgindent
> > is run, it looks more normal, and I am sure that will encourage more
> > people to get in and make changes.
> > 
> 
> I see now the following comment in interfaces/odbc/statement.c.
> Though it's mine(probably), it's hard for me to read.
> Please tell me how to prevent pgindent from changing
> comments.
> 
>  /*
>   * Basically we don't have to begin a transaction in autocommit mode
>   * because Postgres backend runs in autocomit mode. We issue "BEGIN"
>   * in the following cases. 1) we use declare/fetch and the statement
>   * is SELECT (because declare/fetch must be called in a transaction).
>   * 2) we are in autocommit off state and the statement isn't of type
>   * OTHER.
>   */

Sorry that happened.  It is mentioned in the developer's FAQ that the
dashes prevent wrapping.  I know it is hard to remember even if you know
it, and I would be glad to fix any comments that look bad.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026
 


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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Problem migrating dump to latest CVS snapshot.
Следующее
От: Tatsuo Ishii
Дата:
Сообщение: Re: Call for platforms