On Fri, Mar 23, 2018 at 3:12 PM, Junio C Hamano <gits...@pobox.com> wrote:
> Wink Saville <w...@saville.com> writes:
>
>> Signed-off-by: Wink Saville <w...@saville.com>
>> ---
>>  git-rebase--interactive.sh | 432 
>> ++++++++++++++++++++++-----------------------
>>  1 file changed, 215 insertions(+), 217 deletions(-)
>
> Thanks for separating this step out.  "git show -w --stat -p" tells
> us that this is a pure re-indent patch pretty easily ;-).
>
> Overlong lines might want to get rewrapped at some point, and it is
> OK to do that either in this step or in a separate step.
>

The longest line in the file before this change was line 532 which is 108
characters, now there are three lines longer because of the indentation.
Line 762 is 112, line 957 is 110 and 985 110.

My initial reaction is to leave these long lines as is, but if you want them
shorter what is the maximum line length?  At 80 characters per line
I count about 25 lines will need to be shortened.

Also, I assume you want me to only change lines in
git_rebase__interactive.

-- Wink

Reply via email to