* mail/feedmail.el (feedmail-buffer-to-smtpmail):

`smtpmail-via-smtp' now returns the error instead of nil.

1 files changed, 15 insertions, 16 deletions

diff --git a/lisp/mail/feedmail.el b/lisp/mail/feedmail.el
index 934637ecbb..bc9a060427 100644
--- a/lisp/mail/feedmail.el
+++ b/lisp/mail/feedmail.el
@@ -1633,22 +1633,21 @@ local gurus."
   ;; no evil.
   (feedmail-say-debug ">in-> feedmail-buffer-to-smtpmail %s" addr-listoid)
   (require 'smtpmail)
-  (if (not (smtpmail-via-smtp addr-listoid prepped))
-      (progn
-	(set-buffer errors-to)
-	(insert "Send via smtpmail failed.  Probable SMTP protocol error.\n")
-	(insert "Look for details below or in the *Messages* buffer.\n\n")
-	(let ((case-fold-search t)
-	      ;; don't be overconfident about the name of the trace buffer
-	      (tracer (concat "trace.*smtp.*" (regexp-quote smtpmail-smtp-server))))
-	  (mapcar
-	   (lambda (buffy)
-	      (if (string-match tracer (buffer-name buffy))
-		  (progn
-		    (insert "SMTP Trace from " (buffer-name buffy) "\n---------------")
-		    (insert-buffer-substring buffy)
-		    (insert "\n\n"))))
-	   (buffer-list))))))
+  (let ((result (smtpmail-via-smtp addr-listoid prepped)))
+    (when result
+      (set-buffer errors-to)
+      (insert "Send via smtpmail failed: %s" result)
+      (let ((case-fold-search t)
+	    ;; don't be overconfident about the name of the trace buffer
+	    (tracer (concat "trace.*smtp.*" (regexp-quote smtpmail-smtp-server))))
+	(mapcar
+	 (lambda (buffy)
+	   (if (string-match tracer (buffer-name buffy))
+	       (progn
+		 (insert "SMTP Trace from " (buffer-name buffy) "\n---------------")
+		 (insert-buffer-substring buffy)
+		 (insert "\n\n"))))
+	 (buffer-list))))))
 
 (declare-function smtp-via-smtp "ext:smtp" (sender recipients smtp-text-buffer))
 (defvar smtp-server)