unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* describe-key: Wrong type argument: sequencep, C-up
@ 2017-12-29  4:31 zhang cc
  2017-12-29  9:14 ` Alan Mackenzie
  0 siblings, 1 reply; 3+ messages in thread
From: zhang cc @ 2017-12-29  4:31 UTC (permalink / raw)
  To: emacs-devel@gnu.org

[-- Attachment #1: Type: text/plain, Size: 200 bytes --]

Run the command: describe-key, then press the key Control-up, and emacs report the following error message:
describe-key: Wrong type argument: sequencep, C-up

Emacs: 26.0.90
OS: macOS 10.13.2


[-- Attachment #2: Type: text/html, Size: 700 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: describe-key: Wrong type argument: sequencep, C-up
  2017-12-29  4:31 describe-key: Wrong type argument: sequencep, C-up zhang cc
@ 2017-12-29  9:14 ` Alan Mackenzie
  2017-12-29 10:01   ` Alan Mackenzie
  0 siblings, 1 reply; 3+ messages in thread
From: Alan Mackenzie @ 2017-12-29  9:14 UTC (permalink / raw)
  To: zhang cc; +Cc: emacs-devel@gnu.org

Hello, Zhang.

On Fri, Dec 29, 2017 at 04:31:39 +0000, zhang cc wrote:
> Run the command: describe-key, then press the key Control-up, and emacs report the following error message:
> describe-key: Wrong type argument: sequencep, C-up

> Emacs: 26.0.90
> OS: macOS 10.13.2

Yes.  A careless `copy-sequence' has been used.

I'll fix it.

-- 
Alan Mackenzie (Nuremberg, Germany).



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: describe-key: Wrong type argument: sequencep, C-up
  2017-12-29  9:14 ` Alan Mackenzie
@ 2017-12-29 10:01   ` Alan Mackenzie
  0 siblings, 0 replies; 3+ messages in thread
From: Alan Mackenzie @ 2017-12-29 10:01 UTC (permalink / raw)
  To: zhang cc; +Cc: emacs-devel@gnu.org

Hello again, Zhang.

On Fri, Dec 29, 2017 at 09:14:55 +0000, Alan Mackenzie wrote:
> On Fri, Dec 29, 2017 at 04:31:39 +0000, zhang cc wrote:
> > Run the command: describe-key, then press the key Control-up, and emacs report the following error message:
> > describe-key: Wrong type argument: sequencep, C-up

> > Emacs: 26.0.90
> > OS: macOS 10.13.2

> Yes.  A careless `copy-sequence' has been used.

> I'll fix it.

I've committed the following patch to the emacs-26 branch which should
fix the bug.  Thanks for taking the trouble to report it.


commit b240c7846b0d03e9f69af753cd24eb1e550f543c
Author: Alan Mackenzie <acm@muc.de>
Date:   Fri Dec 29 09:54:03 2017 +0000

    * lisp/help.el (describe-key): Only (copy-sequence elt) when elt is a list.

diff --git a/lisp/help.el b/lisp/help.el
index fa7f6b0d5b..8ff27b0b24 100644
--- a/lisp/help.el
+++ b/lisp/help.el
@@ -876,7 +876,7 @@ describe-key
         (when (vectorp key)
           (let* ((last (1- (length key)))
                  (elt (aref key last))
-                 (elt-1 (copy-sequence elt))
+                 (elt-1 (if (listp elt) (copy-sequence elt) elt))
                  key-1 down-event-type)
             (when (and (listp elt-1)
                        (symbolp (car elt-1))


-- 
Alan Mackenzie (Nuremberg, Germany).



^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-12-29 10:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-29  4:31 describe-key: Wrong type argument: sequencep, C-up zhang cc
2017-12-29  9:14 ` Alan Mackenzie
2017-12-29 10:01   ` Alan Mackenzie

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).