GNU bug report logs - #25382
25.1.91; easy-menu-add empty

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 25382 in the body.
You can then email your comments to 25382 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
To: bug-gnu-emacs <at> gnu.org
Subject: 25.1.91; easy-menu-add empty
Date: Sat, 7 Jan 2017 09:45:31 +0100

GNU Emacs 25.1.91.1 (i686-pc-linux-gnu, GTK+ Version 3.14.5) of 2017-01-02

At python-mode.el `easy-menu-add' is used, which stopped working. Having 
a look at the source, looks like an empty body is provided.

Message #8 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: npostavs <at> users.sourceforge.net
To: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
Cc: 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Sat, 07 Jan 2017 09:35:53 -0500

[Message part 1 (text/plain, inline)]

found 25382 23.2
tags 25382 notabug
quit

Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:

> GNU Emacs 25.1.91.1 (i686-pc-linux-gnu, GTK+ Version 3.14.5) of 2017-01-02
>
> At python-mode.el `easy-menu-add' is used, which stopped
> working. Having a look at the source, looks like an empty body is
> provided.

`easy-menu-add's body has been empty since at least 23.2, it was made
empty by [1: 59c7bf4].  As far as I can tell, it only exists to be more
compatible with XEmacs, so this is not a bug.

1: 2008-08-14 17:42:21 +0000 59c7bf4d7bf927e65c6cd40cf36e2e6bcbabfa78
  (easy-menu-add): Don't precompute keybindings since those precomputed values aren't used any more.

I propose the following patch for emacs-25 to clarify that:

[v1-0001-Clarify-that-easy-menu-add-is-a-nop-Bug-25382.patch (text/x-diff, inline)]

From 1a8caf7ad763aa03f279ec54ceec4ff9cc1d7237 Mon Sep 17 00:00:00 2001
From: Noam Postavsky <npostavs <at> gmail.com>
Date: Sat, 7 Jan 2017 09:26:08 -0500
Subject: [PATCH v1] Clarify that easy-menu-add is a nop (Bug#25382)

* lisp/emacs-lisp/easymenu.el (easy-menu-add): Make it into an alias of
`ignore', like `easy-menu-remove'.
---
 lisp/emacs-lisp/easymenu.el | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/lisp/emacs-lisp/easymenu.el b/lisp/emacs-lisp/easymenu.el
index 603bbcd..4fc9a78 100644
--- a/lisp/emacs-lisp/easymenu.el
+++ b/lisp/emacs-lisp/easymenu.el
@@ -489,7 +489,7 @@ easy-menu-change
 
 ;; XEmacs needs the following two functions to add and remove menus.
 ;; In Emacs this is done automatically when switching keymaps, so
-;; here easy-menu-remove is a noop.
+;; here easy-menu-remove and easy-menu-add are a noops.
 (defalias 'easy-menu-remove 'ignore
   "Remove MENU from the current menu bar.
 Contrary to XEmacs, this is a nop on Emacs since menus are automatically
@@ -497,15 +497,16 @@ 'easy-menu-remove
 
 \(fn MENU)")
 
-(defun easy-menu-add (_menu &optional _map)
+(defalias 'easy-menu-add #'ignore
   "Add the menu to the menubar.
-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.
+On Emacs this is a nop, because 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.
 
 You should call this once the menu and keybindings are set up
-completely and menu filter functions can be expected to work."
-  )
+completely and menu filter functions can be expected to work.
+
+\(fn MENU &optional MAP)")
 
 (defun add-submenu (menu-path submenu &optional before in-menu)
   "Add submenu SUBMENU in the menu at MENU-PATH.
-- 
2.9.3

Message #15 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
To: npostavs <at> users.sourceforge.net
Cc: 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Mon, 9 Jan 2017 07:40:39 +0100

On 07.01.2017 15:35, npostavs <at> users.sourceforge.net wrote:
> found 25382 23.2
> tags 25382 notabug
> quit
>
> Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:
>
>> GNU Emacs 25.1.91.1 (i686-pc-linux-gnu, GTK+ Version 3.14.5) of 2017-01-02
>>
>> At python-mode.el `easy-menu-add' is used, which stopped
>> working. Having a look at the source, looks like an empty body is
>> provided.
> `easy-menu-add's body has been empty since at least 23.2, it was made
> empty by [1: 59c7bf4].  As far as I can tell, it only exists to be more
> compatible with XEmacs, so this is not a bug.
>
> 1: 2008-08-14 17:42:21 +0000 59c7bf4d7bf927e65c6cd40cf36e2e6bcbabfa78
>    (easy-menu-add): Don't precompute keybindings since those precomputed values aren't used any more.
>
> I propose the following patch for emacs-25 to clarify that:
>

Looks redundant for me. Should one function "ignore" not be enough?

Message #18 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: npostavs <at> users.sourceforge.net
To: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
Cc: 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Mon, 09 Jan 2017 23:40:33 -0500

Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:
>
> Looks redundant for me. Should one function "ignore" not be enough?

It would seem less redundant if you maintained code that needed to work
on both XEmacs and GNU Emacs.

Message #21 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
To: npostavs <at> users.sourceforge.net
Cc: 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Tue, 10 Jan 2017 08:49:40 +0100

On 10.01.2017 05:40, npostavs <at> users.sourceforge.net wrote:
> Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:
>> Looks redundant for me. Should one function "ignore" not be enough?
> It would seem less redundant if you maintained code that needed to work
> on both XEmacs and GNU Emacs.

Sorry , don't understand. If it's about compatibility, why not keep 
easy-menu-add adding the menu?

Message #24 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: npostavs <at> users.sourceforge.net
To: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
Cc: 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Tue, 10 Jan 2017 08:23:34 -0500

Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:

> On 10.01.2017 05:40, npostavs <at> users.sourceforge.net wrote:
>> Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:
>>> Looks redundant for me. Should one function "ignore" not be enough?
>> It would seem less redundant if you maintained code that needed to work
>> on both XEmacs and GNU Emacs.
>
> Sorry , don't understand. If it's about compatibility, why not keep
> easy-menu-add adding the menu?

As far as I can tell, it never did that in GNU Emacs.  Why do you think
it did?

Message #27 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
To: npostavs <at> users.sourceforge.net
Cc: 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Tue, 10 Jan 2017 17:01:25 +0100

On 10.01.2017 14:23, npostavs <at> users.sourceforge.net wrote:
> Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:
>
>> On 10.01.2017 05:40, npostavs <at> users.sourceforge.net wrote:
>>> Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:
>>>> Looks redundant for me. Should one function "ignore" not be enough?
>>> It would seem less redundant if you maintained code that needed to work
>>> on both XEmacs and GNU Emacs.
>> Sorry , don't understand. If it's about compatibility, why not keep
>> easy-menu-add adding the menu?
> As far as I can tell, it never did that in GNU Emacs.  Why do you think
> it did?

Because we used it following a conditional in python-mode.el. It got 
broken only recently.

Message #30 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: Noam Postavsky <npostavs <at> users.sourceforge.net>
To: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
Cc: 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Tue, 10 Jan 2017 11:21:10 -0500

On Tue, Jan 10, 2017 at 11:01 AM, Andreas Röhler
<andreas.roehler <at> easy-emacs.de> wrote:
>
>
>>>
>>> Sorry , don't understand. If it's about compatibility, why not keep
>>> easy-menu-add adding the menu?
>>
>> As far as I can tell, it never did that in GNU Emacs.  Why do you think
>> it did?
>
>
> Because we used it following a conditional in python-mode.el. It got broken
> only recently.
>

As I mentioned in
https://debbugs.gnu.org/cgi/bugreport.cgi?bug=25382#8, it's been a nop
since 2008. And even before that, it did not add any menus. Perhaps
something else changed recently?

Message #33 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
To: Noam Postavsky <npostavs <at> users.sourceforge.net>
Cc: 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Tue, 10 Jan 2017 19:47:21 +0100

On 10.01.2017 17:21, Noam Postavsky wrote:
> On Tue, Jan 10, 2017 at 11:01 AM, Andreas Röhler
> <andreas.roehler <at> easy-emacs.de> wrote:
>>
>>>> Sorry , don't understand. If it's about compatibility, why not keep
>>>> easy-menu-add adding the menu?
>>> As far as I can tell, it never did that in GNU Emacs.  Why do you think
>>> it did?
>>
>> Because we used it following a conditional in python-mode.el. It got broken
>> only recently.
>>
> As I mentioned in
> https://debbugs.gnu.org/cgi/bugreport.cgi?bug=25382#8, it's been a nop
> since 2008. And even before that, it did not add any menus. Perhaps
> something else changed recently?

That seems the case, thanks!

Message #36 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Andreas Röhler <andreas.roehler <at> easy-emacs.de>,
 25382 <at> debbugs.gnu.org
Cc: Noam Postavsky <npostavs <at> users.sourceforge.net>
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Wed, 11 Jan 2017 13:24:54 -0500

Please give a minimal example starting from emacs -Q that shows what
the problem actually is.

http://xyproblem.info/

Message #39 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
To: Glenn Morris <rgm <at> gnu.org>, 25382 <at> debbugs.gnu.org
Cc: Noam Postavsky <npostavs <at> users.sourceforge.net>
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Thu, 12 Jan 2017 08:38:36 +0100

On 11.01.2017 19:24, Glenn Morris wrote:
> Please give a minimal example starting from emacs -Q that shows what
> the problem actually is.
>
> http://xyproblem.info/

Please see me previous post. That way `easy-menu-add' issues are 
supposed to be unrelated.

Message #44 received at 25382 <at> debbugs.gnu.org (full text, mbox):

From: npostavs <at> users.sourceforge.net
To: Andreas Röhler <andreas.roehler <at> easy-emacs.de>
Cc: Glenn Morris <rgm <at> gnu.org>, 25382 <at> debbugs.gnu.org
Subject: Re: bug#25382: 25.1.91; easy-menu-add empty
Date: Thu, 12 Jan 2017 10:32:54 -0500

close 25382 
quit

Andreas Röhler <andreas.roehler <at> easy-emacs.de> writes:

> On 11.01.2017 19:24, Glenn Morris wrote:
>> Please give a minimal example starting from emacs -Q that shows what
>> the problem actually is.
>>
>> http://xyproblem.info/
>
> Please see me previous post. That way `easy-menu-add' issues are
> supposed to be unrelated.

Yeah, so if you find that the menu problems are from Emacs then open a
new bug, because I've pushed my patch to emacs-25 [1: b0ade0d], and I'm
closing this bug.

1: 2017-01-12 10:18:56 -0500 b0ade0df21d4cde8537c29f81eb10bdcf1cdfbfc
  Clarify that easy-menu-add is a nop (Bug#25382)

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.