diff options
| author | Michal Sojka <sojkam1@fel.cvut.cz> | 2016-01-06 22:28:45 +0100 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2016-01-08 08:32:49 -0400 |
| commit | 10f4724a8d51a411d8f63d9449a38cd71bee6ae9 (patch) | |
| tree | a849ed059ac1c37df94d38139cf93c29673483c0 | |
| parent | 1de4d5bb0d2bdcf9d14736b86a417d32e8c8668a (diff) | |
emacs: Don't use nconc on quoted list
As pointed out by David Bremner, Elisp manual says "A common pitfall
is to use a quoted constant list as a non-last argument to ‘nconc’."
Since this was the case in recently added code, we fix it here.
| -rw-r--r-- | emacs/notmuch-mua.el | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el index 45a6daab..5462f543 100644 --- a/emacs/notmuch-mua.el +++ b/emacs/notmuch-mua.el @@ -322,8 +322,8 @@ modified. This function is notmuch addaptation of (notmuch-mua-pop-to-buffer (message-buffer-name "mail" to) (or switch-function (notmuch-mua-get-switch-function))) (let ((headers - ;; The following sexp is copied from `message-mail' - (nconc + (append + ;; The following is copied from `message-mail' `((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 |