Commit 47989945 authored by Karl Fogel's avatar Karl Fogel
Browse files

* lisp/bookmark.el (bookmark-bmenu-any-marks): New function

  (bookmark-bmenu-save): Clear buffer modification if no marks.
parent 5ae329f4
2010-01-02 Karl Fogel <>
* lisp/bookmark.el (bookmark-bmenu-marks-width): Define to 1, not 2.
* bookmark.el (bookmark-bmenu-any-marks): New function
(bookmark-bmenu-save): Clear buffer modification if no marks.
2010-01-02 Karl Fogel <>
* bookmark.el (bookmark-bmenu-marks-width): Define to 1, not 2.
(bookmark-bmenu-list, bookmark-bmenu-bookmark): Calculate property
positions by using `bookmark-bmenu-marks-width', instead of hardcoding.
This fixes the `bookmark-bmenu-execute-deletions' bug reported here:
......@@ -1836,13 +1836,40 @@ You can mark bookmarks with the \\<bookmark-bmenu-mode-map>\\[bookmark-bmenu-mar
(other-window 1))))
(defun bookmark-bmenu-any-marks ()
"Return non-nil if any bookmarks are marked in the marks column."
(goto-char (point-min))
(catch 'found-mark
(while (not (eobp))
(if (looking-at "^\\S-")
(throw 'found-mark t)
(forward-line 1)))
(defun bookmark-bmenu-save (parg)
"Save the current list into a bookmark file.
With a prefix arg, prompts for a file to save them in."
(interactive "P")
(bookmark-save parg))))
(bookmark-save parg)))
;; Show the buffer as unmodified after saving, but only if there are
;; no marks: marks are not saved with the bookmarks, therefore from
;; the user's point of view they are a "modification" in the buffer
;; FIXME: Ideally, if the buffer were unmodified when there are no
;; marks, and then some marks are made and removed without being
;; executed, then the buffer would be restored to unmodified state.
;; But that would require bookmark-specific logic to track buffer
;; modification. It might be worth it, but it's fine not to have it
;; too -- the worst outcome is that the user might be tempted to
;; save the bookmark list when it technically doesn't need saving.
(if (not (bookmark-bmenu-any-marks))
(set-buffer-modified-p nil)))
(defun bookmark-bmenu-load ()
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment