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

Re: Diff --stat for files that differ only in whitespace

From
Jeff King <peff@peff.net>
Date
Jul 31, 2020, 19:48 UTC
Message-ID
<20200731194810.GA849395@coredump.intra.peff.net>
In-Reply-To
<CAOjrSZskJRj+TVV35fEp-Mhd+proVV72rh3vAcVFnNfhRy2ayA@mail.gmail.com>
On Fri, Jul 31, 2020 at 03:26:16PM -0400, Matt Rogers wrote:
Show 14 quoted lines
> > This seemed familiar, so I dug up some prior discussion here:
> >
> >   https://lore.kernel.org/git/1484704915.2096.16.camel@mattmccutchen.net/
> >
> > We didn't come to a resolution there, but there is a patch to play with,
> > and I think nobody was opposed to the notion that with the right
> > code change we could be suppressing these whitespace-only stat lines
> >
> > -Peff
> 
> 
> I think for now I'm going to feed --numstat into a script like Junio suggested.
> Out of curiosity what would it take to get that patch into git? Is it just a
> matter of someone just verifying it and submitting it?

Yeah, I think it would require making an argument that the patch covers the correct set of cases (or fixing it if it doesn't), and probably adding some tests.

-Peff
Previous: Matt RogersNext: Junio C Hamano
Message 5 of 7 in “Diff --stat for files that differ only in whitespace”
  1. Matt RogersJul 31, 2020
  2. Junio C HamanoJul 31, 2020
  3. Jeff KingJul 31, 2020
  4. Matt RogersJul 31, 2020
  5. Jeff KingJul 31, 2020
  6. Junio C HamanoJul 31, 2020
  7. Matt RogersJul 31, 2020

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.