emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Options menu is broken on CVS


From: Eli Zaretskii
Subject: Re: Options menu is broken on CVS
Date: Sat, 10 Sep 2005 12:18:53 +0300

> Date: Sat, 10 Sep 2005 09:08:40 +0900
> From: YAMAMOTO Mitsuharu <address@hidden>
> Cc: address@hidden, address@hidden, address@hidden
> 
> Besides menu-updating-frame, tmm.el uses x-popup-menu (defined in
> xmenu.c) in order to obtain keyboard equivalents.
> 
>            (condition-case nil
>                (x-popup-menu nil choice) ; Get the shortcuts
>              (error nil))
> 
> Because it is enclosed with condition-case, we don't see any errors
> with it.  But keyboard equivalents are not shown with tmm for the
> non-X version where xmenu.o is not linked.  I confirmed that they were
> shown for the non-X version with the Nick's patch.

I don't think it's right to compile and link in xmenu.c on platforms
where menus are not supported on the C level.  If we want to show
keyboard equivalents in tmm, we should find another way of gleaning
them, without calling x-popup-menu.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]