Damien Cassou | 29 Jun 2012 18:58
Picon
Gravatar

bug#11817: 24.1.50; Useless code in `kmacro-call-macro'

Hi,

In `kmacro-call-macro', the variable `repeat-key-str' is defined and
then set but never used.

In GNU Emacs 24.1.50.1 (i686-pc-linux-gnu, GTK+ Version 3.4.2)
 of 2012-06-28 on iridium, modified by Debian
 (emacs-snapshot package, version 2:20120628-1~ppa1~precise1)

Glenn Morris | 6 Jul 2012 04:05
Picon

bug#11817: 24.1.50; Useless code in `kmacro-call-macro'

Damien Cassou wrote:

> In `kmacro-call-macro', the variable `repeat-key-str' is defined and
> then set but never used.

It's unused since 2012-05-05.
There used to be a "(Type e to repeat macro)" message after running a
macro. Now there isn't. I wonder why, since it has nothing to do with
the use of set-temporary-overlay-map AFAICS.

Stefan Monnier | 6 Jul 2012 23:20
Picon

bug#11817: 24.1.50; Useless code in `kmacro-call-macro'

>> In `kmacro-call-macro', the variable `repeat-key-str' is defined and
>> then set but never used.
> It's unused since 2012-05-05.
> There used to be a "(Type e to repeat macro)" message after running a
> macro. Now there isn't. I wonder why, since it has nothing to do with
> the use of set-temporary-overlay-map AFAICS.

An oversight on my part, it seems, feel free to re-add it,

        Stefan

Glenn Morris | 6 Jul 2012 23:35
Picon

bug#11817: 24.1.50; Useless code in `kmacro-call-macro'

Stefan Monnier wrote:

>> There used to be a "(Type e to repeat macro)" message after running a
>> macro. Now there isn't. I wonder why, since it has nothing to do with
>> the use of set-temporary-overlay-map AFAICS.
>
> An oversight on my part, it seems, feel free to re-add it,

Done.


Gmane