Re: [PATCH 0/3] notmuch show argument parsing

Subject: Re: [PATCH 0/3] notmuch show argument parsing

Date: Fri, 10 Feb 2012 15:59:12 +0000

To: Jani Nikula, notmuch@notmuchmail.org

Cc:

From: Mark Walters

On Mon,  6 Feb 2012 21:57:20 +0200, Jani Nikula <jani@nikula.org> wrote:
> Hi all, 
> 
> v2 addressing Austin's comments in id:"20120206041205.GP10898@mit.edu".
> Separate the bool cleanup into a new patch, cleaning up notmuch reply
> while at it. No functional changes since v1.
> 
> For reviewing convenience, the diff between v1 and v2 is at the end of
> this cover letter.

Just to confirm this version LGTM too.

Best wishes

Mark

• Previous message (by thread): Re: [PATCH 0/3] notmuch show argument parsing

Thread:

• Jani Nikula—[PATCH 1/2] cli: convert "notmuch show" to use the new argument parser [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Jani Nikula—[PATCH 2/2] cli: reach previously unreachable cleanup code in "notmuch show" [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Mark Walters—Re: [PATCH 1/2] cli: convert "notmuch show" to use the new argument parser [inbox, notmuch::review, unread]
    • Jani Nikula—Re: [PATCH 1/2] cli: convert "notmuch show" to use the new argument parser [inbox, unread]
  • Austin Clements—Re: [PATCH 1/2] cli: convert "notmuch show" to use the new argument parser [inbox, notmuch::review, unread]
    • Jani Nikula—Re: [PATCH 1/2] cli: convert "notmuch show" to use the new argument parser [inbox, unread]
      • Austin Clements—Re: [PATCH 1/2] cli: convert "notmuch show" to use the new argument parser [inbox, unread]
  • Jani Nikula—[PATCH 0/3] notmuch show argument parsing [inbox, unread]
    • Jani Nikula—[PATCH v2 1/3] cli: use notmuch_bool_t for boolean fields in notmuch_show_params_t [inbox, notmuch::patch, notmuch::pushed, unread]
    • Jani Nikula—[PATCH v2 2/3] cli: convert "notmuch show" to use the new argument parser [inbox, notmuch::patch, notmuch::pushed, unread]
    • Jani Nikula—[PATCH v2 3/3] cli: reach previously unreachable cleanup code in "notmuch show" [inbox, notmuch::patch, notmuch::pushed, unread]
    • Austin Clements—Re: [PATCH 0/3] notmuch show argument parsing [inbox, notmuch::review, unread]
    • Mark Walters—Re: [PATCH 0/3] notmuch show argument parsing [inbox, notmuch::review, unread]
    • David Bremner—Re: [PATCH 0/3] notmuch show argument parsing [inbox, unread]