* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error @ 2010-06-02 21:42 ` Drew Adams 2010-07-27 1:38 ` Juanma Barranquero 0 siblings, 1 reply; 10+ messages in thread From: Drew Adams @ 2010-06-02 21:42 UTC (permalink / raw To: 6334 emacs -Q Try evaluating either of these examples from (elisp) Defining Minor Modes: (define-minor-mode hungry-mode "Toggle Hungry mode. With no argument, this command toggles the mode. Non-null prefix argument turns on the mode. Null prefix argument turns off the mode. When Hungry mode is enabled, the control delete key gobbles all preceding whitespace except the last. See the command \\[hungry-electric-delete]." ;; The initial value. nil ;; The indicator for the mode line. " Hungry" ;; The minor mode bindings. '(("\C-\^?" . hungry-electric-delete)) :group 'hunger) (define-minor-mode hungry-mode "Toggle Hungry mode. With no argument, this command toggles the mode. Non-null prefix argument turns on the mode. Null prefix argument turns off the mode. When Hungry mode is enabled, the control delete key gobbles all preceding whitespace except the last. See the command \\[hungry-electric-delete]." ;; The initial value. :init-value nil ;; The indicator for the mode line. :lighter " Hungry" ;; The minor mode bindings. :keymap '(("\C-\^?" . hungry-electric-delete) ("\C-\M-\^?" . (lambda () (interactive) (hungry-electric-delete t)))) :group 'hunger) Both raise this error: Debugger entered--Lisp error: (error "Invalid modifier in string") In GNU Emacs 24.0.50.1 (i386-mingw-nt5.1.2600) of 2010-05-23 on G41R2F1 Windowing system distributor `Microsoft Corp.', version 5.1.2600 configured using `configure --with-gcc (3.4) --no-opt --cflags -Ic:/xpm/include' ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-06-02 21:42 ` bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error Drew Adams @ 2010-07-27 1:38 ` Juanma Barranquero 2010-07-27 4:44 ` Sven Joachim 2010-07-27 11:26 ` Štěpán Němec 0 siblings, 2 replies; 10+ messages in thread From: Juanma Barranquero @ 2010-07-27 1:38 UTC (permalink / raw To: Drew Adams; +Cc: 6334-done On Wed, Jun 2, 2010 at 23:42, Drew Adams <drew.adams@oracle.com> wrote: > Debugger entered--Lisp error: (error "Invalid modifier in string") I've fixed it using C-delete and C-M-delete (which are clearer anyway) in the examples. Juanma ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-07-27 1:38 ` Juanma Barranquero @ 2010-07-27 4:44 ` Sven Joachim 2010-07-27 16:15 ` Juanma Barranquero 2010-07-27 11:26 ` Štěpán Němec 1 sibling, 1 reply; 10+ messages in thread From: Sven Joachim @ 2010-07-27 4:44 UTC (permalink / raw To: Juanma Barranquero; +Cc: 6334 On 2010-07-27 03:38 +0200, Juanma Barranquero wrote: > On Wed, Jun 2, 2010 at 23:42, Drew Adams <drew.adams@oracle.com> wrote: > >> Debugger entered--Lisp error: (error "Invalid modifier in string") > > I've fixed it using C-delete and C-M-delete (which are clearer anyway) > in the examples. C-M-delete is not the most fortunate choice since it can induce a reboot on some systems (e.g. on the GNU/Linux console or on MS-DOS). On Windows, it invokes the task manager IIRC. Sven ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-07-27 4:44 ` Sven Joachim @ 2010-07-27 16:15 ` Juanma Barranquero 0 siblings, 0 replies; 10+ messages in thread From: Juanma Barranquero @ 2010-07-27 16:15 UTC (permalink / raw To: Sven Joachim; +Cc: 6334 On Tue, Jul 27, 2010 at 06:44, Sven Joachim <svenjoac@gmx.de> wrote: > C-M-delete is not the most fortunate choice since it can induce a reboot > on some systems (e.g. on the GNU/Linux console or on MS-DOS). On > Windows, it invokes the task manager IIRC. I agree. I didn't want to change the documentation, just fix the bug. Juanma ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-07-27 1:38 ` Juanma Barranquero 2010-07-27 4:44 ` Sven Joachim @ 2010-07-27 11:26 ` Štěpán Němec 2010-07-27 16:18 ` Juanma Barranquero 1 sibling, 1 reply; 10+ messages in thread From: Štěpán Němec @ 2010-07-27 11:26 UTC (permalink / raw To: 6334; +Cc: lekktu Juanma Barranquero <lekktu@gmail.com> writes: > On Wed, Jun 2, 2010 at 23:42, Drew Adams <drew.adams@oracle.com> wrote: > >> Debugger entered--Lisp error: (error "Invalid modifier in string") > > I've fixed it using C-delete and C-M-delete (which are clearer anyway) > in the examples. I'm not sure if you intended to preserve the key binding, let me just point out that ^? (which was in the original example) is (ASCII) DEL, i.e. usually [backspace], not [delete] on (modern?) text terminals. Štěpán ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-07-27 11:26 ` Štěpán Němec @ 2010-07-27 16:18 ` Juanma Barranquero 2010-07-27 17:00 ` Štěpán Němec 0 siblings, 1 reply; 10+ messages in thread From: Juanma Barranquero @ 2010-07-27 16:18 UTC (permalink / raw To: Štěpán Němec; +Cc: 6334 On Tue, Jul 27, 2010 at 13:26, Štěpán Němec <stepnem@gmail.com> wrote: > I'm not sure if you intended to preserve the key binding, let me just > point out that ^? (which was in the original example) is (ASCII) DEL, > i.e. usually [backspace], not [delete] on (modern?) text terminals. The docstring for the sample minor mode says "the control delete key", and my DEL key returns delete. If there's a more apropriate key binding that does not reproduce the bug, it's fine by me. Juanma ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-07-27 16:18 ` Juanma Barranquero @ 2010-07-27 17:00 ` Štěpán Němec 2010-07-27 17:08 ` Juanma Barranquero 0 siblings, 1 reply; 10+ messages in thread From: Štěpán Němec @ 2010-07-27 17:00 UTC (permalink / raw To: Juanma Barranquero; +Cc: 6334 Juanma Barranquero <lekktu@gmail.com> writes: > On Tue, Jul 27, 2010 at 13:26, Štěpán Němec <stepnem@gmail.com> wrote: > >> I'm not sure if you intended to preserve the key binding, let me just >> point out that ^? (which was in the original example) is (ASCII) DEL, >> i.e. usually [backspace], not [delete] on (modern?) text terminals. > > The docstring for the sample minor mode says "the control delete key", > and my DEL key returns delete. If there's a more apropriate key > binding that does not reproduce the bug, it's fine by me. That'd be [C-backspace], if you're using the vector notation. Note we're talking about the DEL _character_, not the key labeled Del(ete). It's the Backspace key that usually sends DEL in the terminal (also note that the example speaks about deleting all _preceding_ whitespace). I suspect "the control delete key" wording comes from times when the keyboards were somewhat different from those of today. But it's only an example, so all this perhaps doesn't matter. I just thought I'd mention the semantic shift introduced by the change, sorry. Štěpán ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-07-27 17:00 ` Štěpán Němec @ 2010-07-27 17:08 ` Juanma Barranquero 2010-07-27 20:22 ` David De La Harpe Golden 0 siblings, 1 reply; 10+ messages in thread From: Juanma Barranquero @ 2010-07-27 17:08 UTC (permalink / raw To: Štěpán Němec; +Cc: 6334 On Tue, Jul 27, 2010 at 19:00, Štěpán Němec <stepnem@gmail.com> wrote: > Note we're > talking about the DEL _character_, not the key labeled Del(ete). The "control delete" bit on the docstring makes me think of pressing the keys labeled Ctrl and Del, not Ctrl + Backspace... > (also note that > the example speaks about deleting all _preceding_ whitespace). ...but I suspect you're right. I'll change it to C-backspace / C-M-backspace, if everybody thinks these are the right keybindings. > But it's only an example, so all this perhaps doesn't matter. I just > thought I'd mention the semantic shift introduced by the change, sorry. No, thanks for bringing this up. Juanma ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-07-27 17:08 ` Juanma Barranquero @ 2010-07-27 20:22 ` David De La Harpe Golden 2010-07-27 22:41 ` Juanma Barranquero 0 siblings, 1 reply; 10+ messages in thread From: David De La Harpe Golden @ 2010-07-27 20:22 UTC (permalink / raw To: Juanma Barranquero; +Cc: 6334, Štěpán Němec On 27/07/10 18:08, Juanma Barranquero wrote: > ...but I suspect you're right. I'll change it to C-backspace / > C-M-backspace, if everybody thinks these are the right keybindings. > Just noting: C-M-backspace traditionally kills an x11 server, usually dumping you back at a login screen when the login manager (x/g/k)dm respawns the server. Sometimes it's disabled these days. ^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error 2010-07-27 20:22 ` David De La Harpe Golden @ 2010-07-27 22:41 ` Juanma Barranquero 0 siblings, 0 replies; 10+ messages in thread From: Juanma Barranquero @ 2010-07-27 22:41 UTC (permalink / raw To: David De La Harpe Golden; +Cc: 6334, Štěpán Němec On Tue, Jul 27, 2010 at 22:22, David De La Harpe Golden <david@harpegolden.net> wrote: > Just noting: C-M-backspace traditionally kills an x11 server, "\C-\M-\^?" is what the original example used. Juanma ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2010-07-27 22:41 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <AcsCnIknca5GRt4nT1+ZpSXlweHjjg==> 2010-06-02 21:42 ` bug#6334: 24.0.50; both examples in (elisp) Defining Minor Modes raise an error Drew Adams 2010-07-27 1:38 ` Juanma Barranquero 2010-07-27 4:44 ` Sven Joachim 2010-07-27 16:15 ` Juanma Barranquero 2010-07-27 11:26 ` Štěpán Němec 2010-07-27 16:18 ` Juanma Barranquero 2010-07-27 17:00 ` Štěpán Němec 2010-07-27 17:08 ` Juanma Barranquero 2010-07-27 20:22 ` David De La Harpe Golden 2010-07-27 22:41 ` Juanma Barranquero
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).