[PATCH] emacs: tree/show remove duplicate function

David Bremner david at tethera.net
Mon Jul 14 18:07:40 PDT 2014


Mark Walters <markwalters1009 at gmail.com> writes:
> +		   (cond ((eq major-mode 'notmuch-show-mode)
> +			  (notmuch-show-get-message-properties))
> +			 ((eq major-mode 'notmuch-tree-mode)
> +			  (notmuch-tree-get-message-properties))))

I see this already existed, but it looks weird to me to have a two test
cond with no else. Is it intentional to have the code drop through and
do nothing if neither case matches?  It seems like it might be better to
signal an error.

d


More information about the notmuch mailing list