From: Michael Heerdegen <michael_heerdegen@web.de>
To: Gilles <gilles.usenet@gmail.com>
Cc: 54161@debbugs.gnu.org
Subject: bug#54161: 27.2; `define-minor-mode' with alist of key bindings
Date: Tue, 01 Mar 2022 02:21:32 +0100 [thread overview]
Message-ID: <87bkyqmpr7.fsf@web.de> (raw)
In-Reply-To: <CAHf9+CvXQXvt5e0Hu64xZOu0USb5LhsnzLZyvQ44FHJ1KLhUKw@mail.gmail.com> (Gilles's message of "Fri, 25 Feb 2022 22:41:05 +0100")
[-- Attachment #1: Type: text/plain, Size: 245 bytes --]
Gilles <gilles.usenet@gmail.com> writes:
> The Lisp *expression* (kbd "C-o") *returns* a suitable argument for
> define-key.
I think the small source of confusion is whether these arguments are
evaluated. How about making that clearer, e.g.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-WIP-Fix-54161.patch --]
[-- Type: text/x-diff, Size: 1366 bytes --]
From 73283a3a37a7df41a0a1ab9d88bcf31704e9e841 Mon Sep 17 00:00:00 2001
From: Michael Heerdegen <michael_heerdegen@web.de>
Date: Tue, 1 Mar 2022 02:13:14 +0100
Subject: [PATCH] WIP: Fix #54161
---
lisp/emacs-lisp/easy-mmode.el | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lisp/emacs-lisp/easy-mmode.el b/lisp/emacs-lisp/easy-mmode.el
index 688c76e0c5..b35be58b3a 100644
--- a/lisp/emacs-lisp/easy-mmode.el
+++ b/lisp/emacs-lisp/easy-mmode.el
@@ -168,9 +168,9 @@ define-minor-mode
If non-nil, it should be an unquoted variable name (whose value
is a keymap), or an expression that returns either a keymap or
a list of (KEY . BINDING) pairs where KEY and BINDING are
- suitable for `define-key'. If you supply a KEYMAP argument
- that is not a symbol, this macro defines the variable MODE-map
- and gives it the value that KEYMAP specifies.
+ suitable values for `define-key'. If you supply a KEYMAP
+ argument that is not a symbol, this macro defines the
+ variable MODE-map and gives it the value that KEYMAP specifies.
:interactive VAL Whether this mode should be a command or not. The default
is to make it one; use nil to avoid that. If VAL is a list,
it's interpreted as a list of major modes this minor mode
--
2.30.2
[-- Attachment #3: Type: text/plain, Size: 93 bytes --]
BTW, should we fix the mixed indentation style in these lines (tabs
vs. spaces)?
Michael.
next prev parent reply other threads:[~2022-03-01 1:21 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-25 17:48 bug#54161: 27.2; `define-minor-mode' with alist of key bindings Drew Adams
2022-02-25 21:41 ` Gilles
2022-02-26 3:20 ` bug#54161: [External] : " Drew Adams
2022-02-28 9:46 ` Lars Ingebrigtsen
2022-03-01 1:21 ` Michael Heerdegen [this message]
2022-03-01 15:41 ` Lars Ingebrigtsen
2022-03-01 18:16 ` bug#54161: [External] : " Drew Adams
2022-03-01 22:18 ` Michael Heerdegen
2022-03-01 22:52 ` Lars Ingebrigtsen
2022-03-01 23:56 ` Michael Heerdegen
2022-03-02 0:04 ` Michael Heerdegen
2022-03-01 18:11 ` Drew Adams
2022-02-25 22:33 ` Drew Adams
2022-02-25 22:43 ` Drew Adams
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87bkyqmpr7.fsf@web.de \
--to=michael_heerdegen@web.de \
--cc=54161@debbugs.gnu.org \
--cc=gilles.usenet@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.