[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] externals/transient 804feb3b41 3/6: transient--{push, pop}-keymap
From: |
Stefan Monnier |
Subject: |
[elpa] externals/transient 804feb3b41 3/6: transient--{push, pop}-keymap: Do nothing if keymap is void |
Date: |
Wed, 19 Jan 2022 18:53:38 -0500 (EST) |
branch: externals/transient
commit 804feb3b418809a831f8ac6d63233d19ef235a90
Author: Jonas Bernoulli <jonas@bernoul.li>
Commit: Jonas Bernoulli <jonas@bernoul.li>
transient--{push,pop}-keymap: Do nothing if keymap is void
Previously nil was pushed/popped using the low-level function, which
didn't have a noticeable effect but it is better to explicitly turn
these cases into noops.
---
lisp/transient.el | 22 +++++++++++++---------
1 file changed, 13 insertions(+), 9 deletions(-)
diff --git a/lisp/transient.el b/lisp/transient.el
index 2e89fb19a0..e4c6a07228 100644
--- a/lisp/transient.el
+++ b/lisp/transient.el
@@ -1634,15 +1634,19 @@ of the corresponding object.")
(defvar transient--redisplay-map nil)
(defvar transient--redisplay-key nil)
-(defun transient--push-keymap (map)
- (transient--debug " push %s%s" map (if (symbol-value map) "" " VOID"))
- (with-demoted-errors "transient--push-keymap: %S"
- (internal-push-keymap (symbol-value map) 'overriding-terminal-local-map)))
-
-(defun transient--pop-keymap (map)
- (transient--debug " pop %s%s" map (if (symbol-value map) "" " VOID"))
- (with-demoted-errors "transient--pop-keymap: %S"
- (internal-pop-keymap (symbol-value map) 'overriding-terminal-local-map)))
+(defun transient--push-keymap (var)
+ (let ((map (symbol-value var)))
+ (transient--debug " push %s%s" var (if map "" " VOID"))
+ (when map
+ (with-demoted-errors "transient--push-keymap: %S"
+ (internal-push-keymap map 'overriding-terminal-local-map)))))
+
+(defun transient--pop-keymap (var)
+ (let ((map (symbol-value var)))
+ (transient--debug " pop %s%s" var (if map "" " VOID"))
+ (when map
+ (with-demoted-errors "transient--pop-keymap: %S"
+ (internal-pop-keymap map 'overriding-terminal-local-map)))))
(defun transient--make-transient-map ()
(let ((map (make-sparse-keymap)))
- [elpa] externals/transient updated (3b4202e8ca -> a19faa1c71), Stefan Monnier, 2022/01/19
- [elpa] externals/transient e0b57679f7 1/6: transient--debug: Re-align output, Stefan Monnier, 2022/01/19
- [elpa] externals/transient b466cd9aab 2/6: transient-toggle-debug: New command, Stefan Monnier, 2022/01/19
- [elpa] externals/transient c2875e705a 4/6: transient--delay-post-command: Add debug statement, Stefan Monnier, 2022/01/19
- [elpa] externals/transient 686b7ebc5f 5/6: Fix handling of sub-prefix command that use the minibuffer, Stefan Monnier, 2022/01/19
- [elpa] externals/transient 804feb3b41 3/6: transient--{push, pop}-keymap: Do nothing if keymap is void,
Stefan Monnier <=
- [elpa] externals/transient a19faa1c71 6/6: Return to outer prefix when minibuffer is aborted for sub-prefix, Stefan Monnier, 2022/01/19