On Mon, 26 Apr 2010 11:43:45 +0100, David Edmondson <dme at dme.org> wrote: > On Fri, 23 Apr 2010 15:52:15 -0700, Dirk Hohndel <hohndel at infradead.org> > wrote: > > Given how little elisp I know I'm quite interested in feedback > > and better implementations > > I think that: > > (defun notmuch-show-forward-message () > "Forward the current message." > (interactive) > (let ((user-mail-address > (shell-command-to-string (concat notmuch-command > "guess-from" > (notmuch-show-get-message-id))))) > (with-current-notmuch-show-message > (notmuch-mua-forward-message)))) > > is more idiomatic.
Thanks - I was sure that my lack of elisp foo was showing (even though I was quite proud that I was able to get it to work at all). I'll submit an updated patch for the 0.4 merge window. /D -- Dirk Hohndel Intel Open Source Technology Center