On Thu, Jan 10 2013, David Bremner <[email protected]> wrote: > Jani Nikula <[email protected]> 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 _______________________________________________ notmuch mailing list [email protected] http://notmuchmail.org/mailman/listinfo/notmuch
