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

Re: color.diff.whitespace unused on removed lines

From
Sandro Santilli <strk@kbt.io>
Date
Oct 4, 2016, 15:35 UTC
Message-ID
<20161004153523.GA2798@localhost>
In-Reply-To
<20161004152954.74bojbyagxr2xefz@sigill.intra.peff.net>
On Tue, Oct 04, 2016 at 11:29:55AM -0400, Jeff King wrote:
Show 12 quoted lines
> On Tue, Oct 04, 2016 at 10:14:29AM +0200, Sandro Santilli wrote:
> 
> > The color.diff.whitespace configuration is not used on
> > removed lines, but only on added lines.
> 
> Right. The original purpose was to warn you when you were introducing
> whitespace breakages. Getting rid of other people's whitespace breakages
> is OK.
> 
> We later did b8767f7 (diff.c: --ws-error-highlight=<kind> option,
> 2015-05-26) to let you see them on other lines, though. I think that
> would do what you want.

Thanks, it does do what I want. Any chance to specify it in the config file that I want it always to behave in a certain way ?

--strk;
Previous: Jeff KingNext: Jeff King
Message 3 of 11 in “color.diff.whitespace unused on removed lines”
  1. Sandro SantilliOct 4, 2016
  2. Jeff KingOct 4, 2016
  3. Sandro SantilliOct 4, 2016
  4. Jeff KingOct 4, 2016
  5. Junio C HamanoOct 4, 2016
  6. 0/4 diff.wsErrorHighlight configuration variableJunio C Hamano, Oct 4, 2016
  7. 1/4 t4015: split out the "setup" part of ws-error-highlight testJunio C Hamano, Oct 4, 2016
  8. 2/4 diff.c: refactor parse_ws_error_highlight()Junio C Hamano, Oct 4, 2016
  9. 4/4 diff: introduce diff.wsErrorHighlight optionJunio C Hamano, Oct 4, 2016
  10. 3/4 diff.c: move ws-error-highlight parsing helpers upJunio C Hamano, Oct 4, 2016
  11. Jeff KingOct 18, 2016

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.