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

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

[debbugs-tracker] bug#19335: closed (24.4.51; ido.el: Include bookmark-f


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#19335: closed (24.4.51; ido.el: Include bookmark-files as virtual buffers)
Date: Thu, 12 Mar 2015 19:04:01 +0000

Your message dated Thu, 12 Mar 2015 15:03:18 -0400
with message-id <address@hidden>
and subject line Re: bug#19335: 24.4.51; ido.el: Include bookmark-files as 
virtual buffers
has caused the debbugs.gnu.org bug report #19335,
regarding 24.4.51; ido.el: Include bookmark-files as virtual buffers
to be marked as done.

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


-- 
19335: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19335
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 24.4.51; ido.el: Include bookmark-files as virtual buffers Date: Wed, 10 Dec 2014 14:00:57 +0100 User-agent: Gnus/5.130012 (Ma Gnus v0.12) Emacs/24.4.51 (gnu/linux)
Hi,

The docstring of ido-add-virtual-buffers-to-list reads:

"Add recently visited files, and bookmark files, to the buffer list.
This is to make them appear as if they were \"virtual buffers\"."

But bookmarked files (C-x r l) are not in fact included (at least on my
system, and if they are it's not obvious how).  Perhaps I'm misreading the
docstring (since it says "bookmark files").  In any case, it would be nice
to include bookmarked files as virtual files (they are bookmarked and
therefore important).

I can add a defcustom to not include bookmarked files, if that's better.

Thanks,
Rasmus

-- 
And when I’m finished thinking, I have to die a lot
>From da38e2da2c13eee221e2530514df842fb471465c Mon Sep 17 00:00:00 2001
From: rasmus <address@hidden>
Date: Wed, 10 Dec 2014 13:49:42 +0100
Subject: [PATCH] ido.el: Include bookmark-files as virtual buffers

* ido.el (ido-add-virtual-buffers-to-list): Include bookmarked files.

The docstring of ido-add-virtual-buffers-to-list suggests that
bookmarked files should include, but this does not in fact seem to
happen.
---
 lisp/ChangeLog | 4 ++++
 lisp/ido.el    | 8 +++++++-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 454ee00..6b2c7f7 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,7 @@
+2014-12-10  Rasmus Pank Roulund  <address@hidden>
+
+       * ido.el (ido-add-virtual-buffers-to-list): Include bookmarked files.
+
 2014-12-10  Andreas Schwab  <address@hidden>

        * files.el (file-tree-walk): Use file-name-as-directory
diff --git a/lisp/ido.el b/lisp/ido.el
index 5f7637c..f7c3f5f 100644
--- a/lisp/ido.el
+++ b/lisp/ido.el
@@ -3482,7 +3482,13 @@ This is to make them appear as if they were \"virtual 
buffers\"."
   (unless recentf-mode (recentf-mode 1))
   (setq ido-virtual-buffers nil)
   (let (name)
-    (dolist (head recentf-list)
+    (dolist (head (append recentf-list
+                          (and (boundp 'bookmark-alist)
+                               bookmark-alist
+                               (mapcar
+                                (lambda (bookmark)
+                                  (cdr-safe (assoc 'filename bookmark)))
+                                bookmark-alist))))
       (setq name (file-name-nondirectory head))
       ;; In case HEAD is a directory with trailing /.  See bug#14552.
       (when (equal name "")
--
2.1.3

--- End Message ---
--- Begin Message --- Subject: Re: bug#19335: 24.4.51; ido.el: Include bookmark-files as virtual buffers Date: Thu, 12 Mar 2015 15:03:18 -0400 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux)
> The attached patch should merge against master.

Thanks, installed,


        Stefan


--- End Message ---

reply via email to

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