[RFC] [PATCH v3] emacs: rewrite `notmuch-cycle-notmuch-buffers'
Pieter Praet
pieter at praet.org
Wed Jan 18 02:04:44 PST 2012
Get rid of the compile-time `cl' requirement for this particular function.
---
emacs/notmuch.el | 40 ++++++++++++++++++++++------------------
1 files changed, 22 insertions(+), 18 deletions(-)
diff --git a/emacs/notmuch.el b/emacs/notmuch.el
index ef4dcc7..2426c0a 100644
--- a/emacs/notmuch.el
+++ b/emacs/notmuch.el
@@ -1084,24 +1084,28 @@ current search results AND that are tagged with the given tag."
If the current buffer is the only notmuch buffer, bury it. If no
notmuch buffers exist, run `notmuch'."
(interactive)
-
- (let (start first)
- ;; If the current buffer is a notmuch buffer, remember it and then
- ;; bury it.
- (when (notmuch-interesting-buffer (current-buffer))
- (setq start (current-buffer))
- (bury-buffer))
-
- ;; Find the first notmuch buffer.
- (setq first (loop for buffer in (buffer-list)
- if (notmuch-interesting-buffer buffer)
- return buffer))
-
- (if first
- ;; If the first one we found is any other than the starting
- ;; buffer, switch to it.
- (unless (eq first start)
- (switch-to-buffer first))
+ (let* ((old-buffer (current-buffer))
+ (next-buffer (first
+ ;; Find interesting buffers:
+ ;; Loop over all existing buffers and return those that
+ ;; satisfy the predicate (`notmuch-interesting-buffer' and
+ ;; not `eq' to current buffer) integrally; All others are
+ ;; replaced with `nil'. Then strip out all `nil's using
+ ;; `delq'. Basically a re-implementation of `cl' package's
+ ;; `delete-if-not'.
+ (delq nil
+ (mapcar (lambda (b)
+ (and (notmuch-interesting-buffer b)
+ (not (eq old-buffer b))
+ b))
+ (buffer-list))))))
+
+ ;; If there's a next buffer, switch to it and bury the
+ ;; original current buffer. Otherwise run `notmuch-hello'.
+ (if next-buffer
+ (progn
+ (switch-to-buffer next-buffer)
+ (bury-buffer old-buffer))
(notmuch))))
(setq mail-user-agent 'notmuch-user-agent)
--
1.7.8.1
More information about the notmuch
mailing list