emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Emacs-bug-tracker] bug#7231: closed (23.3; Don't rebuild buffer list in


From: GNU bug Tracking System
Subject: [Emacs-bug-tracker] bug#7231: closed (23.3; Don't rebuild buffer list in iswitchb-visit-buffer)
Date: Thu, 28 Oct 2010 01:29:02 +0000

Your message dated Wed, 27 Oct 2010 21:31:01 -0400
with message-id <address@hidden>
and subject line Re: bug#7231: 23.3; Don't rebuild buffer list in 
iswitchb-visit-buffer
has caused the GNU bug report #7231,
regarding 23.3; Don't rebuild buffer list in iswitchb-visit-buffer
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
7231: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=7231
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 23.3; Don't rebuild buffer list in iswitchb-visit-buffer Date: Mon, 18 Oct 2010 01:59:52 +0800
The original change was to address buffer name changes due to packages
such as uniquify.el. But it causes another annoying bug: changing the
order of matches seen by users.

This reverts it.

If people are annoyed (which I doubt) by outdated buffer names due to
uniquify.el, one solution is to map the buffer-names to buffer objects
before killing and map them back to names after killing.

>From 7e6597c54a7764688855c3ab2efa6cfa1cffbea6 Mon Sep 17 00:00:00 2001
Date: Mon, 18 Oct 2010 01:44:24 +0800
Subject: [PATCH] Don't rebuild buffer list in iswitchb-visit-buffer

Rebuilding buffer list will lose the order of matches seen by users
and thus cause surprises.
---
 lisp/iswitchb.el |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/lisp/iswitchb.el b/lisp/iswitchb.el
index 081897a..b7baa03 100644
--- a/lisp/iswitchb.el
+++ b/lisp/iswitchb.el
@@ -1042,10 +1042,8 @@ Return the modified list with the last element prepended 
to it."
          (if (get-buffer buf)
              ;; buffer couldn't be killed.
              (setq iswitchb-rescan t)
-           ;; Else `kill-buffer' succeeds so re-make the buffer list
-           ;; taking into account packages like uniquify may rename
-           ;; buffers
-           (iswitchb-make-buflist iswitchb-default))))))
+           ;; else buffer was killed so remove name from list.
+           (setq iswitchb-buflist  (delq buf iswitchb-buflist)))))))
 
 ;;; VISIT CHOSEN BUFFER
 (defun iswitchb-visit-buffer (buffer)
-- 
1.7.3




--- End Message ---
--- Begin Message --- Subject: Re: bug#7231: 23.3; Don't rebuild buffer list in iswitchb-visit-buffer Date: Wed, 27 Oct 2010 21:31:01 -0400 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux)
> Sorry for being too lazy. Please try the following patch.

Thanks, installed in the trunk.


        Stefan


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]