Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...
Дата
Msg-id 200103221454.JAA15296@candle.pha.pa.us
обсуждение исходный текст
Ответ на Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-committers
I am a little confused on this one.  You like the earlier one because
you don't like the equals breaking on the line?


> !        * equal... if key1 != key2, we return false. otherwise key1 =
> !        * key2 so we move on to the next pair of keys.


> Bruce Momjian - CVS <momjian@hub.org> writes:
> >     Remove dashes in comments that don't need them, rewrap with pgindent.
>
> Y'know, the one thing about pgindent that I really hate is its
> assumption that it should override human judgment about block-comment
> formatting.  Without exception, pgindentified text blocks are inferior
> to the originals, and I don't think you should be helping it along
> in this nefarious deed.  A random example of stupid line breaking
> substituted for what had been an intelligently formatted comment:
>
> ***************
> *** 247,257 ****
>             break;
>         }
>
> !       /* ----------------
> !        *       ok, the compare clause failed so we test if the keys
> !        *       are equal... if key1 != key2, we return false.
> !        *       otherwise key1 = key2 so we move on to the next pair of keys.
> !        * ----------------
>          */
>         const_value = ExecEvalExpr((Node *) lfirst(eqclause),
>                                    econtext,
> --- 240,249 ----
>             break;
>         }
>
> !       /*
> !        * ok, the compare clause failed so we test if the keys are
> !        * equal... if key1 != key2, we return false. otherwise key1 =
> !        * key2 so we move on to the next pair of keys.
>          */
>         const_value = ExecEvalExpr((Node *) lfirst(eqclause),
>                                    econtext,
> ***************
> ***
>
>
>             regards, tom lane
>


--
  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, Pennsylvania 19026

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

Предыдущее
От: The Hermit Hacker
Дата:
Сообщение: Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...
Следующее
От: "Marc G. Fournier"
Дата:
Сообщение: pgsql/src/test/regress resultmap expected/geom ...