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

Reply via email to