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
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_ ... |