Re: [PATCH] pager: default to LESS=FRX not LESS=FRSX
"Shawn O. Pearce" <spearce@spearce.org> writes:
Show 13 quoted lines
> Junio C Hamano <junkio@cox.net> wrote:
>> Eygene Ryabinkin <rea-git@codelabs.ru> writes:
>>
>> > Remove unnecessary '-S' option that enabled less to chop long lines.
>> > It used to provide some confusion at least for the 'git diff':
>> > "Where are my long lines? Are they missed from the patch?".
>>
>> It is not "unnecessary", but only "what _you_ are not used to".
>> As this breaks expectentions of people already familiar with
>> what git does by default, I cannot take this patch.
>
> Hear hear. I completely agree with Junio. I (and a number of
> other users that I work with) expect this behevaior by default.
Two tips.
(1) Learn to use left/right arrow keys under "less -S" if you
have occasional lines that are too long. (2) Do not have too many too long lines to begin with. If you
cannot comfortably read your code with "less -S", your
nesting is too deep to be understood with or without line
folding.Junio C Hamano <junkio@cox.net>