Re: [PATCH 5/5] add-patch: render hunks through the pager

From: Junio C Hamano <gitster@pobox.com>
Date: May 23, 2024, 23:04 UTC
Message-ID: <xmqq7cfkqger.fsf@gitster.g>
In-Reply-To: <261636d461e58ac8a16180c4cd6e0460@manjaro.org>

Dragan Simic <dsimic@manjaro.org> writes:

Show 7 quoted lines

>> And for usability, perhaps giving a specific command would change
>> the default program a bare 'P' invokes for the rest of the session
>> until another specific command overrides.  Another usability hack
>> may be "[interactive] pipecommand = less -FX" configuration variable
>> gives the initial default for each session.
>
> I think that would be way too complicated.

It is modelled after how "less" and "vi" remembers the last pattern fed to their "/" command. You once give, say, "/test_<ENTER>" to find one instance of "test_", then "/<ENTER>" takes to the next instance.

As I expect our target audiences are used to such a behaviour, I do not think I agree with your "way too complicated".