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.

diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el
index 45a6daabb0548a15c10e99354e4c35f95544e347..5462f54373cdde9b5f5415875545b3fe08fc0185 100644 (file)
--- 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