From efb24ce1059efc1f7605420e5bc23f47cbbad8f4 Mon Sep 17 00:00:00 2001 From: Michal Sojka Date: Tue, 3 Nov 2015 18:03:08 +0100 Subject: [PATCH] emacs: Fix regression in (notmuch-)message-mode initialization TODO Recent addition of notmuch-message-mode introduced a few regressions. I use message-setup-hook to set some buffer local variables and this stopped working. The reason is that notmuch-message-mode, which cleans all buffer local variables, is called after calling message-mail, which calls the message-setup-hook. Another problem with the previous implementation might be that the message-mode-hook is called twice - first when message-mail calls message-mode and second when notmuch-message-mode calls it. This can be problematic when the hook has side effects. This commit uses advice mechanism to call notmuch-message-mode instead of message-mode. This way, a call to message-mail initializes directly notmuch-message-mode rather than message-mode which is later changed to notmuch-message-mode. The advice is constructed in such a way, that it is effective only once and when called by notmuch. The second call to message-mode (from notmuch-message-mode) calls the original message-mode. This implementation uses the new advice mechanism introduced in Emacs 24.4. If we want to support older version, this must be changed. --- emacs/notmuch-mua.el | 49 ++++++++++++++++++++++++++++++++++++++------ 1 file changed, 43 insertions(+), 6 deletions(-) diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el index 466edd7e..a66a3066 100644 --- a/emacs/notmuch-mua.el +++ b/emacs/notmuch-mua.el @@ -278,10 +278,36 @@ Note that these functions use `mail-citation-hook' if that is non-nil." (define-key notmuch-message-mode-map (kbd "C-c C-c") #'notmuch-mua-send-and-exit) (define-key notmuch-message-mode-map (kbd "C-c C-s") #'notmuch-mua-send) -(defun notmuch-mua-mail (&optional to subject other-headers &rest other-args) - "Invoke the notmuch mail composition window. - -OTHER-ARGS are passed through to `message-mail'." +(defun notmuch-mua-pop-to-buffer (name) + "Pop to buffer NAME, and warn if it already exists and is +modified. This function is notmuch addaptation of +`message-pop-to-buffer'." + (let ((buffer (get-buffer name))) + (if (and buffer + (buffer-name buffer)) + (let ((window (get-buffer-window buffer 0))) + (if window + ;; Raise the frame already displaying the message buffer. + (progn + (gnus-select-frame-set-input-focus (window-frame window)) + (select-window window)) + (funcall (notmuch-mua-get-switch-function) buffer) + (set-buffer buffer)) + (when (and (buffer-modified-p) + (not (prog1 + (y-or-n-p + "Message already being composed; erase? ") + (message nil)))) + (error "Message being composed"))) + (funcall (notmuch-mua-get-switch-function) name) + (set-buffer name)) + (erase-buffer) + (notmuch-message-mode))) + +(defun notmuch-mua-mail (&optional to subject other-headers continue + switch-function yank-action send-actions + return-action &rest ignored) + "Invoke the notmuch mail composition window." (interactive) (when notmuch-mua-user-agent-function @@ -293,8 +319,19 @@ OTHER-ARGS are passed through to `message-mail'." (push (cons 'From (concat (notmuch-user-name) " <" (notmuch-user-primary-email) ">")) other-headers)) - (apply #'message-mail to subject other-headers other-args) - (notmuch-message-mode) + (notmuch-mua-pop-to-buffer (message-buffer-name "mail" to)) + (message-setup-1 + ;; The following sexp is copied from `message-mail' + (nconc + `((To . ,(or to "")) (Subject . ,(or subject ""))) + ;; C-h f compose-mail says that headers should be specified as + ;; (string . value); however all the rest of message expects + ;; headers to be symbols, not strings (eg message-header-format-alist). + ;; http://lists.gnu.org/archive/html/emacs-devel/2011-01/msg00337.html + ;; We need to convert any string input, eg from rmail-start-mail. + (dolist (h other-headers other-headers) + (if (stringp (car h)) (setcar h (intern (capitalize (car h))))))) + yank-action send-actions return-action) (notmuch-fcc-header-setup) (message-sort-headers) (message-hide-headers) -- 2.39.2