From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Philip Warner <pjw(at)rhyme(dot)com(dot)au> |
Cc: | Tom Lane <tgl(at)svr1(dot)postgresql(dot)org>, pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql-server: Fix obviously-utterly-untested |
Date: | 2004-08-31 04:51:59 |
Message-ID: | 200408310451.i7V4px713477@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Philip Warner wrote:
> At 05:44 AM 31/08/2004, Tom Lane wrote:
> >Fix obviously-utterly-untested noTocComments code.
>
> It was extensively tested with a variety of DBs.
>
> I *think* the problem was that the patch may have been against a too-old
> source (8.0b1), which still had _printTocHeader and _printTocData. eg. the
> code:
>
> - if (AH->noTocComments)
> - return;
> -
>
> should have been in _printTocHeader, whereas it ended up in _printTocEntry.
>
> Can this be easily avoided in the future? More context and/or just use
> latest CVS?
Ah, I remember having trouble applying the patch so I had to do it
manually. I looked at the context around the patch, saw a matching
comment, and put it there. It did seem quite high to me in the function
but I figured the author knew the ramifications.
--
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 | Tom Lane | 2004-08-31 04:53:44 | pgsql-server: Replace log_filename_prefix with more general log_filename |
Previous Message | Philip Warner | 2004-08-31 04:35:23 | Re: pgsql-server: Fix obviously-utterly-untested |