Re: [PATCH] diff --shortstat --dirstat: remove duplicate output

2015-02-28 Thread Junio C Hamano
Johan Herland writes: > On Sat, Feb 28, 2015 at 2:19 PM, Mårten Kongstad > wrote: > [...] >> Signed-off-by: Mårten Kongstad > > Acked-by: Johan Herland Interesting. So nobody in real life uses --dirstat and --shortstat together? I am not very happy with the added tests that hardcode exact n

Re: [PATCH] diff --shortstat --dirstat: remove duplicate output

2015-02-28 Thread Johan Herland
On Sat, Feb 28, 2015 at 2:19 PM, Mårten Kongstad wrote: [...] > Signed-off-by: Mårten Kongstad Acked-by: Johan Herland -- Johan Herland, www.herland.net -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info

[PATCH] diff --shortstat --dirstat: remove duplicate output

2015-02-28 Thread Mårten Kongstad
When --shortstat is used in conjunction with --dirstat=changes, git diff will output the dirstat information twice: first as calculated by the 'lines' algorithm, then as calculated by the 'changes' algorithm: $ git diff --dirstat=changes,10 --shortstat v2.2.0..v2.2.1 23 files changed, 453