Code

Show binary file size change in diff --stat
authorAndy Parkins <andyparkins@gmail.com>
Wed, 4 Apr 2007 13:14:14 +0000 (13:14 +0000)
committerJunio C Hamano <junkio@cox.net>
Thu, 5 Apr 2007 22:07:16 +0000 (15:07 -0700)
commitb18825876ae60e67e646097945c8365779175ee2
tree2bcd7842e755a2c8e7783bf176ec3239884c3789
parent566842f62bdf1f16c2e94fb431445d2e6c0f3f0b
Show binary file size change in diff --stat

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.
If a size is 0 it's not shown (although it would probably be better to
treat no-file differently from zero-byte-file).

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 <andyparkins@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
diff.c