branch: externals/bufferlo commit e4ab6444f29dc82178d9c232263c5af0a857ec9c Author: shipmints <shipmi...@gmail.com> Commit: shipmints <shipmi...@gmail.com>
metadata typo. bookmark advice refinement. --- bufferlo.el | 24 +++++++++++++----------- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/bufferlo.el b/bufferlo.el index e8679c7949..16b83567e7 100644 --- a/bufferlo.el +++ b/bufferlo.el @@ -1967,7 +1967,7 @@ current or new frame according to (lambda (str pred flag) (pcase flag ('metadata - (metadata (category . bookmark))) + '(metadata (category . bookmark))) (_ (all-completions str abm-names pred))))) abm-names nil nil)) @@ -2084,28 +2084,30 @@ raised." ;; (defun bookmark-rename (old-name &optional new-name) (defun bufferlo--bookmark-rename-advice (oldfn &optional old-name new-name) - "`bookmark-rename' advice to prevent renaming active bufferlo bookmarks." + "`bookmark-rename' advice to prevent renaming active bufferlo bookmarks. +OLDFN OLD-NAME NEW-NAME" (interactive) - (if (called-interactively-p 'any) + (if (called-interactively-p 'interactive) (setq old-name (bookmark-completing-read "Old bookmark name"))) (if-let ((abm (assoc old-name (bufferlo--active-bookmarks)))) - (error "%s is an active bufferlo bookmark. Close its frame/tab, or clear it before renaming." old-name) - (if (called-interactively-p 'any) + (error "%s is an active bufferlo bookmark. Close its frame/tab, or clear it before renaming" old-name) + (if (called-interactively-p 'interactive) (funcall-interactively oldfn old-name new-name) - (apply oldfn old-name new-name)))) + (funcall oldfn old-name new-name)))) ;; (defun bookmark-delete (bookmark-name &optional batch) (defun bufferlo--bookmark-delete-advice (oldfn &optional bookmark-name batch) - "`bookmark-delete' advice to prevent deleting active bufferlo bookmarks." + "`bookmark-delete' advice to prevent deleting active bufferlo bookmarks. +OLDFN BOOKMARK-NAME BATCH" (interactive) - (if (called-interactively-p 'any) + (if (called-interactively-p 'interactive) (setq bookmark-name (bookmark-completing-read "Delete bookmark" bookmark-current-bookmark))) (if-let ((abm (assoc bookmark-name (bufferlo--active-bookmarks)))) - (error "%s is an active bufferlo bookmark. Close its frame/tab, or clear it before deleting." bookmark-name) - (if (called-interactively-p 'any) + (error "%s is an active bufferlo bookmark. Close its frame/tab, or clear it before deleting" bookmark-name) + (if (called-interactively-p 'interactive) (funcall-interactively oldfn bookmark-name batch) - (apply oldfn bookmark-name batch)))) + (funcall oldfn bookmark-name batch)))) ;; (defun bookmark-delete-all (&optional no-confirm) ;; Leave this alone for now. It does prompt for confirmation.