From: Junio C Hamano <junkio@cox.net>
Date: Mon, 26 Mar 2007 08:44:34 GMT
Subject: Re: [PATCH] pager: default to LESS=FRX not LESS=FRSX
Message-ID: <7v8xdkxukt.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20070326083617.GG13247@spearce.org>

"Shawn O. Pearce" <spearce@spearce.org> writes:

> 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.