Bruce Momjian wrote:
> In testing I found the regression tests were failing because of a divide
> by zero error (fixed), and a missing case where the column delimiter has
> to be ":". In fact I now see all your line continuation cases using ":"
> rather than "!". It actually looks better --- "!" was too close to "|"
> to be easily recognized. (Did you update your patch to use ":". I
> didn't see "!" in your patch.)
I think we should use a different separator when there is an actual
newline in the data. Currently we have a : there, so using a : here is
probably not the best idea.
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
In response to
pgsql-hackers by date
|Next:||From: Peter Eisentraut||Date: 2008-04-17 12:39:43|
|Subject: get rid of psql welcome message|
|Previous:||From: Alvaro Herrera||Date: 2008-04-17 12:30:12|
|Subject: Re: [COMMITTERS] pgsql: Repair two places where SIGTERM exit couldleave shared memory|
pgsql-patches by date
|Next:||From: Peter Eisentraut||Date: 2008-04-17 13:09:23|
|Subject: Re: Proposed patch - psql wraps at window width|
|Previous:||From: Brendan Jurd||Date: 2008-04-17 08:48:42|
|Subject: Re: printTable API (was: Show INHERIT in \du)|