From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Neil Conway <neilc(at)samurai(dot)com> |
Cc: | David Fetter <david(at)fetter(dot)org>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Dollar Quoting doc patch--resend. |
Date: | 2004-05-05 14:42:56 |
Message-ID: | 200405051442.i45Egux22649@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Neil Conway wrote:
> On 5-May-04, at 2:26 AM, David Fetter wrote:
> > Here it is :)
>
> Looks good for the most part. A few minor quibbles / suggestions:
>
> - Where an example uses double-quotes in a function body, we ought to
> change the example to use dollar quoting -- but if no doubled-quotes
> are used, I think it's better to just stick with single quotes. You've
> needlessly converted a few function bodies to use dollar quotes in
> violation of this principle -- please revert them
I disagree. I think swiching between single quote and $$ based on the
content is just too confusing. I would just use $$ in all cases unless
$$ appears in the function (which should be rare), in which case I would
use $quote$ or something generic. I am also not a fan of using
$something$ that varies based on the type of function.
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
From | Date | Subject | |
---|---|---|---|
Next Message | David Fetter | 2004-05-05 15:17:10 | Re: Dollar Quoting doc patch--resend. |
Previous Message | Tom Lane | 2004-05-05 13:55:35 | Re: [PATCHES] Function to do runtime relative directory |