Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'.

Subject: Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'.

Date: Mon, 19 Dec 2011 12:58:53 -0500

To: David Edmondson, notmuch@notmuchmail.org

Cc:

From: Aaron Ecay

(Please excuse the lack of inline comments on the patch – the original
patch email is so old that I had deleted it from my archives!)

progn...set-buffer should be with-current-buffer
or...eq...eq... would be cleaner as (memq major-mode '(foo bar baz))

Otherwise, LGTM

-- 
Aaron Ecay

• Previous message (by thread): Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'.

Thread:

• David Edmondson—[PATCH] emacs: Add `notmuch-jump-to-recent-buffer'. [inbox, notmuch::obsolete, notmuch::patch, unread]
  • David Edmondson—Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'. [inbox, unread]
    • Aaron Ecay—Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'. [inbox, unread]
      • David Edmondson—Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'. [inbox, unread]
  • Tomi Ollila—Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'. [inbox, unread]
    • Aaron Ecay—Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'. [inbox, unread]
    • David Edmondson—Re: [PATCH] emacs: Add `notmuch-jump-to-recent-buffer'. [inbox, unread]