[PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply"

2012-01-09 Thread Mark Walters
> > The reply-to-thread is a rare case anyway, regardless of reply-to-all or > reply-to-sender, and even the current implementation does not gather all > the recipients from all the messages. Try it out, it seems to me it does > not quite do what you think it does. > > IMHO it should use oldest-f

[PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply"

2012-01-09 Thread Jani Nikula
On Sun, 08 Jan 2012 23:23:15 +, Mark Walters wrote: > > I like this version (of the whole series) but have two queries. (Note I > haven't actually tried it out yet: I have just been reading the code.) > > > + /* Force recipient type in reply-to-sender mode just in case replying to > > +

Re: [PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply"

2012-01-09 Thread Mark Walters
> > The reply-to-thread is a rare case anyway, regardless of reply-to-all or > reply-to-sender, and even the current implementation does not gather all > the recipients from all the messages. Try it out, it seems to me it does > not quite do what you think it does. > > IMHO it should use oldest-f

Re: [PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply"

2012-01-09 Thread Jani Nikula
On Sun, 08 Jan 2012 23:23:15 +, Mark Walters wrote: > > I like this version (of the whole series) but have two queries. (Note I > haven't actually tried it out yet: I have just been reading the code.) > > > + /* Force recipient type in reply-to-sender mode just in case replying to > > +

[PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply"

2012-01-08 Thread Jani Nikula
Add new option --reply-to=(all|sender) to "notmuch reply" to select whether to reply to all (sender and all recipients), or just sender. Reply to all remains the default. Credits to Mark Walters for his similar earlier work where I picked up the basic idea of handling reply-to-sender in add_recip

[PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply"

2012-01-08 Thread Mark Walters
I like this version (of the whole series) but have two queries. (Note I haven't actually tried it out yet: I have just been reading the code.) > + /* Force recipient type in reply-to-sender mode just in case replying to > + * user's own message finds recipients in Cc/Bcc fields only. > +

Re: [PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply"

2012-01-08 Thread Mark Walters
I like this version (of the whole series) but have two queries. (Note I haven't actually tried it out yet: I have just been reading the code.) > + /* Force recipient type in reply-to-sender mode just in case replying to > + * user's own message finds recipients in Cc/Bcc fields only. > +

[PATCH v2 3/6] cli: add support for replying just to the sender in "notmuch reply"

2012-01-08 Thread Jani Nikula
Add new option --reply-to=(all|sender) to "notmuch reply" to select whether to reply to all (sender and all recipients), or just sender. Reply to all remains the default. Credits to Mark Walters for his similar earlier work where I picked up the basic idea of handling reply-to-sender in add_recip