From: Stefan Beller <sbeller@google.com>
Date: Wed, 01 Nov 2017 19:51:34 GMT
Subject: Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
Message-ID: <CAGZ79kZ86fEvtF4rdPiAMW_x040AmwTWUnY4g1ixMjooFjU6Fw@mail.gmail.com>
In-Reply-To: <CAPig+cRz6EjdEGUjdMiB8979VybKN8Xza9uUDKNuxKNn+pY8jA@mail.gmail.com>

On Tue, Oct 31, 2017 at 2:49 PM, Eric Sunshine <sunshine@sunshineco.com> wrote:

>> to make up a name for the commit. These names are ambivalent, there might
>
> I guess you meant s/ambivalent/ambiguous/ ?

Indeed!

  ambivalent
  early 20th century: from ambivalence (from German Ambivalenz ),
  on the pattern of equivalent.

In German ambivalent (~synonym to mehrdeutig, "multi meaning")
actually means ambiguous, one of the false friends!

Thanks for pointing out.

>> [1] https://stackoverflow.com/questions/223678/which-commit-has-this-blob
>> Signed-off-by: Stefan Beller <sbeller@google.com>
>
> Blank line before sign-off.

fixes.


>
> Not at all worth a re-roll, but less nesting and a bit less noisy:

fixed.