GNU bug report logs - #75953
30.0.92; Bookmark fringe indicator not removed

Previous Next

Full log

View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Sergio Pastor Pérez <sergio.pastorperez <at> outlook.es>
Cc: 75953 <at> debbugs.gnu.org
Subject: bug#75953: 30.0.92; Bookmark fringe indicator not removed
Date: Fri, 31 Jan 2025 10:03:12 +0200

> Date: Thu, 30 Jan 2025 22:04:20 +0100
> From: Sergio Pastor Pérez <sergio.pastorperez <at> outlook.es>
> 
> Hello.
> 
> I'm sending this bug report as a continuation of this closed report 
> bug#51233.
> 
> The fringe indicator for bookmarks is correctly set, but the procedure 
> `bookmark-delete-all` does not delete the indicator.
> 
> Steps for reproduction:
> 
> 1. Run emacs -q
> 
> 2. Create a new file, say /tmp/test.txt
> 
> 3. Ensure bookmark-set-fringe-mark is t (should be by default)
> 
> 4. Add some text and bookmark the line: a fringe indicator should appear
>     at the left
> 
> 5. Call `bookmark-delete-all` to delete the bookmark: the fringe 
> indicator is still there

Does the patch below give good results?

diff --git a/lisp/bookmark.el b/lisp/bookmark.el
index cd59293..153646c 100644
--- a/lisp/bookmark.el
+++ b/lisp/bookmark.el
@@ -1584,6 +1584,8 @@ bookmark-delete-all
   (when (or no-confirm
             (yes-or-no-p "Permanently delete all bookmarks? "))
     (bookmark-maybe-load-default-file)
+    (dolist (bm bookmark-alist)
+      (bookmark--remove-fringe-mark bm))
     (setq bookmark-alist-modification-count
           (+ bookmark-alist-modification-count (length bookmark-alist)))
     (setq bookmark-alist nil)

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.