|
From: | Bozhidar Batsov |
Subject: | Re: Condider adding buffer-mode |
Date: | Wed, 30 Oct 2013 11:34:11 +0200 |
> Here's some real examples:Both uses of this are in something of the form
> (defun cider-util--clojure-buffers ()
(dolist (buffer (cider-util--clojure-buffers))
(with-current-buffer buffer
so overall, it's exactly the example I provided. I.e. you can just
throw away cider-util--clojure-buffers and replace the two uses with
(dolist (buffer (buffer-list))
(with-current-buffer buffer
(when (derived-mode-p 'closure-mode)
which also fixes the bug in that code when the user uses a mode derived
from closure-mode.
> (when (and buffer
> (eq 'clojure-mode (with-current-buffer buffer major-mode))
> (eq 'cider-repl-mode major-mode))
> (setq cider-last-clojure-buffer buffer)))
> vs
> (when (and buffervs
> (eq 'clojure-mode (buffer-mode buffer))
> (eq 'cider-repl-mode major-mode))
> (setq cider-last-clojure-buffer buffer)))
(when (and buffer
(with-current-buffer buffer (derived-mode-p 'clojure-mode))
(derived-mode-p 'cider-repl-mode))
(setq cider-last-clojure-buffer buffer)))Very marginally so.
> Seems to me `buffer-mode' makes the code clearer, but that's subjective (as
> most things in life :-) )
Stefan
[Prev in Thread] | Current Thread | [Next in Thread] |