bug#9378: 23.3; iswitchb-make-buflist returns a nonexistent buffer

[Paul Du Bois]

emacs -Q
(require 'iswitchb)
(setq read-buffer-function 'iswitchb-read-buffer)
(add-hook 'minibuffer-setup-hook 'iswitchb-minibuffer-setup)
find several files
C-x b C-k C-k

After the first C-k, the buffer is deleted, but remains at the
head of the iswitchb list.  The second C-k produces an error
"No buffer named <the buffer that was just killed>"

The problem is that iswitchb-make-buflist does not correspond to its
documentation:

  "... If DEFAULT is non-nil, and corresponds to an existing buffer,
   it is put to the start of the list."

Changing the implementation to match the documentation fixes the bug:

--- iswitchb.el 2011-06-16 15:36:18.084715400 -0700
+++ iswitchb2.el        2011-08-25 17:23:04.724373400 -0700
@@ -849,7 +849,7 @@
          (run-hooks 'iswitchb-make-buflist-hook)
         ;; Should this be after the hooks, or should the hooks be the
          ;; final thing to be run?
-         (if default
+         (if (and default (get-buffer default))
              (progn
                (setq iswitchb-temp-buflist
                      (delete default iswitchb-temp-buflist))



In GNU Emacs 23.3.1 (i386-mingw-nt6.1.7601)
 of 2011-03-10 on 3249CTO
Windowing system distributor `Microsoft Corp.', version 6.1.7601
configured using `configure --with-gcc (4.5) --no-opt --cflags
-Ic:/imagesupport/include'