GNU bug report logs - #6256
24.0.50; read-event in `repeat' command

Previous Next

Package: emacs;

Reported by: "Drew Adams" <drew.adams <at> oracle.com>

Date: Mon, 24 May 2010 15:13:02 UTC

Severity: normal

Found in version 24.0.50

Done: Stefan Monnier <monnier <at> iro.umontreal.ca>

Bug is archived. No further changes may be made.

Full log


Message #23 received at 6256 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: "Drew Adams" <drew.adams <at> oracle.com>
Cc: 6256 <at> debbugs.gnu.org
Subject: Re: bug#6256: 24.0.50; read-event in `repeat' command
Date: Mon, 05 Jul 2010 00:45:37 +0200
> That does not happen, however, because of this restrictive `eq' test in the
> definition of function `repeat':

> (while (eq (read-event) repeat-repeat-char)
>   (repeat repeat-arg))

> The event read will be something like this, for the wheel action:

> (wheel-down (#<window 8 on foo.el> 2051 (118 . 176) 158455015 nil
>             2051 (59 . 40) nil (26 . 2) (2 . 4)))

> I would think that we would want to change the test to this, or similar:

> (while (let ((evt  (read-event)))
>          (and (equal (event-basic-type evt) (event-basic-type
> repeat-repeat-char))
>               (equal (event-modifiers evt)  (event-modifiers
> repeat-repeat-char))))
>   (repeat repeat-arg))

> And that seems to work OK.  What do you think - is it reasonable to do
> that?

That sounds right, yes,
People, feel free to make such a change,


        Stefan




This bug report was last modified 14 years and 271 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.