Re: double counting of lines in psql

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: David Fetter <david(at)fetter(dot)org>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: double counting of lines in psql
Date: 2014-11-18 17:12:55
Message-ID: 546B7E17.3020700@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 11/18/2014 10:43 AM, David Fetter wrote:
> On Mon, Nov 17, 2014 at 11:13:15AM -0500, Andrew Dunstan wrote:
>> This tiny change fixes what I think is a longstanding bug in psql. I causes
>> the first line of every cell to be counted twice, whereas it should in fact
>> be excluded from extra_lines / extra_row_output_lines. The bug appears to
>> date back to commit 43ee2282 in 2008. Changing it appears to make my
>> proposed pager_min_lines feature work as expected.
>>
>> So, should it be backpatched? It's a behaviour change, albeit that the
>> existing behaviour is a bug, and will cause the pager to be invoked on
>> output that is way too short (by about half a screen's height, I think).
> +1
>
> It's a bug fix that changes behavior no reasonable script could count on.
>

Actually, I think I'm coming down on the side of not changing it in the
back branches. As Andres noted elsewhere, you can overcome most of the
bad effects of this by using the less pager with the -F option.

cheers

andrew

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2014-11-18 17:17:13 Re: proposal: plpgsql - Assert statement
Previous Message Alex Shulgin 2014-11-18 16:58:12 Re: Removing unreferenced files