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: Mon, 16 Jan 2012 22:06:15 +0400

To: David Bremner, Jani Nikula, notmuch@notmuchmail.org

Cc:

Hello.

Since this patches got in, I have yet to send a single email to the
address(es) I intend to :( I am really used to the bindings and this
change is a pain.  From IRC discussion, it seems like I am not alone
here.

There was a proposal to change reply bindings to 'ra' and 'rs'.  When
'r' is pressed an appropriate help message can be given in the
minibuffer.  I like this proposal.  What do others think?

If others like it, how about reverting to the old bindings until the new
ones are implemented?

Regards,
  Dmitry

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]