From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Parkins Subject: [PATCH] Show binary file size change in diff --stat Date: Wed, 28 Feb 2007 15:26:26 +0000 Message-ID: <200702281526.26247.andyparkins@gmail.com> References: Mime-Version: 1.0 Content-Transfer-Encoding: 7bit To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Wed Feb 28 16:26:33 2007 Return-path: Envelope-to: gcvg-git@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1HMQhL-0006oR-5c for gcvg-git@gmane.org; Wed, 28 Feb 2007 16:26:31 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751326AbXB1P03 (ORCPT ); Wed, 28 Feb 2007 10:26:29 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751346AbXB1P03 (ORCPT ); Wed, 28 Feb 2007 10:26:29 -0500 Received: from mail.360visiontechnology.com ([194.70.53.226]:8286 "EHLO 369run02s.360vision.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751326AbXB1P02 (ORCPT ); Wed, 28 Feb 2007 10:26:28 -0500 Received: from dvr.360vision.com ([192.189.1.65]) by 369run02s.360vision.com with Microsoft SMTPSVC(5.0.2195.6713); Wed, 28 Feb 2007 15:28:33 +0000 Received: from localhost ([127.0.0.1]) by dvr.360vision.com with esmtp (Exim 3.36 #1 (Debian)) id 1HMQhH-0001PI-00 for ; Wed, 28 Feb 2007 15:26:27 +0000 In-Reply-To: X-TUID: 0ce8faf4f11db9c6 X-UID: 271 X-Length: 2512 Content-Disposition: inline X-OriginalArrivalTime: 28 Feb 2007 15:28:33.0656 (UTC) FILETIME=[1B92BF80:01C75B4D] Sender: git-owner@vger.kernel.org Precedence: bulk X-Mailing-List: git@vger.kernel.org Archived-At: Previously, a binary file in the diffstat would show as: some-binary-file.bin | Bin The space after the "Bin" was never used. This patch changes binary lines in the diffstat to be: some-binary-file.bin | Bin 12345 -> 123456 bytes The very nice "->" notation was suggested by Johannes Schindelin, and shows the before and after sizes more clearly than "+" and "-" would. The user can see what changed in the binary file, and how big the new file is. This is in keeping with the information in the rest of the diffstat. The diffstat_t members "added" and "deleted" were unused when the file was binary, so this patch loads them with the file sizes in builtin_diffstat(). These figures are then read in show_stats() when the file is marked binary. Signed-off-by: Andy Parkins --- GAHHH!!! Ignore previous; I forgot to swap the order of the added and deleted items. I've also removed the conditionals on the 0 byte sizes - they're always shown. diff.c | 13 ++++++++++--- 1 files changed, 10 insertions(+), 3 deletions(-) diff --git a/diff.c b/diff.c index d1eae72..cfddbb4 100644 --- a/diff.c +++ b/diff.c @@ -808,7 +808,12 @@ static void show_stats(struct diffstat_t* data, struct diff_options *options) if (data->files[i]->is_binary) { show_name(prefix, name, len, reset, set); - printf(" Bin\n"); + printf(" Bin "); + printf("%s%i%s", del_c, deleted, reset); + printf(" -> "); + printf("%s%i%s", add_c, added, reset); + printf(" bytes"); + printf("\n"); goto free_diffstat_file; } else if (data->files[i]->is_unmerged) { @@ -1179,9 +1184,11 @@ static void builtin_diffstat(const char *name_a, const char *name_b, if (fill_mmfile(&mf1, one) < 0 || fill_mmfile(&mf2, two) < 0) die("unable to read files to diff"); - if (mmfile_is_binary(&mf1) || mmfile_is_binary(&mf2)) + if (mmfile_is_binary(&mf1) || mmfile_is_binary(&mf2)) { data->is_binary = 1; - else { + data->added = mf2.size; + data->deleted = mf1.size; + } else { /* Crazy xdl interfaces.. */ xpparam_t xpp; xdemitconf_t xecfg; -- 1.5.0.2.205.gea888-dirty