unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* easy-menu-add - arg MAP not used
@ 2006-05-04 16:23 Lennart Borgman
  2006-05-04 21:31 ` Stefan Monnier
  2006-05-05 19:05 ` Richard Stallman
  0 siblings, 2 replies; 4+ messages in thread
From: Lennart Borgman @ 2006-05-04 16:23 UTC (permalink / raw)


I just tried to use the MAP argument in `easy-menu-add'. The MAP 
argument did not work as I though. And it is not mentioned in the doc 
string. It is even not used in the code as far as I can see. Should it 
not be used?

BTW what I wanted to do was to add a menu defined the easy-menu style as 
a submenu.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: easy-menu-add - arg MAP not used
  2006-05-04 16:23 easy-menu-add - arg MAP not used Lennart Borgman
@ 2006-05-04 21:31 ` Stefan Monnier
  2006-05-05 19:05 ` Richard Stallman
  1 sibling, 0 replies; 4+ messages in thread
From: Stefan Monnier @ 2006-05-04 21:31 UTC (permalink / raw)
  Cc: Emacs Devel

> I just tried to use the MAP argument in `easy-menu-add'. The MAP argument
> did not work as I though.  And it is not mentioned in the doc string.  It is
> even not used in the code as far as I can see. Should it not be used?

> BTW what I wanted to do was to add a menu defined the easy-menu style as
> a submenu.

Easymenu.el was written specifically to make it possible to write menus that
work both under XEmacs and Emacs, even though some of the underlying
primitives are quite different (e.g. XEmacs menus aren't stored in keymaps).

So the easy-menu-add MAP is expected to be used together with the
easy-menu-define MAPS (Emacs ignores the one in easy-menu-add, while XEmacs
ignores the one in easy-menu-define).


        Stefan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: easy-menu-add - arg MAP not used
  2006-05-04 16:23 easy-menu-add - arg MAP not used Lennart Borgman
  2006-05-04 21:31 ` Stefan Monnier
@ 2006-05-05 19:05 ` Richard Stallman
  2006-05-05 19:48   ` Lennart Borgman
  1 sibling, 1 reply; 4+ messages in thread
From: Richard Stallman @ 2006-05-05 19:05 UTC (permalink / raw)
  Cc: emacs-devel

    I just tried to use the MAP argument in `easy-menu-add'. The MAP 
    argument did not work as I though. And it is not mentioned in the doc 
    string. It is even not used in the code as far as I can see. Should it 
    not be used?

It sounds like there is a bug.  If you provide a precise test case,
someone can try to debug it.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: easy-menu-add - arg MAP not used
  2006-05-05 19:05 ` Richard Stallman
@ 2006-05-05 19:48   ` Lennart Borgman
  0 siblings, 0 replies; 4+ messages in thread
From: Lennart Borgman @ 2006-05-05 19:48 UTC (permalink / raw)
  Cc: emacs-devel

Richard Stallman wrote:
>     I just tried to use the MAP argument in `easy-menu-add'. The MAP 
>     argument did not work as I though. And it is not mentioned in the doc 
>     string. It is even not used in the code as far as I can see. Should it 
>     not be used?
>
> It sounds like there is a bug.  If you provide a precise test case,
> someone can try to debug it.
>   
I think Stefan explained that the argument list is a compatibility issue 
with XEmacs:

    http://lists.gnu.org/archive/html/emacs-devel/2006-05/msg00221.html

But I think that the doc string anyway should say that the argument MAP 
is ignored in Emacs.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2006-05-05 19:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-05-04 16:23 easy-menu-add - arg MAP not used Lennart Borgman
2006-05-04 21:31 ` Stefan Monnier
2006-05-05 19:05 ` Richard Stallman
2006-05-05 19:48   ` Lennart Borgman

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).