[PATCH v2 6/9] cli: change the data structure for notmuch address deduplication

Tomi Ollila tomi.ollila at iki.fi
Thu Sep 24 12:55:33 PDT 2015


On Thu, Sep 24 2015, David Bremner <david at tethera.net> wrote:

> David Bremner <david at tethera.net> writes:
>
>> Jani Nikula <jani at nikula.org> writes:
>>
>>
>>> +    else
>>> +	v = !!m1->name - !!m2->name;
>>
>> Is this really idiomatic? It seems a little difficult to follow to me.
>>

For me it took a while to grasp the meaning of that but after I spent
a minute looking at it it become clear to me. this part +1


> Sorry, this stupid MUA I'm using confused me and I sent the reply
> halfway through reading the patch. Anyway, the rest looks OK.
>
> d

Tomi


More information about the notmuch mailing list