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

From: The Hermit Hacker <scrappy(at)hub(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, <pgsql-committers(at)postgresql(dot)org>
Subject: Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...
Date: 2001-03-22 12:24:19
Message-ID: Pine.BSF.4.33.0103220822240.41105-100000@mobile.hub.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers


I think Alfred brought up a great point against pgindent also ... 'cvs
annotate' reports who touched not the file last, but down to the line ...
of course, everything that pgindent modifies is now "last modified by
momjian" ... not a major thing, but still *shrug*

On Thu, 22 Mar 2001, Tom Lane wrote:

> 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
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/users-lounge/docs/faq.html
>

Marc G. Fournier ICQ#7615664 IRC Nick: Scrappy
Systems Administrator @ hub.org
primary: scrappy(at)hub(dot)org secondary: scrappy(at){freebsd|postgresql}.org

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2001-03-22 14:54:39 Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...
Previous Message Tom Lane 2001-03-22 06:34:55 Re: pgsql/ ontrib/pg_dumplo/lo_export.c ontrib/pg_ ...