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>
