Re: [PATCH 25/25] diff: document the new --color-moved setting
From: Simon Ruderich <hidden>
Date: 2017-06-30 20:31:24
On Fri, Jun 30, 2017 at 09:04:50AM -0700, Stefan Beller wrote:
[snip]
However
context
+ moved line, block A or B
+ moved line, block A or B
context
is omitted, because the number of lines
here is fewer than 3 ignoring the block
type.
Maybe
If there are fewer than 3 adjacent lines of
moved code, they are skipped.My issue with "skipped" is that it's not clear what exactly is skipped. Move detection? Inclusion in the diff? Something else? That's why I tried to add the "excluded from move detection" to make it clear that the change is still shown to the user, just not handled by move detection and using the usual diff colors. In your example above, what exactly is "omitted"? The complete hunk from the output or the special coloring? That's what confuses me and what might confuse a future reader of the documentation. Regards Simon -- + privacy is necessary + using gnupg http://gnupg.org + public key id: 0x92FEFDB7E44C32F9