[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#28008: 25.2; Resume kmacro definition errors C-u C-u <F3>
From: |
Allen Li |
Subject: |
bug#28008: 25.2; Resume kmacro definition errors C-u C-u <F3> |
Date: |
Mon, 18 Sep 2017 13:02:57 -0700 |
On Fri, Aug 11, 2017 at 6:17 AM, Tino Calancha <tino.calancha@gmail.com> wrote:
>
> On Fri, 11 Aug 2017, Eli Zaretskii wrote:
>> All these low-level changes just to support an obscure use case? Is
>> really worth the risk to break macros to cater to that?
>
> That depends of how often someone uses kbd macros. I rarely use them, but
> the people using them frequently might suffer D.2 from time to time.
>
> Actually, the patch#1 is quite short: i included a docstring fix from
> the patch#2 by mistake.
> The C code changes in patch#1 are just:
> 3 files changed, 41 insertions(+), 2 deletions(-)
I had to change a c to Qc in macros.c to get patch 1 to compile.
However, it does not seem to fix case D.2. I don't think this was
made explicit, but case D.2 applies to all undefined keybindings, so
for example C-x C-g, but also C-c z.
We should at least fix the behavior for case D.2 The more invasive
patch 2 can be discussed later, but Emacs shouldn't just discard the
entire recorded macro on an undefined key press. This makes me
nervous whenever I use kmacros.
- bug#28008: 25.2; Resume kmacro definition errors C-u C-u <F3>,
Allen Li <=