Thread (309 messages) flat view 309 messages, 17 authors, 2018-10-02

Re: [PATCH v4 11/21] range-diff: add tests

From: Eric Sunshine <hidden>
Date: 2018-07-22 05:04:34

On Sat, Jul 21, 2018 at 6:05 PM Thomas Rast via GitGitGadget
[off-list ref] wrote:
These are essentially lifted from https://github.com/trast/tbdiff, with
light touch-ups to account for the command now being names `git
s/names/named/
range-diff`.

Apart from renaming `tbdiff` to `range-diff`, only one test case needed
to be adjusted: 11 - 'changed message'.

The underlying reason it had to be adjusted is that diff generation is
sometimes ambiguous. In this case, a comment line and an empty line are
added, but it is ambiguous whether they were added after the existing
empty line, or whether an empty line and the comment line are added
*before* the existing empty line. And apparently xdiff picks a different
option here than Python's difflib.

Signed-off-by: Johannes Schindelin <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help