Commit 175d0960 authored by Stefan Monnier's avatar Stefan Monnier
Browse files

(bookmark-completing-read): Use a popup menu if activated from the mouse.

Return a string, instead of a list of one string.
(bookmark-edit-annotation): Remove unused vars.
(bookmark-jump, bookmark-relocate, bookmark-insert-location)
(bookmark-rename, bookmark-insert, bookmark-delete): Adjust calls
to bookmark-completing-read.
(bookmark-bmenu-show-filenames, bookmark-bmenu-hide-filenames)
(bookmark-bmenu-mark, bookmark-bmenu-select, bookmark-bmenu-unmark)
(bookmark-bmenu-delete, bookmark-bmenu-list): Use inhibit-read-only
and erase-buffer.
(bookmark-menu-delete, bookmark-menu-rename, bookmark-menu-locate)
(bookmark-menu-jump, bookmark-menu-insert)
(bookmark-popup-menu-and-apply-function)
(bookmark-menu-popup-paned-bookmark-menu): Remove.
(bookmark-menu-build-paned-menu): Remove by folding it into
bookmark-menu-popup-paned-menu.
(menu-bar-bookmark-map): Move the define-key statements here.
Use the "non-menu" commands since they now pop up a menu if needed.
(bookmark-exit-hook-internal): Simplify.
parent 3666daf6
2003-08-11 Stefan Monnier <monnier@cs.yale.edu>
* bookmark.el (bookmark-completing-read):
Return a string, instead of a list of one string.
Use a popup menu if activated from the mouse.
(bookmark-edit-annotation): Remove unused vars.
(bookmark-jump, bookmark-relocate, bookmark-insert-location)
(bookmark-rename, bookmark-insert, bookmark-delete): Adjust calls
to bookmark-completing-read.
(bookmark-bmenu-show-filenames, bookmark-bmenu-hide-filenames)
(bookmark-bmenu-mark, bookmark-bmenu-select, bookmark-bmenu-unmark)
(bookmark-bmenu-delete, bookmark-bmenu-list): Use inhibit-read-only
and erase-buffer.
(bookmark-menu-delete, bookmark-menu-rename, bookmark-menu-locate)
(bookmark-menu-jump, bookmark-menu-insert)
(bookmark-popup-menu-and-apply-function)
(bookmark-menu-popup-paned-bookmark-menu): Remove.
(bookmark-menu-build-paned-menu): Remove by folding it into
bookmark-menu-popup-paned-menu.
(menu-bar-bookmark-map): Move the define-key statements here.
Use the "non-menu" commands since they now pop up a menu if needed.
(bookmark-exit-hook-internal): Simplify.
2003-08-11 Carsten Dominik <dominik@sand.science.uva.nl>
 
* textmodes/reftex.el: Updated to version 4.21
* reftex-auc.el: likewise
* reftex-cite.el: likewise
* reftex-dcr.el: likewise
* reftex-global.el: likewise
* reftex-index.el: likewise
* reftex-parse.el: likewise
* reftex-ref.el: likewise
* reftex-sel.el: likewise
* reftex-toc.el: likewise
* reftex-vars.el: likewise
* reftex-auc.el: likewise.
* reftex-cite.el: likewise.
* reftex-dcr.el: likewise.
* reftex-global.el: likewise.
* reftex-index.el: likewise.
* reftex-parse.el: likewise.
* reftex-ref.el: likewise.
* reftex-sel.el: likewise.
* reftex-toc.el: likewise.
* reftex-vars.el: likewise.
 
2003-08-08 Vinicius Jose Latorre <viniciusjl@ig.com.br>
 
......@@ -21,13 +44,12 @@
 
* progmodes/ebnf2ps.el (ebnf-begin-job): Code fix.
 
2003-08-06 Glenn Morris <gmorris@ast.cam.ac.uk>
 
* calendar/calendar.el (list-diary-entries-hook)
(diary-display-hook, nongregorian-diary-listing-hook)
(mark-diary-entries-hook, nongregorian-diary-marking-hook): Add
some customize options for these hooks.
(mark-diary-entries-hook, nongregorian-diary-marking-hook):
Add some customize options for these hooks.
(calendar-abbrev-construct): Don't try to take a substring longer
than the original string.
 
......@@ -37,7 +59,7 @@
(testcover-reinstrument-clauses): Doc fix.
 
* emacs-lisp/warnings.el: Doc fixes, args renamed.
(warning-type-format): Renamed from warning-group-format.
(warning-type-format): Rename from warning-group-format.
 
* emacs-lisp/bytecomp.el (byte-compile-not-obsolete-var): New var.
(byte-compile-variable-ref): Handle byte-compile-not-obsolete-var.
......@@ -64,8 +86,8 @@
 
* international/code-pages.el: Don't require mule-diag.
 
* international/mule-diag.el (non-iso-charset-alist): Add
autoload cookie.
* international/mule-diag.el (non-iso-charset-alist):
Add autoload cookie.
 
* language/devan-util.el (dev-glyph-order): Add an entry for the
glyph code #xC4.
......@@ -87,8 +109,7 @@
`calendar-day-name' and `calendar-month-name' functions.
 
* calendar/diary-lib.el (list-diary-entries): Adapt for new
behaviour of `calendar-day-name' and `calendar-month-name'
functions.
behaviour of `calendar-day-name' and `calendar-month-name' functions.
(diary-name-pattern): Use abbrev arrays, rather than fixing
abbrevs at three chars. Calling syntax change.
(mark-diary-entries): Adapt for new behaviours of
......@@ -109,8 +130,8 @@
(mark-hebrew-diary-entries): Adapt for new behaviours of
`diary-name-pattern' and `calendar-make-alist' functions.
 
* calendar/cal-islam.el (calendar-islamic-month-name-array): Add
doc string.
* calendar/cal-islam.el (calendar-islamic-month-name-array):
Add doc string.
(list-islamic-diary-entries): Adapt for new behaviours of
`calendar-day-name' and `add-to-diary-list' functions.
(mark-islamic-diary-entries): Adapt for new behaviours of
......@@ -153,8 +174,8 @@
(c-search-decl-header-end): Don't trip up on operator identifiers
in C++ and operators like == in all languages.
 
* progmodes/cc-engine.el (c-backward-to-decl-anchor): Detect
leading labels correctly.
* progmodes/cc-engine.el (c-backward-to-decl-anchor):
Detect leading labels correctly.
 
2003-08-02 Andreas Schwab <schwab@suse.de>
 
......@@ -180,8 +201,8 @@
 
2003-07-30 Kenichi Handa <handa@m17n.org>
 
* international/fontset.el (setup-default-fontset): Change
registry names of Akurti fonts.
* international/fontset.el (setup-default-fontset):
Change registry names of Akurti fonts.
 
2003-07-29 Jesper Harder <harder@ifa.au.dk> (tiny change)
 
......@@ -194,8 +215,8 @@
 
2003-07-28 Nick Roberts <nick@nick.uklinux.net>
 
* gdb-ui.el (gdb-setup-windows, gdb-restore-windows): Restore
assembler in source window if that is what has been selected.
* gdb-ui.el (gdb-setup-windows, gdb-restore-windows):
Restore assembler in source window if that is what has been selected.
(menu): Add gdb-restore-windows to menu. Make gdba
specific menus only visible from gdba.
 
......@@ -228,8 +249,7 @@
 
2003-07-26 Markus Rost <rost@math.ohio-state.edu>
 
* international/quail.el (quail-translate-key): Fix previous
change.
* international/quail.el (quail-translate-key): Fix previous change.
 
2003-07-25 John Paul Wallington <jpw@gnu.org>
 
......
......@@ -456,6 +456,8 @@ probably don't want to include one yourself.
Optional second arg DEFAULT is a string to return if the user enters
the empty string."
(bookmark-maybe-load-default-file) ; paranoia
(if (listp last-nonmenu-event)
(bookmark-menu-popup-paned-menu t prompt (bookmark-all-names))
(let* ((completion-ignore-case bookmark-completion-ignore-case)
(default default)
(prompt (if default
......@@ -468,9 +470,7 @@ the empty string."
0
nil
'bookmark-history)))
(if (string-equal "" str)
(list default)
(list str))))
(if (string-equal "" str) default str))))
(defmacro bookmark-maybe-historicize-string (string)
......@@ -924,10 +924,8 @@ When you have finished composing, type \\[bookmark-send-annotation].
(defun bookmark-edit-annotation (bookmark)
"Pop up a buffer for editing bookmark BOOKMARK's annotation."
(let ((buf (current-buffer))
(point (point)))
(pop-to-buffer (generate-new-buffer-name "*Bookmark Annotation Compose*"))
(bookmark-edit-annotation-mode bookmark)))
(bookmark-edit-annotation-mode bookmark))
(defun bookmark-insert-current-bookmark ()
......@@ -1046,7 +1044,8 @@ if you wish to give the bookmark a new location, and bookmark-jump
will then jump to the new location, as well as recording it in place
of the old one in the permanent bookmark record."
(interactive
(bookmark-completing-read "Jump to bookmark" bookmark-current-bookmark))
(list (bookmark-completing-read "Jump to bookmark"
bookmark-current-bookmark)))
(bookmark-maybe-historicize-string bookmark)
(let ((cell (bookmark-jump-noselect bookmark)))
(and cell
......@@ -1139,7 +1138,7 @@ be retrieved from a VC backend, else return nil."
This makes an already existing bookmark point to that file, instead of
the one it used to point at. Useful when a file has been renamed
after a bookmark was set in it."
(interactive (bookmark-completing-read "Bookmark to relocate"))
(interactive (list (bookmark-completing-read "Bookmark to relocate")))
(bookmark-maybe-historicize-string bookmark)
(bookmark-maybe-load-default-file)
(let* ((bmrk-filename (bookmark-get-filename bookmark))
......@@ -1156,7 +1155,7 @@ after a bookmark was set in it."
"Insert the name of the file associated with BOOKMARK.
Optional second arg NO-HISTORY means don't record this in the
minibuffer history list `bookmark-history'."
(interactive (bookmark-completing-read "Insert bookmark location"))
(interactive (list (bookmark-completing-read "Insert bookmark location")))
(or no-history (bookmark-maybe-historicize-string bookmark))
(let ((start (point)))
(prog1
......@@ -1191,7 +1190,7 @@ must pass at least OLD when calling from Lisp.
While you are entering the new name, consecutive C-w's insert
consecutive words from the text of the buffer into the new bookmark
name."
(interactive (bookmark-completing-read "Old bookmark name"))
(interactive (list (bookmark-completing-read "Old bookmark name")))
(bookmark-maybe-historicize-string old)
(bookmark-maybe-load-default-file)
......@@ -1224,7 +1223,7 @@ You may have a problem using this function if the value of variable
`bookmark-alist' is nil. If that happens, you need to load in some
bookmarks. See help on function `bookmark-load' for more about
this."
(interactive (bookmark-completing-read "Insert bookmark contents"))
(interactive (list (bookmark-completing-read "Insert bookmark contents")))
(bookmark-maybe-historicize-string bookmark)
(bookmark-maybe-load-default-file)
(let ((orig-point (point))
......@@ -1247,7 +1246,8 @@ one most recently used in this file, if any\).
Optional second arg BATCH means don't update the bookmark list buffer,
probably because we were called from there."
(interactive
(bookmark-completing-read "Delete bookmark" bookmark-current-bookmark))
(list (bookmark-completing-read "Delete bookmark"
bookmark-current-bookmark)))
(bookmark-maybe-historicize-string bookmark)
(bookmark-maybe-load-default-file)
(let ((will-go (bookmark-get-bookmark bookmark)))
......@@ -1533,9 +1533,8 @@ deletion, or > if it is flagged for displaying."
(if (interactive-p)
(switch-to-buffer (get-buffer-create "*Bookmark List*"))
(set-buffer (get-buffer-create "*Bookmark List*")))
(let ((buffer-read-only nil))
(delete-region (point-max) (point-min))
(goto-char (point-min)) ;sure are playing it safe...
(let ((inhibit-read-only t))
(erase-buffer)
(insert "% Bookmark\n- --------\n")
(bookmark-maybe-sort-alist)
(mapcar
......@@ -1638,7 +1637,7 @@ Optional argument SHOW means show them unconditionally."
(goto-char (point-min))
(forward-line 2)
(setq bookmark-bmenu-hidden-bookmarks ())
(let ((buffer-read-only nil))
(let ((inhibit-read-only t))
(while (< (point) (point-max))
(let ((bmrk (bookmark-bmenu-bookmark)))
(setq bookmark-bmenu-hidden-bookmarks
......@@ -1671,7 +1670,7 @@ Optional argument SHOW means show them unconditionally."
(backward-word 1)
(setq bookmark-bmenu-bookmark-column (current-column)))
(save-excursion
(let ((buffer-read-only nil))
(let ((inhibit-read-only t))
(while bookmark-bmenu-hidden-bookmarks
(move-to-column bookmark-bmenu-bookmark-column t)
(bookmark-kill-line)
......@@ -1774,7 +1773,7 @@ if an annotation exists."
(interactive)
(beginning-of-line)
(if (bookmark-bmenu-check-position)
(let ((buffer-read-only nil))
(let ((inhibit-read-only t))
(delete-char 1)
(insert ?>)
(forward-line 1)
......@@ -1793,7 +1792,7 @@ You can mark bookmarks with the \\<bookmark-bmenu-mode-map>\\[bookmark-bmenu-mar
(goto-char (point-min))
(while (re-search-forward "^>" nil t)
(setq tem (bookmark-bmenu-bookmark))
(let ((buffer-read-only nil))
(let ((inhibit-read-only t))
(delete-char -1)
(insert ?\ ))
(or (string-equal tem bmrk)
......@@ -1940,7 +1939,7 @@ Optional BACKUP means move up."
(beginning-of-line)
(if (bookmark-bmenu-check-position)
(progn
(let ((buffer-read-only nil))
(let ((inhibit-read-only t))
(delete-char 1)
;; any flags to reset according to circumstances? How about a
;; flag indicating whether this bookmark is being visited?
......@@ -1967,7 +1966,7 @@ To carry out the deletions that you've marked, use \\<bookmark-bmenu-mode-map>\\
(interactive)
(beginning-of-line)
(if (bookmark-bmenu-check-position)
(let ((buffer-read-only nil))
(let ((inhibit-read-only t))
(delete-char 1)
(insert ?D)
(forward-line 1)
......@@ -2047,149 +2046,38 @@ To carry out the deletions that you've marked, use \\<bookmark-bmenu-mode-map>\\
;;; Menu bar stuff. Prefix is "bookmark-menu".
(defun bookmark-menu-build-paned-menu (name entries)
"Build a multi-paned menu named NAME from the strings in ENTRIES.
(defun bookmark-menu-popup-paned-menu (event name entries)
"Pop up multi-paned menu at EVENT, return string chosen from ENTRIES.
That is, ENTRIES is a list of strings which appear as the choices
in the menu. The number of panes depends on the number of entries.
in the menu.
The number of panes depends on the number of entries.
The visible entries are truncated to `bookmark-menu-length', but the
strings returned are not."
(let* ((f-height (/ (frame-height) 2))
(pane-list
(let (temp-pane-list
(let ((f-height (/ (frame-height) 2))
(pane-list nil)
(iter 0))
(while entries
(let (lst
(count 0))
(while (and (< count f-height) entries)
(let ((str (car entries)))
(setq lst (cons
(cons
(push (cons
(if (> (length str) bookmark-menu-length)
(substring str 0 bookmark-menu-length)
str)
str)
lst))
lst)
(setq entries (cdr entries))
(setq count (1+ count))))
(setq iter (1+ iter))
(setq
temp-pane-list
(cons
(cons
(push (cons
(format "-*- %s (%d) -*-" name iter)
(nreverse lst))
temp-pane-list))))
(nreverse temp-pane-list))))
;; Return the menu:
(cons (concat "-*- " name " -*-") pane-list)))
(defun bookmark-menu-popup-paned-menu (event name entries)
"Pop up multi-paned menu at EVENT, return string chosen from ENTRIES.
That is, ENTRIES is a list of strings which appear as the choices
in the menu.
The number of panes depends on the number of entries."
(interactive "e")
(x-popup-menu event (bookmark-menu-build-paned-menu name entries)))
(defun bookmark-menu-popup-paned-bookmark-menu (event name)
"Pop up menu of bookmarks, return chosen bookmark.
Pop up at EVENT, menu's name is NAME.
The number of panes depends on the number of bookmarks."
(bookmark-menu-popup-paned-menu event name (bookmark-all-names)))
(defun bookmark-popup-menu-and-apply-function (func-sym menu-label event)
;; help function for making menus that need to apply a bookmark
;; function to a string.
(let* ((choice (bookmark-menu-popup-paned-bookmark-menu
event menu-label)))
(if choice (apply func-sym (list choice)))))
;;;###autoload
(defun bookmark-menu-insert (event)
"Insert the text of the file pointed to by bookmark BOOKMARK.
You may have a problem using this function if the value of variable
`bookmark-alist' is nil. If that happens, you need to load in some
bookmarks. See help on function `bookmark-load' for more about
this.
Warning: this function only takes an EVENT as argument. Use the
corresponding bookmark function from Lisp \(the one without the
\"-menu-\" in its name\)."
(interactive "e")
(bookmark-popup-menu-and-apply-function
'bookmark-insert "Insert Bookmark Contents" event))
;;;###autoload
(defun bookmark-menu-jump (event)
"Jump to bookmark BOOKMARK (a point in some file).
You may have a problem using this function if the value of variable
`bookmark-alist' is nil. If that happens, you need to load in some
bookmarks. See help on function `bookmark-load' for more about
this.
Warning: this function only takes an EVENT as argument. Use the
corresponding bookmark function from Lisp \(the one without the
\"-menu-\" in its name\)."
(interactive "e")
(bookmark-popup-menu-and-apply-function
'bookmark-jump "Jump to Bookmark" event))
;;;###autoload
(defun bookmark-menu-locate (event)
"Insert the name of the file associated with BOOKMARK.
\(This is not the same as the contents of that file\).
Warning: this function only takes an EVENT as argument. Use the
corresponding bookmark function from Lisp \(the one without the
\"-menu-\" in its name\)."
(interactive "e")
(bookmark-popup-menu-and-apply-function
'bookmark-insert-location "Insert Bookmark Location" event))
;;;###autoload
(defun bookmark-menu-rename (event)
"Change the name of OLD-BOOKMARK to NEWNAME.
If called from keyboard, prompts for OLD-BOOKMARK and NEWNAME.
If called from menubar, OLD-BOOKMARK is selected from a menu, and
prompts for NEWNAME.
If called from Lisp, prompts for NEWNAME if only OLD-BOOKMARK was
passed as an argument. If called with two strings, then no prompting
is done. You must pass at least OLD-BOOKMARK when calling from Lisp.
While you are entering the new name, consecutive C-w's insert
consecutive words from the text of the buffer into the new bookmark
name.
Warning: this function only takes an EVENT as argument. Use the
corresponding bookmark function from Lisp \(the one without the
\"-menu-\" in its name\)."
(interactive "e")
(bookmark-popup-menu-and-apply-function
'bookmark-rename "Rename Bookmark" event))
pane-list)))
;;;###autoload
(defun bookmark-menu-delete (event)
"Delete the bookmark named NAME from the bookmark list.
Removes only the first instance of a bookmark with that name. If
there are one or more other bookmarks with the same name, they will
not be deleted. Defaults to the \"current\" bookmark \(that is, the
one most recently used in this file, if any\).
Warning: this function only takes an EVENT as argument. Use the
corresponding bookmark function from Lisp \(the one without the
\"-menu-\" in its name\)."
(interactive "e")
(bookmark-popup-menu-and-apply-function
'bookmark-delete "Delete Bookmark" event))
;; Popup the menu and return the string.
(x-popup-menu event (cons (concat "-*- " name " -*-")
(nreverse pane-list)))))
;; Thanks to Roland McGrath for fixing menubar.el so that the
......@@ -2201,10 +2089,22 @@ corresponding bookmark function from Lisp \(the one without the
;; Emacs menubar stuff.
;;;###autoload
(defvar menu-bar-bookmark-map (make-sparse-keymap "Bookmark functions"))
(defvar menu-bar-bookmark-map
(let ((map (make-sparse-keymap "Bookmark functions")))
(define-key map [load] '("Load a Bookmark File..." . bookmark-load))
(define-key map [write] '("Save Bookmarks As..." . bookmark-write))
(define-key map [save] '("Save Bookmarks" . bookmark-save))
(define-key map [edit] '("Edit Bookmark List" . bookmark-bmenu-list))
(define-key map [delete] '("Delete Bookmark" . bookmark-delete))
(define-key map [rename] '("Rename Bookmark" . bookmark-rename))
(define-key map [locate] '("Insert Location" . bookmark-locate))
(define-key map [insert] '("Insert Contents" . bookmark-insert))
(define-key map [set] '("Set Bookmark" . bookmark-set))
(define-key map [jump] '("Jump to Bookmark" . bookmark-jump))
map))
;;;###autoload
(defalias 'menu-bar-bookmark-map (symbol-value 'menu-bar-bookmark-map))
(defalias 'menu-bar-bookmark-map menu-bar-bookmark-map)
;; make bookmarks appear toward the right side of the menu.
(if (boundp 'menu-bar-final-items)
......@@ -2213,53 +2113,22 @@ corresponding bookmark function from Lisp \(the one without the
(cons 'bookmark menu-bar-final-items)))
(setq menu-bar-final-items '(bookmark)))
;;;###autoload (define-key menu-bar-bookmark-map [load]
;;;###autoload '("Load a Bookmark File..." . bookmark-load))
;;;###autoload (define-key menu-bar-bookmark-map [write]
;;;###autoload '("Save Bookmarks As..." . bookmark-write))
;;;###autoload (define-key menu-bar-bookmark-map [save]
;;;###autoload '("Save Bookmarks" . bookmark-save))
;;;###autoload (define-key menu-bar-bookmark-map [edit]
;;;###autoload '("Edit Bookmark List" . bookmark-bmenu-list))
;;;###autoload (define-key menu-bar-bookmark-map [delete]
;;;###autoload '("Delete Bookmark" . bookmark-menu-delete))
;;;###autoload (define-key menu-bar-bookmark-map [rename]
;;;###autoload '("Rename Bookmark" . bookmark-menu-rename))
;;;###autoload (define-key menu-bar-bookmark-map [locate]
;;;###autoload '("Insert Location" . bookmark-menu-locate))
;;;###autoload (define-key menu-bar-bookmark-map [insert]
;;;###autoload '("Insert Contents" . bookmark-menu-insert))
;;;###autoload (define-key menu-bar-bookmark-map [set]
;;;###autoload '("Set Bookmark" . bookmark-set))
;;;###autoload (define-key menu-bar-bookmark-map [jump]
;;;###autoload '("Jump to Bookmark" . bookmark-menu-jump))
;;;; end bookmark menu stuff ;;;;
;;; Load Hook
(defvar bookmark-load-hook nil
"Hook to run at the end of loading bookmark.")
"Hook run at the end of loading bookmark.")
;;; Exit Hook, called from kill-emacs-hook
(defvar bookmark-exit-hook nil
"Hook to run when emacs exits")
"Hook run when emacs exits.")
(defun bookmark-exit-hook-internal ()
"Save bookmark state, if necessary, at Emacs exit time.
This also runs `bookmark-exit-hooks'."
(and
(progn (run-hooks 'bookmark-exit-hooks) t)
bookmark-alist
(run-hooks 'bookmark-exit-hooks)
(and bookmark-alist
(bookmark-time-to-save-p t)
(bookmark-save)))
......
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