emacs: Remove conditional from notmuch-fcc-initialization.
authorJesse Rosenthal <jrosenthal@jhu.edu>
Tue, 27 Apr 2010 16:33:57 +0000 (12:33 -0400)
committerCarl Worth <cworth@cworth.org>
Tue, 27 Apr 2010 18:10:50 +0000 (11:10 -0700)
The fcc code would only initialize if notmuch-fcc-dirs was set. This was
a problem if you reset the variable, or added the variable later during
initialization. Now we always add the fcc hook, but it doesn't do
anything unless notmuch-fcc-dirs are set.
(cherry picked from commit 80a90787163690d2d87571327ba504a470798c60)

emacs/notmuch-maildir-fcc.el

index 8af6f009e4289afa767b4ba05ab6f709ccca5eb4..d9a8f9b6c258d2ca3db3c37a85a31fb9bf8d988c 100644 (file)
@@ -47,7 +47,6 @@
 (defun notmuch-fcc-initialization ()
   "If notmuch-fcc-directories is set,
    hook them into the message-fcc-handler-function"
-(if (not (eq notmuch-fcc-dirs nil)) (progn
     ;Set up the message-fcc-handler to move mails to the maildir in Fcc
     ;The parameter is hardcoded to mark messages as "seen"
     (setq message-fcc-handler-function
@@ -57,7 +56,7 @@
     ;(preferrably we would use message-header-setup-up, but notmuch-reply
     ; munges headers after that is run, so it won't work for replies within
     ; notmuch)
-    (add-hook 'message-send-hook 'notmuch-fcc-header-setup))))
+    (add-hook 'message-send-hook 'notmuch-fcc-header-setup))
 
 (defun notmuch-fcc-header-setup ()
   "Can be added to message-send-hook and will set the FCC header