(easy-menu-make-symbol): Don't wrap keyboard macros within lambdas.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index a26d399..6a7c7e1 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2008-04-24  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * emacs-lisp/easymenu.el (easy-menu-make-symbol): Don't wrap keyboard
+       macros within lambdas.
+
 2008-04-23  Stefan Monnier  <monnier@iro.umontreal.ca>
 
        * minibuffer.el (completion-try-completion): Add `point' argument.

diff --git a/lisp/emacs-lisp/easymenu.el b/lisp/emacs-lisp/easymenu.el
index ca5151f..8e2f432 100644 (file)
--- a/lisp/emacs-lisp/easymenu.el
+++ b/lisp/emacs-lisp/easymenu.el
@@ -470,7 +470,10 @@ When non-nil, NOEXP indicates that CALLBACK cannot be an expression
         (make-symbol (format "menu-function-%d" easy-menu-item-count))))
     (setq easy-menu-item-count (1+ easy-menu-item-count))
     (fset command
-         (if (or (keymapp callback) (functionp callback) noexp) callback
+         (if (or (keymapp callback) (commandp callback)
+                  ;; `functionp' is probably not needed.
+                  (functionp callback) noexp)
+              callback
            `(lambda () (interactive) ,callback)))
     command))