[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: bug#58799: Make winner key sequences repeatable in repeat-mode
From: |
Eli Zaretskii |
Subject: |
Re: bug#58799: Make winner key sequences repeatable in repeat-mode |
Date: |
Mon, 31 Oct 2022 17:28:18 +0200 |
> From: Robert Pluim <rpluim@gmail.com>
> Cc: Lars Ingebrigtsen <larsi@gnus.org>, emacs-devel@gnu.org
> Date: Mon, 31 Oct 2022 16:09:54 +0100
>
> diff --git c/lisp/keymap.el i/lisp/keymap.el
> index 107565590c..750095fab4 100644
> --- c/lisp/keymap.el
> +++ i/lisp/keymap.el
> @@ -545,6 +545,10 @@ define-keymap
> (let ((def (pop definitions)))
> (if (eq key :menu)
> (easy-menu-define nil keymap "" def)
> + (unless (key-valid-p key)
> + (error
> + "Key '%s' in keymap '%s' is not `key-valid-p'"
> + key keymap))
> (if (member key seen-keys)
> (error "Duplicate definition for key: %S %s" key keymap)
> (push key seen-keys))
>
> which will give a run-time error for things like "[left]" and
> '[left]', but thatʼs redundant with the compile time warning:
>
> make[3]: Entering directory '/home/rpluim/repos/emacs/lispʼ
> ELC winner.elc
>
> In toplevel form:
> winner.el:314:15: Warning: Invalid ‘kbd’ syntax: "[left]"
>
> Eli? Lars? Is this overkill for such a rare case?
I don't know. I guess it depends on how many people will be tripped
by this. The change itself is small enough, I think.