Re: emacs: more flexible and consistent tagging operations

Subject: Re: emacs: more flexible and consistent tagging operations

Date: Mon, 30 Jan 2012 10:20:02 -0500

The whole series LGTM.

Quoth Dmitry Kurochkin on Jan 30 at  9:33 am:
> Changes:
> 
> v3:
> 
> * merged 3 `notmuch-show-tag-all'-related patches into one
> 
> * add patch to clean up tagging function argument names
> 
> * fix other comments from Austin's reviews [5,6]
> 
> v2:
> 
> * add patch to remove "No tags given" error from `notmuch-tag' as
>   suggested by Austin in [1]
> 
> * split patch 3 in two (search and show) for easier review
> 
> * add patch with NEWS entry
> 
> * rename `notmuch-{search,show}-operate-all' to
>   `notmuch-{search,show}-tag-all'
> 
> * fix other comments from Austin's reviews [2,3,4]
> 
> Regards,
>   Dmitry
> 
> [1] id:"20120129231650.GK17991@mit.edu"
> [2] id:"20120129225710.GG17991@mit.edu"
> [3] id:"20120129230229.GI17991@mit.edu"
> [4] id:"20120129231120.GJ17991@mit.edu"
> [5] id:"20120130044806.GM17991@mit.edu"
> [6] id:"20120130050402.GP17991@mit.edu"

Previous message (by thread): Re: [PATCH v2 3/6] test: emacs: `notmuch-show-tag-all' with prefix arg should only tag open messages

Thread:

Dmitry Kurochkin—[PATCH 1/6] emacs: move tag format validation to `notmuch-tag' function [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH 2/6] emacs: remove text properties from `notmuch-search-get-tags' result [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH 3/6] emacs: make "+" and "-" tagging operations more robust [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Jeremy Nickurak—Re: [PATCH 3/6] emacs: make "+" and "-" tagging operations more robust [inbox, unread]
    - Dmitry Kurochkin—Re: [PATCH 3/6] emacs: make "+" and "-" tagging operations more robust [inbox, unread]
  - Austin Clements—Re: [PATCH 3/6] emacs: make "+" and "-" tagging operations more robust [inbox, notmuch::review, unread]
    - Dmitry Kurochkin—Re: [PATCH 3/6] emacs: make "+" and "-" tagging operations more robust [inbox, unread]
- Dmitry Kurochkin—[PATCH 4/6] test: fix emacs tests after tagging operations changes [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
  - Austin Clements—Re: [PATCH 4/6] test: fix emacs tests after tagging operations changes [inbox, notmuch::review, unread]
    - Dmitry Kurochkin—Re: [PATCH 4/6] test: fix emacs tests after tagging operations changes [inbox, unread]
- Dmitry Kurochkin—[PATCH 5/6] emacs: add "*" binding for notmuch-show view [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH 6/6] emacs: separate history for operations which accept single and multiple tags [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Austin Clements—Re: [PATCH 6/6] emacs: separate history for operations which accept single and multiple tags [inbox, notmuch::review, unread]
    - Dmitry Kurochkin—Re: [PATCH 6/6] emacs: separate history for operations which accept single and multiple tags [inbox, unread]
- Dmitry Kurochkin—[PATCH 7/6] emacs: relax tag syntax check in `notmuch-tag' function [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Jani Nikula—Re: [PATCH 7/6] emacs: relax tag syntax check in `notmuch-tag' function [inbox, notmuch::review, unread]
    - Dmitry Kurochkin—Re: [PATCH 7/6] emacs: relax tag syntax check in `notmuch-tag' function [inbox, unread]
    - Pieter Praet—Re: [PATCH 7/6] emacs: relax tag syntax check in `notmuch-tag' function [inbox, unread]
- Dmitry Kurochkin—[PATCH 8/6] emacs: use message ids instead of thread id in `notmuch-show-operate-all' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH 9/6] emacs: code cleanup in `notmuch-show-operate-all', no functional changes [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Jani Nikula—Re: [PATCH 8/6] emacs: use message ids instead of thread id in `notmuch-show-operate-all' [inbox, notmuch::review, unread]
    - Dmitry Kurochkin—Re: [PATCH 8/6] emacs: use message ids instead of thread id in `notmuch-show-operate-all' [inbox, unread]
  - Austin Clements—Re: [PATCH 8/6] emacs: use message ids instead of thread id in `notmuch-show-operate-all' [inbox, notmuch::review, unread]
    - Dmitry Kurochkin—Re: [PATCH 8/6] emacs: use message ids instead of thread id in `notmuch-show-operate-all' [inbox, unread]
- Austin Clements—Re: [PATCH 1/6] emacs: move tag format validation to `notmuch-tag' function [inbox, notmuch::review, unread]
  - Dmitry Kurochkin—Re: [PATCH 1/6] emacs: move tag format validation to `notmuch-tag' function [inbox, unread]
    - Austin Clements—Re: [PATCH 1/6] emacs: move tag format validation to `notmuch-tag' function [inbox, unread]
      - Dmitry Kurochkin—Re: [PATCH 1/6] emacs: move tag format validation to `notmuch-tag' function [inbox, unread]
- Dmitry Kurochkin—[PATCH v2 00/13] emacs: more robust and consistent tagging operations [inbox, unread]
  - Dmitry Kurochkin—[PATCH v2 01/13] emacs: move tag format validation to `notmuch-tag' function [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v2 02/13] emacs: remove text properties from `notmuch-search-get-tags' result [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v2 03/13] emacs: make "+" and "-" tagging operations in notmuch-search more robust [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Austin Clements—Re: [PATCH v2 03/13] emacs: make "+" and "-" tagging operations in notmuch-search more robust [inbox, notmuch::review, unread]
      - Dmitry Kurochkin—Re: [PATCH v2 03/13] emacs: make "+" and "-" tagging operations in notmuch-search more robust [inbox, unread]
  - Dmitry Kurochkin—[PATCH v2 04/13] emacs: make "+" and "-" tagging operations in notmuch-show more robust [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v2 05/13] test: fix emacs tests after tagging operations changes [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
  - Dmitry Kurochkin—[PATCH v2 06/13] emacs: rename `notmuch-search-operate-all' to `notmuch-search-tag-all' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v2 07/13] emacs: add "*" binding for notmuch-show view [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v2 08/13] emacs: separate history for operations which accept single and multiple tags [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v2 09/13] emacs: relax tag syntax check in `notmuch-tag' function [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v2 10/13] emacs: use message ids instead of thread id in `notmuch-show-operate-all' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Austin Clements—Re: [PATCH v2 10/13] emacs: use message ids instead of thread id in `notmuch-show-operate-all' [inbox, notmuch::review, unread]
      - Dmitry Kurochkin—Re: [PATCH v2 10/13] emacs: use message ids instead of thread id in `notmuch-show-operate-all' [inbox, unread]
  - Dmitry Kurochkin—[PATCH v2 11/13] emacs: code cleanup in `notmuch-show-operate-all', no functional changes [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Austin Clements—Re: [PATCH v2 11/13] emacs: code cleanup in `notmuch-show-operate-all', no functional changes [inbox, notmuch::review, unread]
      - Dmitry Kurochkin—Re: [PATCH v2 11/13] emacs: code cleanup in `notmuch-show-operate-all', no functional changes [inbox, unread]
  - Dmitry Kurochkin—[PATCH v2 12/13] emacs: accept empty tag list in `notmuch-tag' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v2 13/13] NEWS: document Emacs UI tagging operations changes [inbox, notmuch::obsolete, notmuch::patch, unread]
    - Austin Clements—Re: [PATCH v2 13/13] NEWS: document Emacs UI tagging operations changes [inbox, notmuch::review, unread]
      - Dmitry Kurochkin—Re: [PATCH v2 13/13] NEWS: document Emacs UI tagging operations changes [inbox, unread]
- Dmitry Kurochkin—emacs: more flexible and consistent tagging operations [inbox, unread]
  - Dmitry Kurochkin—[PATCH v3 01/12] emacs: move tag format validation to `notmuch-tag' function [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 02/12] emacs: remove text properties from `notmuch-search-get-tags' result [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 03/12] emacs: make "+" and "-" tagging operations in notmuch-search more flexible [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 04/12] emacs: make "+" and "-" tagging operations in notmuch-show more flexible [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 05/12] test: fix emacs tests after tagging operations changes [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
  - Dmitry Kurochkin—[PATCH v3 06/12] emacs: rename `notmuch-search-operate-all' to `notmuch-search-tag-all' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 07/12] emacs: add "*" binding for notmuch-show view [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 08/12] emacs: separate history for operations which accept single and multiple tags [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 09/12] emacs: relax tag syntax check in `notmuch-tag' function [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 10/12] emacs: accept empty tag list in `notmuch-tag' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 11/12] emacs: s/tags/tag-changes/ for arguments of tagging functions [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—[PATCH v3 12/12] NEWS: document Emacs UI tagging operations changes [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  - Austin Clements—Re: emacs: more flexible and consistent tagging operations [inbox, notmuch::review, unread]
  - Tomi Ollila—Re: emacs: more flexible and consistent tagging operations [inbox, unread]
  - Dmitry Kurochkin—[PATCH v4 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
    - Dmitry Kurochkin—[PATCH v4 01/12] emacs: move tag format validation to `notmuch-tag' function [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 02/12] emacs: remove text properties from `notmuch-search-get-tags' result [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
      - David Bremner—Re: [PATCH v4 02/12] emacs: remove text properties from `notmuch-search-get-tags' result [inbox, unread]
    - Dmitry Kurochkin—[PATCH v4 03/12] emacs: make "+" and "-" tagging operations in notmuch-search more flexible [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 04/12] emacs: make "+" and "-" tagging operations in notmuch-show more flexible [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 05/12] test: fix emacs tests after tagging operations changes [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
    - Dmitry Kurochkin—[PATCH v4 06/12] emacs: rename `notmuch-search-operate-all' to `notmuch-search-tag-all' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 07/12] emacs: add "*" binding for notmuch-show view [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 08/12] emacs: separate history for operations which accept single and multiple tags [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 09/12] emacs: relax tag syntax check in `notmuch-tag' function [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 10/12] emacs: accept empty tag list in `notmuch-tag' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 11/12] emacs: s/tags/tag-changes/ for arguments of tagging functions [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    - Dmitry Kurochkin—[PATCH v4 12/12] NEWS: document Emacs UI tagging operations changes [inbox, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
  - Dmitry Kurochkin—[PATCH v5 01/12] emacs: move tag format validation to `notmuch-tag' function [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 02/12] emacs: remove text properties from `notmuch-search-get-tags' result [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 03/12] emacs: make "+" and "-" tagging operations in notmuch-search more flexible [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 04/12] emacs: make "+" and "-" tagging operations in notmuch-show more flexible [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 05/12] test: fix emacs tests after tagging operations changes [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, notmuch::test, unread]
  - Dmitry Kurochkin—[PATCH v5 06/12] emacs: rename `notmuch-search-operate-all' to `notmuch-search-tag-all' [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 07/12] emacs: add "*" binding for notmuch-show view [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 08/12] emacs: separate history for operations which accept single and multiple tags [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 09/12] emacs: relax tag syntax check in `notmuch-tag' function [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 10/12] emacs: accept empty tag list in `notmuch-tag' [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 11/12] emacs: s/tags/tag-changes/ for arguments of tagging functions [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - Dmitry Kurochkin—[PATCH v5 12/12] NEWS: document Emacs UI tagging operations changes [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
  - David Bremner—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
    - Pieter Praet—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
      - Pieter Praet—[PATCH 1/6] test: emacs: new tests "notmuch-show: {add, remove} multiple tags {to, from} single message" [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
        Dmitry Kurochkin—Re: [PATCH 1/6] test: emacs: new tests "notmuch-show: {add, remove} multiple tags {to, from} single message" [inbox, notmuch::review, unread]
        Pieter Praet—Re: [PATCH 1/6] test: emacs: new tests "notmuch-show: {add, remove} multiple tags {to, from} single message" [inbox, unread]
      - Pieter Praet—[PATCH 2/6] test: emacs: new test "notmuch-show: change tags of all messages in current buffer" [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
      - Pieter Praet—[PATCH 3/6] test: emacs: `notmuch-show-tag-all' sans prefix arg should only tag open messages [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
      - Pieter Praet—[PATCH 4/6] emacs: add predicate arg to `notmuch-show-mapc' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
        Dmitry Kurochkin—Re: [PATCH 4/6] emacs: add predicate arg to `notmuch-show-mapc' [inbox, notmuch::review, unread]
        Pieter Praet—Re: [PATCH 4/6] emacs: add predicate arg to `notmuch-show-mapc' [inbox, unread]
      - Pieter Praet—[PATCH 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}' [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
        Dmitry Kurochkin—Re: [PATCH 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}' [inbox, notmuch::review, unread]
        Pieter Praet—Re: [PATCH 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}' [inbox, unread]
      - Pieter Praet—[PATCH 6/6] emacs: `notmuch-show-tag-all' sans prefix arg only tags open messages [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
        Dmitry Kurochkin—Re: [PATCH 6/6] emacs: `notmuch-show-tag-all' sans prefix arg only tags open messages [inbox, notmuch::review, unread]
        Pieter Praet—Re: [PATCH 6/6] emacs: `notmuch-show-tag-all' sans prefix arg only tags open messages [inbox, unread]
      - Dmitry Kurochkin—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, notmuch::review, unread]
        Dmitry Kurochkin—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
        Jani Nikula—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
        Pieter Praet—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
        Pieter Praet—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
        Pieter Praet—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
      - David Bremner—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
        Tomi Ollila—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
        Pieter Praet—Re: [PATCH v5 00/12] emacs: more flexible and consistent tagging operations [inbox, unread]
    - Pieter Praet—[PATCH v2 0/6] `notmuch-show-tag-all' with prefix arg only tags open messages [inbox, unread]
      - Pieter Praet—[PATCH v2 1/6] test: emacs: new tests "notmuch-show: {add, remove} multiple tags {to, from} single message" [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        Mark Walters—Re: [PATCH v2 1/6] test: emacs: new tests "notmuch-show: {add, remove} multiple tags {to, from} single message" [inbox, notmuch::review, unread]
        David Bremner—Re: [PATCH v2 1/6] test: emacs: new tests "notmuch-show: {add, remove} multiple tags {to, from} single message" [inbox, unread]
        David Bremner—Re: [PATCH v2 1/6] test: emacs: new tests "notmuch-show: {add, remove} multiple tags {to, from} single message" [inbox, unread]
      - Pieter Praet—[PATCH v2 2/6] test: emacs: new test "notmuch-show: change tags of all messages in current buffer" [inbox, notmuch::moreinfo, notmuch::patch, notmuch::test, unread]
        Ethan Glasser-Camp—Re: [PATCH v2 2/6] test: emacs: new test "notmuch-show: change tags of all messages in current buffer" [inbox, notmuch::review, unread]
      - Pieter Praet—[PATCH v2 3/6] test: emacs: `notmuch-show-tag-all' with prefix arg should only tag open messages [inbox, notmuch::patch, notmuch::stale, notmuch::test, unread]
        David Bremner—Re: [PATCH v2 3/6] test: emacs: `notmuch-show-tag-all' with prefix arg should only tag open messages [inbox, unread]
      - Pieter Praet—[PATCH v2 4/6] emacs: add optional predicate arg to `notmuch-show-mapc' [inbox, notmuch::emacs, notmuch::moreinfo, notmuch::patch, unread]
        Mark Walters—Re: [PATCH v2 4/6] emacs: add optional predicate arg to `notmuch-show-mapc' [inbox, notmuch::review, unread]
        Ethan Glasser-Camp—Re: [PATCH v2 4/6] emacs: add optional predicate arg to `notmuch-show-mapc' [inbox, unread]
        Ethan Glasser-Camp—Re: [PATCH v2 4/6] emacs: add optional predicate arg to `notmuch-show-mapc' [inbox, notmuch::review, unread]
      - Pieter Praet—[PATCH v2 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}' [inbox, notmuch::emacs, notmuch::patch, notmuch::stale, unread]
        Mark Walters—Re: [PATCH v2 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}' [inbox, notmuch::review, unread]
        Ethan Glasser-Camp—Re: [PATCH v2 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}' [inbox, notmuch::review, unread]
        Mark Walters—Re: [PATCH v2 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}' [inbox, unread]
      - Pieter Praet—[PATCH v2 6/6] emacs: `notmuch-show-tag-all' with prefix arg only tags open messages [inbox, notmuch::emacs, notmuch::patch, notmuch::stale, unread]
        Ethan Glasser-Camp—Re: [PATCH v2 6/6] emacs: `notmuch-show-tag-all' with prefix arg only tags open messages [inbox, notmuch::review, unread]