From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Strasser Subject: [PATCH v3 1/2] diff: Only count lines in show_shortstats Date: Fri, 15 Jun 2012 23:50:30 +0200 Message-ID: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Zbigniew =?utf-8?Q?J=C4=99drzejewski-Szmek?= , mj@ucw.cz, Junio C Hamano To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Fri Jun 15 23:50:57 2012 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1SfePv-00050v-DF for gcvg-git-2@plane.gmane.org; Fri, 15 Jun 2012 23:50:55 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758051Ab2FOVuq (ORCPT ); Fri, 15 Jun 2012 17:50:46 -0400 Received: from mailout-de.gmx.net ([213.165.64.22]:33635 "HELO mailout-de.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1757884Ab2FOVuo (ORCPT ); Fri, 15 Jun 2012 17:50:44 -0400 Received: (qmail invoked by alias); 15 Jun 2012 21:50:41 -0000 Received: from p5DC36C83.dip.t-dialin.net (EHLO gmx.net) [93.195.108.131] by mail.gmx.net (mp032) with SMTP; 15 Jun 2012 23:50:41 +0200 X-Authenticated: #8251126 X-Provags-ID: V01U2FsdGVkX18vCyMohEwv/5U1ELR61S1wHpNYB7X7mMjaRm84Lu aoXhPXcWKyvsJE Received: by gmx.net (sSMTP sendmail emulation); Fri, 15 Jun 2012 23:50:30 +0200 Content-Disposition: inline User-Agent: Mutt/1.5.21 (2010-09-15) X-Y-GMX-Trusted: 0 Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Do not mix byte and line counts. Binary files have byte counts; skip them when accumulating line insertions/deletions. The regression was introduced in e18872b. Signed-off-by: Alexander Strasser --- diff.c | 2 +- t/t4012-diff-binary.sh | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/diff.c b/diff.c index 77edd50..1a594df 100644 --- a/diff.c +++ b/diff.c @@ -1700,7 +1700,7 @@ static void show_shortstats(struct diffstat_t *data, struct diff_options *option continue; if (!data->files[i]->is_renamed && (added + deleted == 0)) { total_files--; - } else { + } else if (!data->files[i]->is_binary) { /* don't count bytes */ adds += added; dels += deleted; } diff --git a/t/t4012-diff-binary.sh b/t/t4012-diff-binary.sh index 8b4e80d..7d03c1d 100755 --- a/t/t4012-diff-binary.sh +++ b/t/t4012-diff-binary.sh @@ -36,6 +36,12 @@ test_expect_success '"apply --stat" output for binary file change' ' test_i18ncmp expected current ' +test_expect_success 'diff --shortstat output for binary file change' ' + echo " 4 files changed, 2 insertions(+), 2 deletions(-)" >expected && + git diff --shortstat >current && + test_i18ncmp expected current +' + test_expect_success 'apply --numstat notices binary file change' ' git diff >diff && git apply --numstat current && -- 1.7.10.2.552.gaa3bb87