Re: Counting lines correctly in psql help displays

From: Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Greg Stark <stark(at)mit(dot)edu>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Counting lines correctly in psql help displays
Date: 2015-09-08 23:41:58
Message-ID: 55EF7246.4070904@BlueTreble.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 9/5/15 3:50 PM, Tom Lane wrote:
> Greg Stark <stark(at)mit(dot)edu> writes:
>> But that said, here's a tricksy patch that triggers an assertion
>> failure if the expected_lines is wrong. I intended it to trigger in
>> the regression tests so it only checks if the pager is actually off.
>> It wouldn't be hard to make it always check though.
>
> Hmm ... that would put a premium on the linecount always being exactly
> right (for all callers, not just the help functions). Not sure that
> I want to buy into that --- it would require more complexity in the
> callers than is there now, for sure.

But only in an assert-enabled build. Surely there's enough other
performance hits with asserts enabled that this wouldn't matter?

As for paging, ISTM the only people that would care are those with
enormous terminal sizes would care, and assuming their pager is less
simply adding -F to $LESS would get them their old behavior. So I think
it's safe to just force paging.
--
Jim Nasby, Data Architect, Blue Treble Consulting, Austin TX
Experts in Analytics, Data Architecture and PostgreSQL
Data in Trouble? Get it in Treble! http://BlueTreble.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2015-09-08 23:55:47 Re: Counting lines correctly in psql help displays
Previous Message Thomas Munro 2015-09-08 23:33:37 Re: Making tab-complete.c easier to maintain