On Fri, 27 Jan 2012 20:36:40 +0400, Dmitry Kurochkin <dmitry.kuroch...@gmail.com> wrote: > On Fri, 27 Jan 2012 09:42:23 +0100, Pieter Praet <pie...@praet.org> wrote: > > On Thu, 26 Jan 2012 14:40:26 +0000, David Edmondson <d...@dme.org> wrote: > > > In general, I like this. > > > > > > - I think that the stash function(s) should take an optional argument > > > specifying the archive to use. That will make testing simpler and also > > > allow people to produce preferred bindings more easily. > > > > > > > Agreed. In fact, apparently that last patch [1] of mine made the test > > suite hang @ emacs:"Stashing in notmuch-show" due to it waiting for > > `completing-read' to finish... Sorry for that. > > > > > - "Message archive: " feels better than "ML Archive: ", but I don't > > > really care. > > > > > > > Agreed. > > > > FWIW I believe "Mailing list archive" would be better. >
Agreed. > Regards, > Dmitry > > > > - Don't base the patch on the thing that I posted, just on master from > > > the repository - no need to make David's life harder. > > > > Seeing as how it was only a minor improvement to your idea (and further > > discussion/correction was probably appropriate), I intended for you to > > squash it into your original patch. > > > > Anyways, patch (relative to master) follows... > > > > > > Peace > > > > -- > > Pieter > > > > [1] id:"1327583610-30085-1-git-send-email-pie...@praet.org" > > _______________________________________________ > > notmuch mailing list > > notmuch@notmuchmail.org > > http://notmuchmail.org/mailman/listinfo/notmuch Peace -- Pieter _______________________________________________ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch