git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Show binary file size change in diff --stat

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Feb 28, 2007, 14:44 UTC
Message-ID
<Pine.LNX.4.63.0702281535070.22628@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To
<200702281303.11951.andyparkins@gmail.com>
Hi,
On Wed, 28 Feb 2007, Andy Parkins wrote:
Show 8 quoted lines
> 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 +123456B -12345B

This seemed as a good idea to me at first. But I think it is a little misleading: Imagine for example that you have raw image data (which I track with Git, too). If only part of that changes, this representation is misleading.

How about "Bin 123456 -> 12345 bytes" instead?

Ciao, Dscho

Previous: Andy ParkinsNext: Nicolas Pitre
Message 2 of 13 in “Show binary file size change in diff --stat”
  1. Show binary file size change in diff --statAndy Parkins, Feb 28, 2007
  2. Johannes SchindelinFeb 28, 2007
  3. Nicolas PitreFeb 28, 2007
  4. Show binary file size change in diff --statAndy Parkins, Feb 28, 2007
  5. Johannes SchindelinFeb 28, 2007
  6. Andy ParkinsFeb 28, 2007
  7. Johannes SchindelinFeb 28, 2007
  8. Show binary file size change in diff --statAndy Parkins, Feb 28, 2007
  9. Rogan DawesFeb 28, 2007
  10. Johannes SchindelinFeb 28, 2007
  11. Rogan DawesFeb 28, 2007
  12. Johannes SchindelinMar 1, 2007
  13. Rogan DawesMar 1, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.