Re: [PATCH v2 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}'

Subject: Re: [PATCH v2 5/6] emacs: simplify `notmuch-show-get-messages-ids{, -search}'

Date: Mon, 15 Oct 2012 23:12:51 -0400

To: Mark Walters, Pieter Praet, David Bremner, Dmitry Kurochkin

Mark Walters <markwalters1009@gmail.com> writes:

> I like the use of separator rather than hard-wiring " or ". My personal
> preference would be to make that change but keep the two functions
> separate (my "C"ness makes me like functions that have clear return
> types!) But I am happy with the change too.

I agree with this comment. Especially in dynamic languages, it's a good
idea for each function to have only one return type.

Also, it seems that the commit message has a TAB character in it (",<TAB>-search").

Ethan

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]