Re: wrapping in extended mode doesn't work well with default pager

On Wed, Jun 11, 2014 at 7:52 PM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
> On 6/8/14, 11:29 PM, Noah Misch wrote:
>> The patch did not restore 9.3 behavior for that one. Starting with commit
>> 6513633, the first line of letters is space-padded on the right to the width
>> of the second line of letters. To illustrate, I have attached raw psql output
>> from both commit 6513633 and its predecessor. Also note that
>> psql-wrapped-expanded-fix-v4.patch expands each [ RECORD x ] header from 509
>> bytes to 511 bytes; 509 is the longstanding width.
>
> I noticed that (or perhaps a related) problem today. Here is a simple demo:

I don't think these two issues are related.

The leading space that you (ie Peter) are complaining about in:

col | 1
----+--
col | 2

Is there because if the cell wrapped it would get an ellipsis (ie
'...' but it's a single unicode character) in that column to indicate
that it's wrapped. However we don't wrap headers so the only reason to
change it is for the "old-ascii" linestyle:

stark=***# select * from (values (1),(2)) as _ ("col
col");
stark"***#
col | 1
+col ;
-----+-----
col | 2
+col ;

Noah's complaint is about the space padding on the *right*. Ie

stark=***# select * from (values ('foo'),('foo bar baz')) as _ ("col");
col | foo
<- This is the end of the line
-----+--------------------------------------------------------------------------<-
This is the end of the line
col | foo bar baz
<- This is the end of the line

We didn't used to do that in expanded and I kind of agree it would be
nice to avoid. But then there are lots of cases where it would still
be necessary:

stark=***# select * from (values ('foo'),('foo bar
baz')) as _ ("col");
stark'***#
col | foo
<- This is the end of the line
-----+--------------------------------------------------------------------------<-
This is the end of the line
col | foo bar
+<- This is the end of the line
| baz
<- This is the end of the line

Obviously we would need to space padd to insert the "+" there.

I think this whole exercise has mostly just convinced me we should
implement an HTTP interface and reimplement psql as a browser app.

--
greg