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

2012-01-16 Thread Dmitry Kurochkin
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 p

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

2012-01-16 Thread David Edmondson
On Mon, 16 Jan 2012 22:06:15 +0400, Dmitry Kurochkin wrote: > 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. > > T

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

2012-01-16 Thread David Bremner
On Mon, 16 Jan 2012 22:06:15 +0400, Dmitry Kurochkin wrote: > > If others like it, how about reverting to the old bindings until the new > ones are implemented? > Flip-flopping the bindings will just cause more confusion, in my opinion. It is easy (and documented in the wiki) to customize the

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

2012-01-16 Thread Jameson Graef Rollins
On Mon, 16 Jan 2012 22:06:15 +0400, Dmitry Kurochkin wrote: > 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. Yeah, this has b

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

2012-01-16 Thread Jameson Graef Rollins
On Mon, 16 Jan 2012 22:06:15 +0400, Dmitry Kurochkin wrote: > 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. Yeah, this has bi

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

2012-01-16 Thread David Bremner
On Mon, 16 Jan 2012 22:06:15 +0400, Dmitry Kurochkin wrote: > > If others like it, how about reverting to the old bindings until the new > ones are implemented? > Flip-flopping the bindings will just cause more confusion, in my opinion. It is easy (and documented in the wiki) to customize the

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

2012-01-16 Thread David Edmondson
On Mon, 16 Jan 2012 22:06:15 +0400, Dmitry Kurochkin wrote: > 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. > >

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

2012-01-16 Thread Dmitry Kurochkin
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 p

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

2012-01-14 Thread Jani Nikula
For those not on IRC: On Sat, 14 Jan 2012 11:31:16 -0400, David Bremner wrote: > This series definitely needs a NEWS item. id:"1326559168-29178-1-git-send-email-jani at nikula.org" > Perhaps some kind soul could add a wiki entry explaining to people how > to swap the bindings, just in case th

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

2012-01-14 Thread Jani Nikula
Slightly refactor "notmuch reply" recipient and user from address scanning functions in preparation for reply-to-sender feature. Add support for not adding recipients at all (just scan for user from address), and returning the number of recipients added. No externally visible functional changes.

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

2012-01-14 Thread David Bremner
On Sat, 14 Jan 2012 16:46:15 +0200, Jani Nikula 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

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

2012-01-14 Thread Jani Nikula
For those not on IRC: On Sat, 14 Jan 2012 11:31:16 -0400, David Bremner wrote: > This series definitely needs a NEWS item. id:"1326559168-29178-1-git-send-email-j...@nikula.org" > Perhaps some kind soul could add a wiki entry explaining to people how > to swap the bindings, just in case there

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

2012-01-14 Thread David Bremner
On Sat, 14 Jan 2012 16:46:15 +0200, Jani Nikula 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

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

2012-01-14 Thread Jani Nikula
Slightly refactor "notmuch reply" recipient and user from address scanning functions in preparation for reply-to-sender feature. Add support for not adding recipients at all (just scan for user from address), and returning the number of recipients added. No externally visible functional changes.