Re: [RFC PATCH v5 3/8] Indent function git_rebase__interactive

2018-03-23 Thread Wink Saville
> I actually do not care if line-wrapping is done; it is perfectly > fine to leave it for future clean-up and leave it outside the scope > of this series. If you are going to do as a part of the series, > yes, I do prefer you limit yourself to those lines that are involved > in the series in some

Re: [RFC PATCH v5 3/8] Indent function git_rebase__interactive

2018-03-23 Thread Junio C Hamano
Wink Saville writes: > Also, I assume you want me to only change lines in > git_rebase__interactive. I actually do not care if line-wrapping is done; it is perfectly fine to leave it for future clean-up and leave it outside the scope of this series. If you are going to do as a part of the serie

Re: [RFC PATCH v5 3/8] Indent function git_rebase__interactive

2018-03-23 Thread Wink Saville
On Fri, Mar 23, 2018 at 3:12 PM, Junio C Hamano wrote: > Wink Saville writes: > >> Signed-off-by: Wink Saville >> --- >> git-rebase--interactive.sh | 432 >> ++--- >> 1 file changed, 215 insertions(+), 217 deletions(-) > > Thanks for separating this step

Re: [RFC PATCH v5 3/8] Indent function git_rebase__interactive

2018-03-23 Thread Junio C Hamano
Wink Saville writes: > Signed-off-by: Wink Saville > --- > 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-inde

[RFC PATCH v5 3/8] Indent function git_rebase__interactive

2018-03-23 Thread Wink Saville
Signed-off-by: Wink Saville --- git-rebase--interactive.sh | 432 ++--- 1 file changed, 215 insertions(+), 217 deletions(-) diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh index 213d75f43..a79330f45 100644 --- a/git-rebase--interactive