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

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

Date: Wed, 09 Jan 2013 23:14:02 -0400

To: Jani Nikula, notmuch@notmuchmail.org

Cc:

From: David Bremner

Jani Nikula <jani@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.

d

• Previous message (by thread): Re: [PATCH] notmuch-tag: initialize with enum instead of 0.

Thread:

• Jani Nikula—[PATCH] cli: bail out and propagate tagging errors in notmuch tag [inbox, notmuch::patch, notmuch::pushed, unread]
  • Tomi Ollila—Re: [PATCH] cli: bail out and propagate tagging errors in notmuch tag [inbox, notmuch::review, unread]
  • David Bremner—Re: [PATCH] cli: bail out and propagate tagging errors in notmuch tag [inbox, unread]
    • Tomi Ollila—Re: [PATCH] cli: bail out and propagate tagging errors in notmuch tag [inbox, unread]
      • David Bremner—Re: [PATCH] cli: bail out and propagate tagging errors in notmuch tag [inbox, unread]
      • david@tethera.net—[PATCH] notmuch-tag: initialize with enum instead of 0. [inbox, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
        • Tomi Ollila—Re: [PATCH] notmuch-tag: initialize with enum instead of 0. [inbox, unread]
        • David Bremner—Re: [PATCH] notmuch-tag: initialize with enum instead of 0. [inbox, unread]