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

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...
Date: 2001-03-22 06:34:55
Message-ID: 24332.985242895@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Bruce Momjian - CVS <momjian(at)hub(dot)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

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message The Hermit Hacker 2001-03-22 12:24:19 Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...
Previous Message Bruce Momjian - CVS 2001-03-22 06:16:21 pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...