Hi This is looking good: I have two comments the second of which is significant. The first is do you want to sort (alphabetically) the headerline tags? As it stands they are in the order they appear in the thread which is probably not what is wanted. The second is that there is a notmuch-show-tag-all functions to tag all messages in the thread. Your patch is quadratic for the update (as it calculates the list of thread tags once for each message). The attached patch would avoid this and doesn't look too bad. (Note I retained no-headerline-update as an optional argument in case there are out of tree callers, eg users' .emacs files) [Note this is not purely of academic interest: on my test large thread (178 messages) updating the display after tagging all messages took some seconds without my patch and almost no time with it.] Best wishes Mark
On Tue, 11 Dec 2012, Damien Cassou <damien.cassou@gmail.com> wrote: > From: Damien Cassou <damien.cassou@gmail.com> > Subject: [PATCH v4] emacs: display tags in notmuch-show with links > In-Reply-To: > > This patch obsoletes: > id:1355149964-27905-1-git-send-email-damien.cassou@gmail.com > > [PATCH 1/4] emacs: Add a thread's tags to notmuch-show header-line > [PATCH 2/4] emacs: Make tags in notmuch-show header-line clickable > [PATCH 3/4] emacs: Make all tags in `notmuch-show' clickable > [PATCH 4/4] emacs: Add unit-tests for clickable tags > > These patches make clickable all tags that appear in notmuch-show > buffers. Each tag is a link to open a new notmuch-search buffer for > this tag. Additionally, the buffer's header-line now shows the > thread's tags (clickable only if the `header-button' library is loaded > or loadable). > > These patches are the first of an upcoming series whose goal is to > integrate notmuch-labeler into notmuch. See the following for more > details: https://github.com/DamienCassou/notmuch-labeler > > With respect to v3, I took care of the comments you made: > - the header-line now updates when tags are changed > - the tags in the body stays clickable when tags are changed > > Additionally, I added two unit tests to cover the above two comments > and fixed some others unit tests of mine. > _______________________________________________ > notmuch mailing list > notmuch@notmuchmail.org > http://notmuchmail.org/mailman/listinfo/notmuch