From: Stefan Beller <sbeller@google.com>
Date: Tue, 30 May 2017 21:33:02 GMT
Subject: Re: [PATCH 1/1] diff.c: color moved lines differently
Message-ID: <CAGZ79kbq3XiP8W_01FV133aMjZP9_GvpEg86N=XC2rTy24ZZGQ@mail.gmail.com>
In-Reply-To: <39D67E58AC02490FB6D6124BDF3EE2A9@PhilipOakley>

On Sat, May 27, 2017 at 12:05 AM, Philip Oakley <philipoakley@iee.org> wrote:
> a couple of mispellings in the doc parts:
>  s/on location/one location/
> [code not checked]

Thanks for proofreading the documentation!

>
> s/on location/one location/
>
>> + in another location will be colored with 'color.diff.newmoved'.
>> + Any line that is removed in on location and was added
>
> s/on location/one location/

For a moment I was worried that my 'e' key is stuck as it happens
to be the same misspelling here. However then I realized I copied
over part of the sentence and just replaced added/removed.

Stepping back a bit, maybe the second sentence can be partially
elided to be more concise. Maybe:

    Similarly 'color.diff.oldmoved' will be used for removed lines
    that are added somewhere else in the diff.

Thanks,
Stefan