[PATCH] cli: add --output=address-{from, to, all} to notmuch search

Michal Sojka sojkam1 at fel.cvut.cz
Sat Sep 20 01:04:08 PDT 2014


On Fri, Sep 19 2014, David Bremner wrote:
> Jani Nikula <jani at nikula.org> writes:
>
>> +	    if (name && *name)
>> +		full_address = talloc_asprintf (NULL, "%s <%s>", name, addr);
>> +	    else
>> +		full_address = talloc_asprintf (NULL, "<%s>", addr
>
> Is there some reason not to use sprinter as a talloc context here?
>
>> +
>> +	    if (!full_address)
>> +		break;
>
> Is the error here out of memory? Maybe an error message would be a good
> idea.
>
>
> Obviously the docs need to be updated as well, and ideally the tests.  I
> think Jani was hinting that he didn't want to be the person to do
> that. Any volunteers?

Yes, I'll look at that. It will probably take me a few days.

-Michal


More information about the notmuch mailing list