[PATCH 2/4] emacs: repurpose notmuch-show-archive-thread-internal function for general thread tagging

[Jameson Graef Rollins]

On Wed, 11 Jan 2012 00:53:35 -0500, Aaron Ecay <aaronecay at gmail.com> wrote:
> On Tue, 10 Jan 2012 18:56:29 -0800, Jameson Graef Rollins <jrollins at finestructure.net> wrote:
> > Actually, the show-next argument was already part of the function.  I
> > did not introduce it.  And it wasn't optional originally, so if we want
> > to change that behavior we should probably do so in a separate patch.
> 
> Hrm.  I didn’t communicate as clearly as I could have – you are correct
> that the show-next argument to the notmuch-show-archive-internal function
> was pre-existing.  But notmuch-show-tag-thread-internal is a new function,
> with potentially expanded usefulness to third-party code.  Thus I think
> I’m in the clear to bikeshed about its calling convention.  :) It’s your
> patch, though, so it’s your call if you feel that the &optional goes best
> in a new change.

No, I see your point, Aaron.  I'll rework it to make it a little clean
and more flexible when I resend.  Thanks again.

jamie.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/57bde9d1/attachment.pgp>