[PATCH] cli: bail out and propagate tagging errors in notmuch tag

Tomi Ollila tomi.ollila at iki.fi
Wed Jan 9 20:26:54 PST 2013


On Thu, Jan 10 2013, David Bremner <david at tethera.net> wrote:

> Jani Nikula <jani at nikula.org> writes:
>> +	if (ret)
>> +	    break;
>
> I amended this to (ret != NOTMUCH_STATUS_SUCCESS) 
>
> There are a few different families of status codes flying around here,
> and I liked the extra documentation.

That sure is a good point, should follow-up patch do this:

-    int ret = 0;
+    int ret = NOTMUCH_STATUS_SUCCESS;

?

> d

Tomi


More information about the notmuch mailing list