On Tue, Jan 11, 2011 at 11:25:50AM +0100, Agustin Martin wrote: > On Mon, Jan 10, 2011 at 05:17:54PM -0800, Steve VanDevender wrote: > > Package: dictionaries-common > > Version: 1.5.16 > > Severity: important > > Tags: squeeze > > > > /usr/share/emacs/site-lisp/dictionaries-common/ispell.el creates a bogus > > entry in default-menubar under XEmacs 21 (xemacs21-nomule is what I am > > running). > > > > After ispell.el changes the "Kill Process" menu item under Cmds -> > > Spell-Check, any attempt to click on the menubar produces the error > > "Wrong type argument: stringp, clear" because a symbol is not allowed as > > an optional fourth item in a menu specification. I originally > > encountered this when using the VM mail reader that is part of > > xemacs21-basesupport, and finding that after the first time I ran > > vm-compose-mail, the menubar became unusable. However, it seems likely > > this affects other Emacs/XEmacs packages as well. > > Cannot reproduce that here at all, neither with normal xemacs21 nor with > xemacs21-nomule. Using XEmacs, open a file, spellcheck, kill process, no > problem, menu is accesible. Am I missing something?
Tried also with lenny's xemacs21 and dictionaries-common 1.5.14, that also contains the change. No problem found. Note that this change was introduced in dictionaries-common 1.5.3, released in early May. Is strange that nobody else experienced this problem before. Which XEmacs 21 version and flavour are you using? Has your system anything special? > > I find that the patch below fixes the issue locally (after I apply it > > and recompile ispell.elc). If adding "'clear" to the menu was meant > > to accomplish something, that probably needs to be done in some more > > portable way. My testing does not suggest that the function of that > > menu entry is impaired by removal of "'clear". > > > > --- ispell.el.orig 2010-11-15 05:21:32.000000000 -0800 > > +++ ispell.el 2011-01-10 16:50:35.000000000 -0800 > > @@ -1352,7 +1352,7 @@ > > ["Continue Check" ispell-continue t] > > ["Complete Word Frag"ispell-complete-word-interior-frag t] > > ["Complete Word" ispell-complete-word t] > > - ["Kill Process" ispell-kill-ispell t 'clear] > > + ["Kill Process" ispell-kill-ispell t] > > ["Customize..." (customize-group 'ispell) t] > > ;; flyspell-mode may not be bound... > > ;;["flyspell" flyspell-mode > > This is there to clean session localwords. In case a change is really needed, I'd expect something like (untested) ["Kill Process" (ispell-kill-ispell t 'clear) t] to work. But I'd like to know first what is happening. Thanks for your feedback, -- Agustin -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
