On Mon, Oct 23, 2017 at 7:52 PM, Stefan Beller <sbeller@xxxxxxxxxx> wrote[1]: > On Mon, Oct 23, 2017 at 6:54 PM, Junio C Hamano <gitster@xxxxxxxxx> 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@xxxxxxxxxxxxxx [2] https://public-inbox.org/git/20171025184912.21657-1-sbeller@xxxxxxxxxx/ 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