Re: Proposed patch - psql wraps at window width - Mailing list pgsql-patches

From Alvaro Herrera
Subject Re: Proposed patch - psql wraps at window width
Date
Msg-id 20080417123452.GC3846@alvh.no-ip.org
Whole thread Raw
In response to Re: Proposed patch - psql wraps at window width  (Bruce Momjian <bruce@momjian.us>)
List pgsql-patches
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

pgsql-patches by date:

Previous
From: "Brendan Jurd"
Date:
Subject: Re: printTable API (was: Show INHERIT in \du)
Next
From: Peter Eisentraut
Date:
Subject: Re: Proposed patch - psql wraps at window width