Re: [PATCH] stripspace: add --line-count flag

2016-03-04 Thread Junio C Hamano
Junio C Hamano writes: > Sidhant Sharma writes: > >> This is my first attempt at the small project listed here: >> https://git.wiki.kernel.org/index.php/SmallProjectsIdeas#implement_.27--count-lines.27_in_.27git_stripspace.27. > >> Comments? > > Isn't that page somewhat stale? > > http://git.66

Re: [PATCH] stripspace: add --line-count flag

2016-03-04 Thread Sidhant Sharma
On Saturday 05 March 2016 12:19 AM, Junio C Hamano wrote: > Sidhant Sharma writes: > >> This is my first attempt at the small project listed here: >> https://git.wiki.kernel.org/index.php/SmallProjectsIdeas#implement_.27--count-lines.27_in_.27git_stripspace.27. >> Comments? > Isn't that page so

Re: [PATCH] stripspace: add --line-count flag

2016-03-04 Thread Junio C Hamano
Sidhant Sharma writes: > This is my first attempt at the small project listed here: > https://git.wiki.kernel.org/index.php/SmallProjectsIdeas#implement_.27--count-lines.27_in_.27git_stripspace.27. > Comments? Isn't that page somewhat stale? http://git.661346.n2.nabble.com/PATCH-0-3-stripspac

Re: [PATCH] stripspace: add --line-count flag

2016-03-04 Thread Sidhant Sharma
> builtin/stripspace.c | 22 +- > git-rebase--interactive.sh | 6 +++--- > 2 files changed, 24 insertions(+), 4 deletions(-) > This is my first attempt at the small project listed here: https://git.wiki.kernel.org/index.php/SmallProjectsIdeas#implement_.27--count-lines

[PATCH] stripspace: add --line-count flag

2016-03-04 Thread Sidhant Sharma [:tk]
When used, this flag outputs number of lines after stripspace has removed trailing whitespace. With `--line-count`, git-rebase--interactive.sh need not rely on `wc -l` for line count. Signed-off-by: Sidhant Sharma [:tk] --- This the first version of the patch for the small project listed here