[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#13017: 24.2.90; minibuf-eldef-setup-minibuffer should append minibuf
From: |
Christopher Schmidt |
Subject: |
bug#13017: 24.2.90; minibuf-eldef-setup-minibuffer should append minibuf-eldef-update-minibuffer to post-command-hook |
Date: |
Wed, 28 Nov 2012 00:10:54 +0000 (GMT) |
In minibuf-eldef-setup-minibuffer there is this form
(add-hook 'post-command-hook #'minibuf-eldef-update-minibuffer nil t)
I think this is not correct. The APPEND parameter of add-hook should be
t so all changes made to the minibuffer content by previous functions in
post-command-hook are taken into account by
minibuf-eldef-update-minibuffer.
Here is a patch for the trunk.
=== modified file 'lisp/ChangeLog'
--- lisp/ChangeLog 2012-11-27 16:40:04 +0000
+++ lisp/ChangeLog 2012-11-28 00:07:24 +0000
@@ -1,3 +1,8 @@
+2012-11-28 Christopher Schmidt <christopher@ch.ristopher.com>
+
+ * minibuf-eldef.el (minibuf-eldef-setup-minibuffer): Append
+ minibuf-eldef-update-minibuffer to post-command-hook. (Bug#)
+
2012-11-27 Glenn Morris <rgm@gnu.org>
* emacs-lisp/derived.el (derived-mode-make-docstring):
=== modified file 'lisp/minibuf-eldef.el'
--- lisp/minibuf-eldef.el 2012-11-07 20:43:38 +0000
+++ lisp/minibuf-eldef.el 2012-11-28 00:05:45 +0000
@@ -140,7 +140,7 @@
(minibuffer-contents-no-properties))
(setq minibuf-eldef-initial-buffer-length (point-max))
(add-to-list 'minibuf-eldef-frobbed-minibufs (current-buffer))
- (add-hook 'post-command-hook #'minibuf-eldef-update-minibuffer nil t))))
+ (add-hook 'post-command-hook #'minibuf-eldef-update-minibuffer t t))))
;; post-command-hook to swap prompts when necessary
(defun minibuf-eldef-update-minibuffer ()
Christopher
- bug#13017: 24.2.90; minibuf-eldef-setup-minibuffer should append minibuf-eldef-update-minibuffer to post-command-hook,
Christopher Schmidt <=