On Fri, Jun 30, 2017 at 1:31 PM, Simon Ruderich <simon@xxxxxxxxxxxx> wrote: > 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. Oh. I see. That was not the point of confusion that I thought. skipped/omitted both refer to the move detection and the lines will be included in the diff, but just as regular added/removed lines. Will fix. Thanks for bearing with me, Stefan
