Leo <g...@relevant-information.com> writes: > From: Leo Okawa Ericson <g...@relevant-information.com> > > Having notmuch-show-next-thread return non-nil on success and nil on > failure makes it easier for users to interact with notmuch via elisp.
You talk about notmuch-show-next-thread, but you seem to be modifying notmuch-search-show-thread. That actually seems like a bit of a strange function to invoke programatically, since it deduces the argument from the current buffer. It might be better for your code to call notmuch-show directly, since notmuch-search-show-thread is pretty tied to the internals of notmuch-search-mode. In any case: If you change the API to have a meaningful return value, please update the docstring for the function We generally need a test for any new feature, and this is arguably a new feature. If you need help constructing a test, feel free to ask. There are several examples in test/T*emacs*.sh. _______________________________________________ notmuch mailing list -- notmuch@notmuchmail.org To unsubscribe send an email to notmuch-le...@notmuchmail.org
