[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#24939: [PATCH] Add tests for lisp/kmacro.el
From: |
Eli Zaretskii |
Subject: |
bug#24939: [PATCH] Add tests for lisp/kmacro.el |
Date: |
Sat, 04 Feb 2017 13:57:16 +0200 |
> From: Gemini Lasswell <gazally@runbox.com>
> Cc: 24939@debbugs.gnu.org
> Date: Sat, 31 Dec 2016 09:42:25 -0800
>
> Eli Zaretskii <eliz@gnu.org> writes:
> > Allow me a few comments to your proposed patch.
>
> Here is an updated version taking your comments into account. I was able
> to remove the use of advice on all the functions in macros.c except for
> end-kbd-macro, because I couldn't come up with another way to make
> kmacro-end-macro recognize a keyboard macro defined by a test.
> kmacro-end-macro tests last-kbd-macro right after it calls
> end-kbd-macro, and end-kbd-macro is going to leave last-kbd-macro empty
> unless it has something in current_kboard->kbd_macro_buffer, which Lisp
> can't access. If there is another strategy you'd like me to try there,
> let me know.
Thanks, I pushed this now.
> This patch contains kmacro-tests-with-message-capture which is the same
> as the macro proposed as an addition to ert-x.el in bug#25158, so if
> that patch is adopted, it could be removed from this patch.
I've replaced the macro with ert-with-message-capture.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- bug#24939: [PATCH] Add tests for lisp/kmacro.el,
Eli Zaretskii <=