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

[PATCH 0/2] color-moved: ignore all space changes by default

From
Stefan Beller <sbeller@google.com>
Date
Oct 25, 2017, 22:46 UTC
Message-ID
<20171025224620.27657-1-sbeller@google.com>
On Mon, Oct 23, 2017 at 7:52 PM, Stefan Beller <sbeller@google.com> wrote[1]:
Show 9 quoted lines
> On Mon, Oct 23, 2017 at 6:54 PM, Junio C Hamano <gitster@pobox.com> wrote:
>>
>>  * As moved-lines display is mostly a presentation thing, I wonder
>>    if it makes sense to always match loosely wrt whitespace
>>    differences. 
>
> Well, sometimes the user wants to know if it is byte-for-byte identical
> (unlikely to be code, but maybe column oriented data for input;
> think of all our FORTRAN users. ;)

... and this is the implementation and the flip of the default setting to ignore all white space for the move detection.

It applies on "v3 (x)diff cleanup: remove duplicate code" [2].

Thanks, Stefan

[1] https://public-inbox.org/git/CAGZ79kbba9KuDX7HsxhW3jXs7ocWfZg=KSHE-GsXjntnT4PWdg@mail.gmail.com [2] https://public-inbox.org/git/20171025184912.21657-1-sbeller@google.com/

Stefan Beller (2):
  diff: decouple white space treatment for move detection from generic
    option
  diff.c: ignore all white space changes by default in the move
    detection
 Documentation/diff-options.txt |  15 ++++
 diff.c                         |  18 ++++-
 diff.h                         |   1 +
 t/t4015-diff-whitespace.sh     | 156 +++++++++++++++++++++++++++++++++++++++--
 4 files changed, 181 insertions(+), 9 deletions(-)
-- 
2.15.0.rc2.6.g953226eb5f
Next: Stefan Beller
Message 1 of 6 in “color-moved: ignore all space changes by default”
  1. 0/2 color-moved: ignore all space changes by defaultStefan Beller, Oct 25, 2017
  2. 1/2 diff: decouple white space treatment for move detection from generic optionStefan Beller, Oct 25, 2017
  3. 2/2 diff.c: ignore all white space changes by default in the move detectionStefan Beller, Oct 25, 2017
  4. Simon RuderichOct 26, 2017
  5. Jacob KellerOct 26, 2017
  6. Stefan BellerOct 26, 2017

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.