Re: [PATCH v2 0/2] improve interactive-patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 27/03/2024 15:43, Junio C Hamano wrote:
Phillip Wood <phillip.wood123@xxxxxxxxx> writes:

... My
reasoning was that 'p' does not do anything useful for the user, if
they press it they end up with exactly the same content being printed
to the screen ...

Actually I do not agree that it necessarily is useless that the same
content is shown.  Especially since we do not page, it is plausible
for a user, who saw a huge hunk, to want to tweak terminal setting
and raise scrollbuffer size (which may be set to 0 for usual
sessions, like me), and say "please show it again".  Or even in a
more primitive environment, just say "please show it again" and
immediately type \C-s to stop while the early part of the hunk is
shown ;-).

Good point, I'd not thought of that - it is useful after all.

Thinking about the name of the option again, if we are omitting to
show a hunk in some situations, the request to ask for the current
hunk to be shown is "please show me", not "please reshow me", so the
verb 'print' may apply to wider situations than the verb 'reprint',
strictly speaking.

We're only omitting to show the hunk if we've already shown it so I think of it as showing it again but I don't mind much either way.

Thanks

Phillip




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux