From 8292be61c58c9eed64f132ccd688f889fa389230 Mon Sep 17 00:00:00 2001 From: David Kastrup Date: Fri, 20 May 2005 15:30:59 +0000 Subject: [PATCH] (easy-menu-add): Correct docstring since easy-menu-add is not a nop on Emacs; and clarify when to call it. --- lisp/ChangeLog | 5 +++++ lisp/emacs-lisp/easymenu.el | 14 +++++++++----- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/lisp/ChangeLog b/lisp/ChangeLog index f35eec9e1f0..00f41a56f10 100644 --- a/lisp/ChangeLog +++ b/lisp/ChangeLog @@ -1,3 +1,8 @@ +2005-05-20 David Kastrup + + * emacs-lisp/easymenu.el (easy-menu-add): Correct docstring since + easy-menu-add is not a nop on Emacs; and clarify when to call it. + 2005-05-20 Carsten Dominik * textmodes/org.el (org-agenda-toggle-time-grid): New command. diff --git a/lisp/emacs-lisp/easymenu.el b/lisp/emacs-lisp/easymenu.el index c9f1769ae14..b3160c9b752 100644 --- a/lisp/emacs-lisp/easymenu.el +++ b/lisp/emacs-lisp/easymenu.el @@ -471,11 +471,15 @@ Contrary to XEmacs, this is a nop on Emacs since menus are automatically (defun easy-menu-add (menu &optional map) "Add the menu to the menubar. -This is a nop on Emacs since menus are automatically activated when the -corresponding keymap is activated. On XEmacs this is needed to actually -add the menu to the current menubar. -Maybe precalculate equivalent key bindings. -Do it only if `easy-menu-precalculate-equivalent-keybindings' is on." +On Emacs, menus are already automatically activated when the +corresponding keymap is activated. On XEmacs this is needed to +actually add the menu to the current menubar. + +This also precalculates equivalent key bindings when +`easy-menu-precalculate-equivalent-keybindings' is on. + +You should call this once the menu and keybindings are set up +completely and menu filter functions can be expected to work." (when easy-menu-precalculate-equivalent-keybindings (if (and (symbolp menu) (not (keymapp menu)) (boundp menu)) (setq menu (symbol-value menu))) -- 2.39.5