[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
detectionDocumentation/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