Re: [PATCH 4/6] combine-diff: treat --shortstat like --stat
- From
Stefan Beller <sbeller@google.com>
- Date
- Jan 24, 2019, 19:02 UTC
- Message-ID
- <CAGZ79kbz9QFLG1sr=WQ_kvLMis1hf41wCjK1Ngu4+1iAv3XO1g@mail.gmail.com>
- In-Reply-To
- <20190124123450.GD11354@sigill.intra.peff.net>
On Thu, Jan 24, 2019 at 4:34 AM Jeff King <peff@peff.net> wrote:
Show 26 quoted lines
> > The --stat of a combined diff is defined as the first-parent stat, > going all the way back to 965f803c32 (combine-diff: show diffstat with > the first parent., 2006-04-17). > > Naturally, we gave --numstat the same treatment in 74e2abe5b7 (diff > --numstat, 2006-10-12). > > But --shortstat, which is really just the final line of --stat, does > nothing, which produces confusing results: > > $ git show --oneline --stat eab7584e37 > eab7584e37 Merge branch 'en/show-ref-doc-fix' > > Documentation/git-show-ref.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > $ git show --oneline --shortstat eab7584e37 > eab7584e37 Merge branch 'en/show-ref-doc-fix' > > [nothing! We'd expect to see the "1 file changed..." line] > > This patch teaches combine-diff to treats the two formats identically. > > Reported-by: David Turner <novalis@novalis.org> > Signed-off-by: Jeff King <peff@peff.net>
Both the previous patch and this one is reviewed by me.
Thanks, Stefan