Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions

Subject: Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions

Date: Sat, 14 Jan 2012 11:31:16 -0400

To: Jani Nikula, notmuch@notmuchmail.org

Cc:

On Sat, 14 Jan 2012 16:46:15 +0200, Jani Nikula <jani@nikula.org> wrote:
> Slightly refactor "notmuch reply" recipient and user from address scanning
> functions in preparation for reply-to-sender feature.
> 

Pushed, bindings change and all.

This series definitely needs a NEWS item. 

Perhaps some kind soul could add a wiki entry explaining to people how
to swap the bindings, just in case there people who don't like the
"one-true-reply-bindings" (cough).

d

Previous message (by thread): Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions

Thread:

Jani Nikula—[PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, unread]
- Jani Nikula—[PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply" [inbox, notmuch::patch, notmuch::pushed, unread]
  - David Bremner—Re: [PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply" [inbox, unread]
    - Jani Nikula—Re: [PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply" [inbox, unread]
      - David Bremner—Re: [PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply" [inbox, unread]
  - David Bremner—Re: [PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply" [inbox, unread]
    - Jani Nikula—Re: [PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply" [inbox, unread]
      - Mark Walters—Re: [PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply" [inbox, unread]
      - David Bremner—Re: [PATCH 1/4] cli: fix use of uninitialized variable in "notmuch reply" [inbox, unread]
- Jani Nikula—[PATCH 2/4] cli: convert "notmuch reply" to use the new argument parser [inbox, notmuch::patch, notmuch::pushed, unread]
- Jani Nikula—[PATCH 3/4] cli: add support for replying just to the sender in "notmuch reply" [inbox, notmuch::obsolete, notmuch::patch, unread]
- Jani Nikula—[PATCH 4/4] emacs: add support for replying just to the sender of messages and threads [inbox, notmuch::obsolete, notmuch::patch, unread]
- Jameson Graef Rollins—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, signed, unread]
  - Mueen Nawaz—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, unread]
- Mark Walters—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, unread]
  - Jani Nikula—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, unread]
    - Mark Walters—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, unread]
      - Mark Walters—[PATCH 1/4] Add the option "--reply-to" to notmuch reply. [inbox, unread]
        Jani Nikula—Re: [PATCH 1/4] Add the option "--reply-to" to notmuch reply. [inbox, unread]
        Mark Walters—Re: [PATCH 1/4] Add the option "--reply-to" to notmuch reply. [inbox, unread]
        Jani Nikula—Re: [PATCH 1/4] Add the option "--reply-to" to notmuch reply. [inbox, unread]
      - Mark Walters—[PATCH 2/4] Update the man page for notmuch-reply to give the syntax for the --reply-to option. [inbox, unread]
      - Mark Walters—[PATCH 3/4] Emacs changes for reply-to-sender [inbox, unread]
      - Mark Walters—[PATCH 4/4] add tests for reply-to-sender [inbox, unread]
    - Mark Walters—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, unread]
- Adam Wolfe Gordon—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, unread]
  - Tomi Ollila—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, unread]
- David Edmondson—Re: [PATCH 0/4] notmuch reply bugfix & reply to sender only [inbox, signed, unread]
- Jani Nikula—[PATCH v2 0/6] reply to sender [inbox, unread]
  - Jani Nikula—[PATCH v2 1/6] cli: fix use of uninitialized variable in "notmuch reply" [inbox, unread]
  - Jani Nikula—[PATCH v2 2/6] cli: convert "notmuch reply" to use the new argument parser [inbox, unread]
  - Jani Nikula—[PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply" [inbox, unread]
    - Mark Walters—Re: [PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply" [inbox, unread]
      - Jani Nikula—Re: [PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply" [inbox, unread]
        Mark Walters—Re: [PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply" [inbox, unread]
  - Jani Nikula—[PATCH v2 4/6] emacs: add support for replying just to the sender [inbox, unread]
  - Jani Nikula—[PATCH v2 5/6] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all [inbox, unread]
    - Jeremy Nickurak—Re: [PATCH v2 5/6] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all [inbox, unread]
      - Jameson Graef Rollins—Re: [PATCH v2 5/6] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all [inbox, signed, unread]
        Jeremy Nickurak—Re: [PATCH v2 5/6] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all [inbox, unread]
      - Gregor Zattler—is there a default reply function? (was: Re: [PATCH v2 5/6] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all) [inbox, unread]
  - Jani Nikula—[PATCH v2 6/6] test: add tests for "notmuch reply" --reply-to=sender [inbox, unread]
  - David Edmondson—Re: [PATCH v2 0/6] reply to sender [inbox, notmuch::review, signed, unread]
  - Dmitry Kurochkin—Re: [PATCH v2 0/6] reply to sender [inbox, unread]
- Jani Nikula—[PATCH v3 0/4] reply to sender [inbox, unread]
  - Jani Nikula—[PATCH v3 1/4] cli: add support for replying just to the sender in "notmuch reply" [inbox, notmuch::obsolete, notmuch::patch, unread]
    - Austin Clements—Re: [PATCH v3 1/4] cli: add support for replying just to the sender in "notmuch reply" [inbox, notmuch::review, unread]
  - Jani Nikula—[PATCH v3 2/4] emacs: add support for replying just to the sender [inbox, notmuch::obsolete, notmuch::patch, unread]
    - David Edmondson—Re: [PATCH v3 2/4] emacs: add support for replying just to the sender [inbox, notmuch::review, signed, unread]
      - Jani Nikula—Re: [PATCH v3 2/4] emacs: add support for replying just to the sender [inbox, unread]
        David Edmondson—Re: [PATCH v3 2/4] emacs: add support for replying just to the sender [inbox, signed, unread]
      - Aaron Ecay—Re: [PATCH v3 2/4] emacs: add support for replying just to the sender [inbox, unread]
        Jani Nikula—Re: [PATCH v3 2/4] emacs: add support for replying just to the sender [inbox, unread]
  - Jani Nikula—[PATCH v3 3/4] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all [inbox, notmuch::obsolete, notmuch::patch, unread]
    - David Edmondson—Re: [PATCH v3 3/4] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all [inbox, notmuch::review, signed, unread]
  - Jani Nikula—[PATCH v3 4/4] test: add tests for "notmuch reply" --reply-to=sender [inbox, notmuch::obsolete, notmuch::patch, unread]
    - Austin Clements—Re: [PATCH v3 4/4] test: add tests for "notmuch reply" --reply-to=sender [inbox, notmuch::review, unread]
      - Tomi Ollila—Re: [PATCH v3 4/4] test: add tests for "notmuch reply" --reply-to=sender [inbox, unread]
- Jani Nikula—[PATCH v4 0/5] reply to sender [inbox, unread]
  - Jani Nikula—[PATCH v4 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, notmuch::obsolete, notmuch::patch, unread]
    - Austin Clements—Re: [PATCH v4 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, notmuch::review, unread]
      - Jani Nikula—Re: [PATCH v4 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, unread]
    - Mark Walters—Re: [PATCH v4 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, unread]
  - Jani Nikula—[PATCH v4 2/5] cli: add support for replying just to the sender in "notmuch reply" [inbox, notmuch::obsolete, notmuch::patch, unread]
    - Austin Clements—Re: [PATCH v4 2/5] cli: add support for replying just to the sender in "notmuch reply" [inbox, notmuch::review, unread]
    - Mark Walters—Re: [PATCH v4 2/5] cli: add support for replying just to the sender in "notmuch reply" [inbox, unread]
  - Jani Nikula—[PATCH v4 3/5] emacs: add support for replying just to the sender [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Jani Nikula—[PATCH v4 4/5] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Jani Nikula—[PATCH v4 5/5] test: add tests for "notmuch reply" --reply-to=sender [inbox, notmuch::obsolete, notmuch::patch, unread]
- Jani Nikula—[PATCH v5 0/5] reply to sender [inbox, unread]
  - Jani Nikula—[PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, notmuch::patch, notmuch::pushed, unread]
    - David Bremner—Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, unread]
      - Jani Nikula—Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, unread]
      - Dmitry Kurochkin—Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, unread]
        David Edmondson—Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, signed, unread]
        David Bremner—Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, unread]
        Jameson Graef Rollins—Re: [PATCH v5 1/5] cli: slightly refactor "notmuch reply" address scanning functions [inbox, signed, unread]
  - Jani Nikula—[PATCH v5 2/5] cli: add support for replying just to the sender in "notmuch reply" [inbox, notmuch::patch, notmuch::pushed, unread]
  - Jani Nikula—[PATCH v5 3/5] emacs: add support for replying just to the sender [inbox, notmuch::patch, notmuch::pushed, unread]
  - Jani Nikula—[PATCH v5 4/5] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all [inbox, notmuch::patch, notmuch::pushed, unread]
  - Jani Nikula—[PATCH v5 5/5] test: add tests for "notmuch reply" --reply-to=sender [inbox, notmuch::patch, notmuch::pushed, unread]