v2:
I think I have addressed Jonathans feedback
* by using a string instead of counting the first character only.
* refined tests slightly (easier to read)
* moved white space handling for moved blocks into its own flag field,
keeping the enum for the actual mode of move detection.
v1:
This is a re-attempt of [1], which allows the moved code detection to
ignore blanks in various modes.
patches 1-5 are refactoring, patch 6 adds all existing white space options
of regular diff to the move detection. (I am unsure about this patch,
as I presume we want to keep the option space at a minimum if possible).
The fun is in the last patch, which allows white space sensitive
languages to trust the move detection, too. Each block that is marked as
moved will have the same delta in {in-, de-}dentation.
I would think this mode might be a reasonable default eventually.
Thanks,
Stefan
[1] https://public-inbox.org/git/20171025224620.27657-1-sbeller@xxxxxxxxxx/
Stefan Beller (7):
xdiff/xdiff.h: remove unused flags
xdiff/xdiffi.c: remove unneeded function declarations
diff.c: do not pass diff options as keydata to hashmap
diff.c: adjust hash function signature to match hashmap expectation
diff.c: add a blocks mode for moved code detection
diff.c: decouple white space treatment from move detection algorithm
diff.c: add --color-moved-ignore-space-delta option
Documentation/diff-options.txt | 25 ++++-
diff.c | 128 ++++++++++++++++++----
diff.h | 8 +-
t/t4015-diff-whitespace.sh | 192 +++++++++++++++++++++++++++++++--
xdiff/xdiff.h | 8 --
xdiff/xdiffi.c | 17 ---
6 files changed, 322 insertions(+), 56 deletions(-)
--
2.17.0.441.gb46fe60e1d-goog
