emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: feedmail.el in the public domain?


From: Stefan Monnier
Subject: Re: feedmail.el in the public domain?
Date: Mon, 17 Sep 2018 22:53:12 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)

> That is true IF we have made substantial changes.
> But if we have not made much change, then the file is still in
> the public domain.  We should check the facts, not just presume.

Indeed.  See attached patch between the initial version and the
current version.
A large part of the patch is cosmetic, but the non-cosmetic part is
quite larger than the "20 lines worth" test we use for trivial changes.


        Stefan


diff --git a/lisp/mail/feedmail.el b/lisp/mail/feedmail.el
index 38d4c55680..ec4a1162b2 100644
--- a/lisp/mail/feedmail.el
+++ b/lisp/mail/feedmail.el
@@ -1,32 +1,44 @@
-;;; feedmail.el --- assist other email packages to massage outgoing messages 
-;;; A replacement for parts of GNUemacs' sendmail.el (specifically,
-;;; it's what handles your outgoing mail after you hit C-c C-c in mail
-;;; mode).  See below for a list of additional features, including the
-;;; ability to queue messages for later sending.  If you are using
-;;; fakemail as a subprocess, you can switch to feedmail and eliminate
-;;; the use of fakemail.  feedmail works with recent versions of
-;;; GNUemacs (mostly, but not exclusively, tested against 19.34 on
-;;; Win95; some testing on 20.x) and XEmacs (tested with 20.4 and
-;;; later betas).  It probably no longer works with GNUemacs v18,
-;;; though I haven't tried that in a long time.  Sorry, no manual yet
-;;; in this release.  Look for one with the next release.
+;;; feedmail.el --- assist other email packages to massage outgoing messages  
-*- lexical-binding:t -*-
+
+;; This file is in the public domain.
+
+;; This file is part of GNU Emacs.
+
+;; Author: Bill Carpenter <address@hidden>
+;; Version: 11
+;; Keywords: email, queue, mail, sendmail, message, spray, smtp, draft
+;; X-URL: <URL:http://www.carpenter.org/feedmail/feedmail.html>
+
+;;; Commentary:
+
+;; A replacement for parts of sendmail.el (specifically,
+;; it's what handles your outgoing mail after you hit C-c C-c in mail
+;; mode).  See below for a list of additional features, including the
+;; ability to queue messages for later sending.  This replaces
+;; the standalone fakemail program that used to be distributed with Emacs.
+
+;; feedmail works with recent versions of Emacs (20.x series) and
+;; XEmacs (tested with 20.4 and later betas).  It probably no longer
+;; works with Emacs v18, though I haven't tried that in a long
+;; time.  address@hidden reports: "I have a report
+;; that with a help of APEL library, feedmail works fine under emacs
+;; 19.28.  You can get APEL from ftp://ftp.m17n.org/pub/mule/apel/.
+;; you need apel-10.2 or later to make feedmail work under emacs
+;; 19.28."
+
+;; Sorry, no manual yet in this release.  Look for one with the next
+;; release.  Or the one after that.  Or maybe later.
 
 ;; As far as I'm concerned, anyone can do anything they want with
 ;; this specific piece of code.  No warranty or promise of support is
 ;; offered.  This code is hereby released into the public domain.
 
-;; Author: Bill Carpenter <address@hidden>, <address@hidden>
-;; Version: 8
-;; Keywords: email, queue, mail, sendmail, message, spray, smtp, draft
-;; Where: <URL:http://www.carpenter.org/feedmail/feedmail.html>
 ;; Thanks: My thanks to the many people who have sent me suggestions
 ;;    and fixes over time, as well as those who have tested many beta
 ;;    iterations.  Some are cited in comments in code fragments below,
 ;;    but that doesn't correlate well with the list of folks who have
 ;;    actually helped me along the way.
 
-;;; Commentary:
-;;
 ;; If you use feedmail, I invite you to send me some email about it.
 ;; I appreciate feedback about problems you find or suggestions for
 ;; improvements or added features (even though I can't predict when
@@ -35,11 +47,11 @@
 ;; tried feedmail and didn't find it useful, so I stopped using it".
 ;;
 ;; It is most useful, when sending a bug report, if you tell me what
-;; version of emacs you are using, what version of feedmail you are
+;; version of Emacs you are using, what version of feedmail you are
 ;; using, and what versions of other email-related elisp packages you
 ;; are using.  If in doubt about any of that, send the bug report
 ;; anyhow.
-;; 
+;;
 ;; =====
 ;; A NOTE TO THOSE WHO WOULD CHANGE THIS CODE...  Since it is PD,
 ;; you're within your rights to do whatever you want.  If you do
@@ -63,7 +75,8 @@
 ;; This file requires the mail-utils library.
 ;;
 ;; This file requires the smtpmail library if you use
-;; feedmail-buffer-to-smtpmail.
+;; feedmail-buffer-to-smtpmail.  It requires the smtp library if
+;; you use feedmail-buffer-smtp.
 ;;
 ;; This file requires the custom library.  Unfortunately, there are
 ;; two incompatible versions of the custom library.  If you don't have
@@ -71,18 +84,17 @@
 ;; work properly.  If you don't know what custom is all about and want
 ;; to edit your user option elisp variables the old fashioned way,
 ;; just imagine that all the "defcustom" stuff you see below is really
-;; "defvar", and ignore everthing else.  For info about custom, see
+;; "defvar", and ignore everything else.  For info about custom, see
 ;; <URL:http://www.dina.kvl.dk/~abraham/custom/>.
 ;;
 ;; This code does in elisp a superset of the stuff that used to be done
 ;; by the separate program "fakemail" for processing outbound email.
 ;; In other words, it takes over after you hit "C-c C-c" in mail mode.
-;; By appropriate setting of options, you can still use "fakemail",
-;; or you can even revert to sendmail (which is not too popular
-;; locally).  See the variables at the top of the elisp for how to
-;; achieve these effects (there are more features than in this bullet
-;; list, so trolling through the variable and function doc strings may
-;; be worth your while):
+;; By appropriate setting of options, you can even revert to sendmail
+;; (which is not too popular locally).  See the variables at the top
+;; of the elisp for how to achieve these effects (there are more
+;; features than in this bullet list, so trolling through the variable
+;; and function doc strings may be worth your while):
 ;;
 ;;    --- you can park outgoing messages into a disk-based queue and
 ;;        stimulate sending them all later (handy for laptop users);
@@ -91,19 +103,19 @@
 ;;    --- you can get one last look at the prepped outbound message and
 ;;        be prompted for confirmation
 ;;
-;;    --- removes BCC:/RESENT-BCC: headers after getting address info
+;;    --- removes Bcc:/Resent-Bcc: headers after getting address info
 ;;
 ;;    --- does smart filling of address headers
 ;;
-;;    --- calls a routine to process FCC: lines and removes them
+;;    --- calls a routine to process Fcc: lines and removes them
 ;;
 ;;    --- empty headers are removed
 ;;
-;;    --- can force FROM: or SENDER: line
+;;    --- can force From: or Sender: line
 ;;
-;;    --- can generate a MESSAGE-ID: line
+;;    --- can generate a Message-Id: line
 ;;
-;;    --- can generate a DATE: line; the date can be the time the
+;;    --- can generate a Date: line; the date can be the time the
 ;;        message was written or the time it is being sent
 ;;
 ;;    --- strips comments from address info (both "()" and "<>" are
@@ -122,15 +134,14 @@
 ;;    --- you can specify something other than /bin/mail for the
 ;;        subprocess
 ;;
-;;    --- you can generate/modify an X-MAILER: message header
+;;    --- you can generate/modify an X-Mailer: message header
 ;;
 ;; After a long list of options below, you will find the function
 ;; feedmail-send-it. Hers's the best way to use the stuff in this
 ;; file:
 ;;
-;; Save this file as feedmail.el somewhere on your elisp
-;; loadpath; byte-compile it.  Put the following lines somewhere in
-;; your ~/.emacs stuff:
+;; Save this file as feedmail.el somewhere on your elisp loadpath;
+;; byte-compile it.  Put the following lines in your init file:
 ;;
 ;;     (setq send-mail-function 'feedmail-send-it)
 ;;     (autoload 'feedmail-send-it "feedmail")
@@ -142,7 +153,33 @@
 ;;     (autoload 'feedmail-run-the-queue-no-prompts "feedmail")
 ;;     (setq auto-mode-alist (cons '("\\.fqm$" . mail-mode) auto-mode-alist))
 ;;
-;; If you are using the desktop.el library to restore your sessions, you might 
+;; though VM users might find it more comfortable to use this instead of
+;; the above example's last line:
+;;
+;;     (setq auto-mode-alist (cons '("\\.fqm$" . feedmail-vm-mail-mode) 
auto-mode-alist))
+;;
+;; If you end up getting asked about killing modified buffers all the time
+;; you are probably being prompted from outside feedmail.  You can probably
+;; get cured by doing the defadvice stuff described in the documentation
+;; for the variable feedmail-queue-buffer-file-name below.
+;;
+;; If you are wondering how to send your messages to some SMTP server
+;; (which is not really a feedmail-specific issue), you are probably
+;; looking for smtpmail.el, and it is probably already present in your
+;; emacs installation.  Look at smtpmail.el for how to set that up, and
+;; then do this to hook it into feedmail:
+;;
+;;     (autoload 'feedmail-buffer-to-smtpmail "feedmail" nil t)
+;;     (setq feedmail-buffer-eating-function 'feedmail-buffer-to-smtpmail)
+;;
+;; Alternatively, the FLIM <http://www.m17n.org/FLIM/> project
+;; provides a library called smtp.el.  If you want to use that, the above lines
+;; would be:
+;;
+;;     (autoload 'feedmail-buffer-to-smtp "feedmail" nil t)
+;;     (setq feedmail-buffer-eating-function 'feedmail-buffer-to-smtp)
+;;
+;; If you are using the desktop.el library to restore your sessions, you might
 ;; like to add the suffix ".fqm" to the list of non-saved things via the 
variable
 ;; desktop-files-not-to-save.
 ;;
@@ -169,20 +206,34 @@
 ;;     (setq message-send-mail-function 'feedmail-send-it)
 ;;     (add-hook 'message-mail-send-hook 'feedmail-mail-send-hook-splitter)
 ;;
+;; If you use message-mode and you make use of feedmail's queueing
+;; stuff, you might also like to adjust these variables to appropriate
+;; values for message-mode:
+;;
+;;     feedmail-queue-runner-mode-setter
+;;     feedmail-queue-runner-message-sender
+;;
+;; If you are using the "cmail" email package, there is some built-in
+;; support for feedmail in recent versions.  To enable it, you should:
+;;
+;;     (setq cmail-use-feedmail t)
+;;
+;;;;;;;;
+;;
 ;; I think the LCD is no longer being updated, but if it were, this
 ;; would be a proper LCD record.  There is an old version of
 ;; feedmail.el in the LCD archive.  It works but is missing a lot of
 ;; features.
 ;;
 ;; LCD record:
-;; feedmail|Bill Carpenter|address@hidden,address@hidden|Outbound mail queue 
handling|98-06-15|8|feedmail.el
+;; feedmail|WJCarpenter|address@hidden|Outbound mail queue 
handling|01-??-??|11-beta-??|feedmail.el
 ;;
 ;; Change log:
 ;; original,      31 March 1991
 ;; patchlevel 1,   5 April 1991
 ;; patchlevel 2,  24 May   1991
 ;; 5-may-92  jwz       Conditionalized calling expand-mail-aliases, since that
-;;                     function doesn't exist in Lucid GNU Emacs or when using
+;;                     function doesn't exist in Lucid Emacs or when using
 ;;                     mail-abbrevs.el.
 ;; patchlevel 3,   3 October 1996
 ;;         added queue stuff; still works in v18
@@ -190,19 +241,19 @@
 ;; patchlevel 5, issued by someone else
 ;; patchlevel 6, not issued as far as I know
 ;; patchlevel 7,  20 May 1997
-;;         abandon futile support of GNUemacs v18 (sorry if that hurts you)
-;;         provide a DATE: header by default
-;;         provide a default for generating MESSAGE-ID: header contents
+;;         abandon futile support of Emacs 18 (sorry if that hurts you)
+;;         provide a Date: header by default
+;;         provide a default for generating Message-Id: header contents
 ;;            and use it by default (slightly changed API)
 ;;         return value from feedmail-run-the-queue
 ;;         new wrapper function feedmail-run-the-queue-no-prompts
-;;         user-mail-address as default for FROM:
-;;         properly deal with RESENT-{TO,CC,BCC}
-;;         BCC and RESENT-* now included in smart filling
+;;         user-mail-address as default for From:
+;;         properly deal with Resent-{To,Cc,Bcc}
+;;         Bcc and Resent-* now included in smart filling
 ;;         limited support for a "drafts" directory
 ;;         user-configurable default message action
 ;;         allow timeout for confirmation prompt (where available)
-;;         move FCC handling to as late as possible to get max
+;;         move Fcc handling to as late as possible to get max
 ;;            header munging in the saved file
 ;;         work around sendmail.el's prompts when working from queue
 ;;         more reliably detect voluntary user bailouts
@@ -223,7 +274,7 @@
 ;;            any other way
 ;;         cleanup some of my sloppiness about case-fold-search (a strange
 ;;            variable)
-;;         best effort following coding conventions from GNUemacs
+;;         best effort following coding conventions from Emacs
 ;;            elisp manual appendix
 ;;         "customize" (see custom.el)
 ;;         when user selects "immediate send", clear action prompt since
@@ -236,10 +287,10 @@
 ;;         eliminate reliance on directory-sep-char and feedmail-sep-thing
 ;;         tweak smart filling (reminded of comma problem by address@hidden)
 ;;         option to control writing in text vs binary mode
-aqq2;; patchlevel 8, 15 June 1998
+;; patchlevel 8, 15 June 1998
 ;;         reliable re-editing of text-mode (vs binary) queued messages
-;;         user option to keep BCC: in FCC: copy (keep by default)
-;;         user option to delete body from FCC: copy (keep by default)
+;;         user option to keep Bcc: in Fcc: copy (keep by default)
+;;         user option to delete body from Fcc: copy (keep by default)
 ;;         feedmail-deduce-bcc-where for envelope (API change for
 ;;           feedmail-deduce-address list)
 ;;         feedmail-queue-alternative-mail-header-separator
@@ -261,7 +312,7 @@
 ;;           address@hidden (Shane Burgess); bumped up the default value of
 ;;           feedmail-queue-chatty-sit-for since info is more complex sometimes
 ;;         feedmail-enable-spray (individual transmissions, crude mailmerge)
-;;         blank SUBJECT: no longer a special case; see 
feedmail-nuke-empty-headers
+;;         blank Subject: no longer a special case; see 
feedmail-nuke-empty-headers
 ;;         fiddle-plexes data structure used lots of places; see 
feedmail-fiddle-plex-blurb
 ;;         feedmail-fiddle-plex-user-list
 ;;         feedmail-is-a-resend
@@ -272,34 +323,60 @@
 ;;         feedmail-queue-auto-file-nuke
 ;;         feedmail-queue-express-to-queue and feedmail-queue-express-to-draft
 ;;         strong versions of "q"ueue and "d"raft answers (always make a new 
file)
+;; patchlevel 9, 23 March 2001
+;;         feedmail-queue-buffer-file-name to work around undesirable 
mail-send prompt
+;;         at message action prompt, can scroll message buffer with "<" and 
">";
+;;           C-v no longer scrolls help buffer
+;;         conditionalize (discard-input) in message action prompt to avoid 
killing
+;;           define-kbd-macro
+;;         fixed error if feedmail-x-mailer-line was nil
+;;         feedmail-binmail-template only uses /bin/rmail if it exists
+;;         relocate feedmail-queue-alternative-mail-header-separator stuff
+;;         added feedmail-vm-mail-mode, which make a good auto-mode-alist entry
+;;           for FQM files if you're a VM user
+;;         change buffer-substring calls to buffer-substring-no-properties for
+;;           speed-up (suggested by Howard Melman <address@hidden>)
+;;         feedmail-sendmail-f-doesnt-sell-me-out to control "-f" in call to 
sendmail
+;;           in feedmail-buffer-to-sendmail
+;;         better trapping of odd conditions during the running of the queue;
+;;           thanks to Yigal Hochberg for helping me test much of this by 
remote
+;;           control
+;;         feedmail-debug and feedmail-debug-sit-for
+;;         feedmail-display-full-frame
+;;         feedmail-queue-express-hook
+;;         added example function feedmail-spray-via-bbdb
+;;         use expand-file-name for setting default directory names
+;;         define feedmail-binmail-gnulinuxish-template as a suggestion for
+;;           the value of feedmail-binmail-template on GNU/Linux and maybe 
other
+;;           systems with non-classic /bin/[r]mail behavior
+;;         guard against nil user-mail-address in generating MESSAGE-ID:
+;;         feedmail-queue-slug-suspect-regexp is now a variable to
+;;           accommodate non-ASCII environments (thanks to
+;;           address@hidden for this suggestion)
+;;         feedmail-buffer-to-smtp, to parallel feedmail-buffer-to-smtpmail
+;; patchlevel 10, 22 April 2001
+;;         DATE: and MESSAGE-ID stuff now forces system-time-locale to "C"
+;;           (brought to my attention by address@hidden)
+;; patchlevel 11
+;;         tweak default FROM: calculation to look at mail-host-address
+;;           (suggested by "Jason Eisner" <address@hidden>)
 ;;
-;; todo (probably in patchlevel 9):
+;; todo:
 ;;         write texinfo manual
 ;;         maybe partition into multiple files, including files of examples
 ;;
 ;;; Code:
 
-(defconst feedmail-patch-level "8")
+(defconst feedmail-patch-level "11-beta-1")
 
+(require 'mail-utils)               ; pick up mail-strip-quoted-names
 
-;; from <URL:http://www.dina.kvl.dk/~abraham/custom/>:
-;; If you write software that must work without the new custom, you
-;; can use this hack stolen from w3-cus.el:
-(eval-and-compile
-  (condition-case ()
-      (require 'custom)
-    (error nil))
-  (if (and (featurep 'custom) (fboundp 'custom-declare-variable))
-      nil ;; We've got what we needed
-    ;; We have the old custom-library, hack around it!
-    (defmacro defgroup (&rest args)
-      nil)
-    (defmacro defcustom (var value doc &rest args)
-      (` (defvar (, var) (, value) (, doc))))))
-
+(autoload 'mail-do-fcc "sendmail")
 
 (defgroup feedmail nil
   "Assist other email packages to massage outgoing messages."
+  :link '(url-link "http://www.carpenter.org/feedmail/feedmail.html";)
+  :link '(emacs-commentary-link "feedmail")
   :group 'mail)
 
 (defgroup feedmail-misc nil
@@ -314,145 +391,172 @@ feedmail-queue
   "Options related to queuing messages for later sending."
   :group 'feedmail)
 
+(defgroup feedmail-debug nil
+  "Options related to debug messages for later sending."
+  :group 'feedmail)
+
 
 (defcustom feedmail-confirm-outgoing nil
-  "*If non-nil, give a y-or-n confirmation prompt before sending mail.
+  "If non-nil, give a y-or-n confirmation prompt before sending mail.
 This is done after the message is completely prepped, and you'll be
 looking at the top of the message in a buffer when you get the prompt.
-If set to the symbol 'queued, give the confirmation prompt only while
+If set to the symbol `queued', give the confirmation prompt only while
 running the queue (however, the prompt is always suppressed if you are
-processing the queue via feedmail-run-the-queue-no-prompts).  If set
-to the symbol 'immediate, give the confirmation prompt only when
+processing the queue via `feedmail-run-the-queue-no-prompts').  If set
+to the symbol `immediate', give the confirmation prompt only when
 sending immediately.  For any other non-nil value, prompt in both
 cases.  You can give a timeout for the prompt; see variable
-feedmail-confirm-outgoing-timeout."
+`feedmail-confirm-outgoing-timeout'."
   :group 'feedmail-misc
-  :type 'boolean
-  )
+  :type '(choice (const nil)
+                (const queued)
+                (const immediate)
+                (other t)))
+
+
+(defcustom feedmail-display-full-frame 'queued
+  "If non-nil, show prepped messages in a full frame.
+If nil, the prepped message will be shown, for confirmation or
+otherwise, in some window in the current frame without resizing
+anything.  That may or may not display enough of the message to
+distinguish it from others.  If set to the symbol `queued', take
+this action only when running the queue.  If set to the symbol
+`immediate', take this action only when sending immediately.  For
+any other non-nil value, take the action in both cases.  Even if
+you're not confirming the sending of immediate or queued messages,
+it can still be interesting to see a lot about them as they are
+shuttled robotically onward."
+  :version "24.1"
+  :group 'feedmail-misc
+  :type '(choice (const nil)
+                (const queued)
+                (const immediate)
+                (other t)))
 
 
 (defcustom feedmail-confirm-outgoing-timeout nil
-  "*If non-nil, a timeout in seconds at the send confirmation prompt.
+  "If non-nil, a timeout in seconds at the send confirmation prompt.
 If a positive number, it's a timeout before sending.  If a negative
 number, it's a timeout before not sending.  This will not work if your
-version of emacs doesn't include the function y-or-n-p-with-timeout
-(e.g., some versions of XEmacs)."
+version of Emacs doesn't include the function `y-or-n-p-with-timeout'
+\(e.g., some versions of XEmacs)."
+  :version "24.1"
   :group 'feedmail-misc
   :type '(choice (const nil) integer)
-)
+  )
 
 
 (defcustom feedmail-nuke-bcc t
-  "*If non-nil remove BCC: lines from the message headers.
-In any case, the BCC: lines do participate in the composed address
+  "If non-nil remove Bcc: lines from the message headers.
+In any case, the Bcc: lines do participate in the composed address
 list.  You may want to leave them in if you're using sendmail
-(see feedmail-buffer-eating-function)."
+\(see `feedmail-buffer-eating-function')."
   :group 'feedmail-headers
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-nuke-resent-bcc t
-  "*If non-nil remove RESENT-BCC: lines from the message headers.
-In any case, the RESENT-BCC: lines do participate in the composed
+  "If non-nil remove Resent-Bcc: lines from the message headers.
+In any case, the Resent-Bcc: lines do participate in the composed
 address list.  You may want to leave them in if you're using sendmail
-(see feedmail-buffer-eating-function)."
+\(see `feedmail-buffer-eating-function')."
   :group 'feedmail-headers
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-deduce-bcc-where nil
-  "*Where should BCC:/RESENT-BCC: addresses appear in the envelope list?
+  "Where Bcc:/Resent-Bcc: addresses should appear in the envelope list.
 Addresses for the message envelope are deduced by examining
 appropriate address headers in the message.  Generally, they will show
 up in the list of deduced addresses in the order that the headers
 happen to appear (duplicate addresses are eliminated in any case).
-This variable can be set to the symbol 'first, in which case the
-BCC:/RESENT-BCC: addresses will appear at the beginning in the list;
-or, it can be set to the symbol 'last, in which case they will appear
+This variable can be set to the symbol `first', in which case the
+Bcc:/Resent-Bcc: addresses will appear at the beginning in the list;
+or, it can be set to the symbol `last', in which case they will appear
 at the end of the list.
 
 Why should you care?  Well, maybe you don't, and certainly the same
 things could be accomplished by affecting the order of message headers
-in the outgoing message.  Some people use BCC: as a way of getting
-their own \"come back\" copy of each message they send.  If BCC:
+in the outgoing message.  Some people use Bcc: as a way of getting
+their own \"come back\" copy of each message they send.  If Bcc:
 addresses are not handled first, there can be substantial delays in
 seeing the message again.  Some configurations of sendmail, for example,
 seem to try to deliver to each addressee at least once, immediately
 and serially, so slow SMTP conversations can add up to a delay.  There
-is an option for either 'first or 'last because you might have a
+is an option for either `first' or `last' because you might have a
 delivery agent that processes the addresses backwards."
   :group 'feedmail-headers
-  :type 'boolean
-)
+  :type '(choice (const nil)
+                (const first)
+                (const last)))
 
 
 (defcustom feedmail-fill-to-cc t
-  "*If non-nil do smart filling of addressee header lines.
+  "If non-nil do smart filling of addressee header lines.
 Smart filling means breaking long lines at appropriate points and
 making continuation lines.  Despite the function name, it includes
-TO:, CC:, BCC: (and their RESENT-* forms), as well as FROM: and
-REPLY-TO: (though they seldom need it).  If nil, the lines are left
+To:, Cc:, Bcc: (and their Resent-* forms), as well as From: and
+Reply-To: (though they seldom need it).  If nil, the lines are left
 as-is.  The filling is done after mail address alias expansion."
   :group 'feedmail-headers
   :type 'boolean
-)
+  )
 
 
-(defcustom feedmail-fill-to-cc-fill-column default-fill-column
-  "*Fill column used by feedmail-fill-to-cc."
+(defcustom feedmail-fill-to-cc-fill-column (default-value 'fill-column)
+  "Fill column used by `feedmail-fill-to-cc'."
   :group 'feedmail-headers
   :type 'integer
-)
+  )
 
 
 (defcustom feedmail-nuke-bcc-in-fcc nil
-  "*If non-nil remove [RESENT-]BCC: lines in message copies saved via FCC:.
-This is independent of whether the BCC: header lines are actually sent
+  "If non-nil remove [Resent-]Bcc: lines in message copies saved via Fcc:.
+This is independent of whether the Bcc: header lines are actually sent
 with the message (see feedmail-nuke-bcc).  Though not implied in the name,
-the same FCC: treatment applies to both BCC: and RESENT-BCC: lines."
+the same Fcc: treatment applies to both Bcc: and Resent-Bcc: lines."
   :group 'feedmail-headers
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-nuke-body-in-fcc nil
-  "*If non-nil remove body of message in copies saved via FCC:.
-If an positive integer value, leave (up to) that many lines of the
-beginning of the body intact.  The result is that the FCC: copy will
+  "If non-nil remove body of message in copies saved via Fcc:.
+If a positive integer value, leave (up to) that many lines of the
+beginning of the body intact.  The result is that the Fcc: copy will
 consist only of the message headers, serving as a sort of an outgoing
 message log."
   :group 'feedmail-headers
   :type '(choice (const nil) (const t) integer)
-;;  :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-force-expand-mail-aliases nil
-  "*If non-nil force the calling of expand-mail-aliases.
+  "If non-nil, force the calling of `expand-mail-aliases'.
 Normally, feedmail tries to figure out if you're using mailalias or
-mailabbrevs and only calls expand-mail-aliases if it thinks you're
+mailabbrevs and only calls `expand-mail-aliases' if it thinks you're
 using the mailalias package.  This user option can be used to force
 the issue since there are configurations which fool the figuring
 out."
   :group 'feedmail-headers
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-nuke-empty-headers t
-  "*If non-nil, remove header lines which have no contents.
-A completely empty SUBJECT: header is always removed, regardless of
+  "If non-nil, remove header lines which have no contents.
+A completely empty Subject: header is always removed, regardless of
 the setting of this variable.  The only time you would want them left
 in would be if you used some headers whose presence indicated
 something rather than their contents.  This is rare in Internet email
 but common in some proprietary systems."
   :group 'feedmail-headers
   :type 'boolean
-)
+  )
 
-;; wjc sez:  I think the use of the SENDER: line is pretty pointless,
+;; wjc sez:  I think the use of the Sender: line is pretty pointless,
 ;; but I left it in to be compatible with sendmail.el and because
 ;; maybe some distant mail system needs it.  Really, though, if you
 ;; want a sender line in your mail, just put one in there and don't
@@ -460,12 +564,12 @@ feedmail-nuke-empty-headers
 ;; RFC-822 and RFC-1123, but are you *really* one of those cases
 ;; they're talking about?  I doubt it.)
 (defcustom feedmail-sender-line nil
-  "*If non-nil and the email has no SENDER: header, use this value.
+  "If non-nil and the email has no Sender: header, use this value.
 May be nil, in which case nothing in particular is done with respect
-to SENDER: lines.  By design, will not replace an existing SENDER:
-line, but you can achieve that with a fiddle-plex 'replace action.
+to Sender: lines.  By design, will not replace an existing Sender:
+line, but you can achieve that with a fiddle-plex replace action.
 NB: it makes no sense to use the value t since there is no sensible
-default for SENDER:.
+default for Sender:.
 
 If not nil, it may be a string, a fiddle-plex, or a function which
 returns either nil, t, a string, or a fiddle-plex (or, in fact,
@@ -473,38 +577,37 @@ feedmail-sender-line
 should be just the contents of the header, not the name of the header
 itself nor the trailing newline.  If a function, it will be called
 with no arguments.  For an explanation of fiddle-plexes, see the
-documentation for the variable feedmail-fiddle-plex-blurb.  In all
+documentation for the variable `feedmail-fiddle-plex-blurb'.  In all
 cases the name element of the fiddle-plex is ignored and is hardwired
-by feedmail to either \"X-Sender\" or \"X-Resent-Sender\".
+by feedmail to either \"Sender\" or \"Resent-Sender\".
 
 You can probably leave this nil, but if you feel like using it, a good
 value would be a string of a fully-qualified domain name form of your
-address.  For example, \"address@hidden (WJCarpenter)\".  The SENDER:
-header is fiddled after the FROM: header is fiddled."
+address.  For example, \"address@hidden (WJCarpenter)\".  The Sender:
+header is fiddled after the From: header is fiddled."
   :group 'feedmail-headers
   :type '(choice (const nil) string)
-)
+  )
 
 
 (defcustom feedmail-force-binary-write t
-  "*If non-nil, force writing file as binary.  Applies to queues and FCC:.
+  "If non-nil, force writing file as binary (this applies to queues and Fcc:).
 On systems where there is a difference between binary and text files,
-feedmail will temporarily manipulate the values of buffer-file-type
-and/or default-buffer-file-type to make the writing as binary.  If
-nil, writing will be in text mode.  On systems where there is no
-distinction or where it is controlled by other variables or other
-means, this option has no effect."
+feedmail will temporarily manipulate the value of `coding-system-for-write'
+to make the writing as binary.  If nil, writing will be in text mode.
+On systems where there is no distinction or where it is controlled by other
+variables or other means, this option has no effect."
   :group 'feedmail-misc
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-from-line t
-  "*If non-nil and the email has no FROM: header, use this value.
+  "If non-nil and the email has no From: header, use this value.
 May be t, in which case a default is computed (and you probably won't
 be happy with it).  May be nil, in which case nothing in particular is
-done with respect to FROM: lines.  By design, will not replace an
-existing FROM: line, but you can achieve that with a fiddle-plex 'replace
+done with respect to From: lines.  By design, will not replace an
+existing From: line, but you can achieve that with a fiddle-plex `replace'
 action.
 
 If neither nil nor t, it may be a string, a fiddle-plex, or a function
@@ -513,33 +616,56 @@ feedmail-from-line
 should be just the contents of the header, not the name of the header
 itself nor the trailing newline.  If a function, it will be called
 with no arguments.  For an explanation of fiddle-plexes, see the
-documentation for the variable feedmail-fiddle-plex-blurb.  In all
+documentation for the variable `feedmail-fiddle-plex-blurb'.  In all
 cases the name element of the fiddle-plex is ignored and is hardwired
-by feedmail to either \"X-From\" or \"X-Resent-From\".
+by feedmail to either \"From\" or \"Resent-From\".
 
 A good value would be a string fully-qualified domain name form of
-your address.  For example, \"address@hidden (WJCarpenter)\".  The
+your address.  For example, \"address@hidden (WJCarpenter)\".  The
 default value of this variable uses the standard elisp variable
-user-mail-address which should be set on every system but has a decent
-chance of being wrong.  It also honors mail-from-style.  Better to set
+`user-mail-address' which should be set on every system but has a decent
+chance of being wrong.  It also honors `mail-from-style'.  Better to set
 this variable explicitly to the string you want or find some other way
-to arrange for the message to get a FROM: line."
+to arrange for the message to get a From: line."
   :group 'feedmail-headers
-  :type '(choice (const nil) string)
+  :type '(choice (const t) (const nil) string)
+  )
+
+
+(defcustom feedmail-sendmail-f-doesnt-sell-me-out nil
+  "Says whether the sendmail program issues a warning header if called with 
\"-f\".
+The sendmail program has a useful feature to let you set the envelope FROM
+address via a command line option, \"-f\".  Unfortunately, it also has a widely
+disliked default behavior of selling you out if you do that by inserting
+an unattractive warning in the headers.  It looks something like this:
+
+  X-Authentication-Warning: u1.example.com: niceguy set sender to 
address@hidden using -f
+
+It is possible to configure sendmail to not do this, but such a reconfiguration
+is not an option for many users.  As this is the default behavior of most
+sendmail installations, one can mostly only wish it were otherwise.  If 
feedmail
+believes the sendmail program will sell you out this way, it won't use the 
\"-f\"
+option when calling sendmail.  If it doesn't think sendmail will sell you out,
+it will use the \"-f\" \(since it is a handy feature).  You control what
+feedmail thinks with this variable.  The default is nil, meaning that feedmail
+will believe that sendmail will sell you out."
+  :version "24.1"
+  :group 'feedmail-headers
+  :type 'boolean
 )
 
 
 (defcustom feedmail-deduce-envelope-from t
-  "*If non-nil, deduce message envelope \"from\" from header FROM: or SENDER:.
-In other words, if there is a SENDER: header in the message, temporarily
-change the value of user-mail-address to be the same while the message
-is being sent.  If there is no SENDER: header, use the FROM: header,
+  "If non-nil, deduce message envelope \"from\" from header From: or Sender:.
+In other words, if there is a Sender: header in the message, temporarily
+change the value of `user-mail-address' to be the same while the message
+is being sent.  If there is no Sender: header, use the From: header,
 if any.  Address values are taken from the actual message just before
 it is sent, and the process is independent of the values of
-feedmail-from-line and/or feedmail-sender-line.
+`feedmail-from-line' and/or `feedmail-sender-line'.
 
 There are many and good reasons for having the message header
-FROM:/SENDER: be different from the message envelope \"from\"
+From:/Sender: be different from the message envelope \"from\"
 information.  However, for most people and for most circumstances, it
 is usual for them to be the same (this is probably especially true for
 the case where the user doesn't understand the difference between the
@@ -547,38 +673,38 @@ feedmail-deduce-envelope-from
 
 The idea behind this feature is that you can have everything set up
 some normal way for yourself.  If for some reason you want to send a
-message with another FROM: line, you can just type it at the top of
+message with another From: line, you can just type it at the top of
 the message, and feedmail will take care of \"fixing up\" the envelope
 \"from\".  This only works for mail senders which make use of
-user-mail-address as the envelope \"from\" value.  For some mail
+`user-mail-address' as the envelope \"from\" value.  For some mail
 senders (e.g., feedmail-buffer-to-bin-mail), there is no simple way to
 influence what they will use as the envelope."
   :group 'feedmail-headers
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-x-mailer-line-user-appendage nil
-  "*See feedmail-x-mailer-line."
+  "See feedmail-x-mailer-line."
   :group 'feedmail-headers
-  :type '(choice (const nil) string)
-)
+  :type '(choice (const nil) (const t) string)
+  )
 
 
 (defcustom feedmail-x-mailer-line t
-  "*Control the form of an X-MAILER: header in an outgoing message.
+  "Control the form of an X-Mailer: header in an outgoing message.
 Moderately useful for debugging, keeping track of your correspondents'
 mailer preferences, or just wearing your MUA on your sleeve.  You
 should probably know that some people are fairly emotional about the
-presence of X-MAILER: lines in email.
+presence of X-Mailer: lines in email.
 
-If nil, nothing is done about X-MAILER:.
+If nil, nothing is done about X-Mailer:.
 
-If t, an X-MAILER: header of a predetermined format is produced,
-combining its efforts with any existing X-MAILER: header.  If you want
+If t, an X-Mailer: header of a predetermined format is produced,
+combining its efforts with any existing X-Mailer: header.  If you want
 to take the default construct and just add a little blob of your own
 at the end, define the variable feedmail-x-mailer-line-user-appendage
-as that blob string.  A value of t is equivalent to using the function 
+as that blob string.  A value of t is equivalent to using the function
 feedmail-default-x-mailer-generator.
 
 If neither nil nor t, it may be a string, a fiddle-plex, or a function
@@ -587,21 +713,21 @@ feedmail-x-mailer-line
 should be just the contents of the header, not the name of the header
 itself nor the trailing newline.  If a function, it will be called
 with no arguments.  For an explanation of fiddle-plexes, see the
-documentation for the variable feedmail-fiddle-plex-blurb.  In all
+documentation for the variable `feedmail-fiddle-plex-blurb'.  In all
 cases the name element of the fiddle-plex is ignored and is hardwired
 by feedmail to either \"X-Mailer\" or \"X-Resent-Mailer\"."
   :group 'feedmail-headers
   :type '(choice (const t) (const nil) string function)
-)
+  )
 
 
 (defcustom feedmail-message-id-generator t
-  "*Specifies the creation of a MESSAGE-ID: header field.
+  "Specifies the creation of a Message-Id: header field.
 
-If nil, nothing is done about MESSAGE-ID:.
+If nil, nothing is done about Message-Id:.
 
-If t, a MESSAGE-ID: header of a predetermined format is produced, but
-only if there is not already a MESSAGE-ID: in the message.  A value of
+If t, a Message-Id: header of a predetermined format is produced, but
+only if there is not already a Message-Id: in the message.  A value of
 t is equivalent to using the function feedmail-default-message-id-generator.
 
 If neither nil nor t, it may be a string, a fiddle-plex, or a function
@@ -611,42 +737,43 @@ feedmail-message-id-generator
 itself nor the trailing newline.  If a function, it will be called
 with one argument: the possibly-nil name of the file associated with
 the message buffer.  For an explanation of fiddle-plexes, see the
-documentation for the variable feedmail-fiddle-plex-blurb.  In all
+documentation for the variable `feedmail-fiddle-plex-blurb'.  In all
 cases the name element of the fiddle-plex is ignored and is hardwired
-by feedmail to either \"Message-ID\" or \"Resent-Message-ID\".
+by feedmail to either \"Message-Id\" or \"Resent-Message-Id\".
 
-You should let feedmail generate a MESSAGE-ID: for you unless you are sure
+You should let feedmail generate a Message-Id: for you unless you are sure
 that whatever you give your messages to will do it for you (e.g., most
 configurations of sendmail).  Even if the latter case is true, it
 probably won't hurt you to generate your own, and it will then show up
-in the saved message if you use FCC:."
+in the saved message if you use Fcc:."
   :group 'feedmail-headers
-  :type '(choice (const nil) function)
-)
+  :type '(choice (const t) (const nil) function)
+  )
 
 
 (defcustom feedmail-message-id-suffix nil
-  "*If non-nil, used as a suffix in generated MESSAGE-ID: headers for 
uniqueness.
-The function feedmail-default-message-id-generator creates its work based
+  "If non-nil, used as a suffix for generating unique Message-Id: headers.
+The function `feedmail-default-message-id-generator' creates its work based
 on a formatted date-time string, a random number, and a domain-looking suffix.
-You can control the suffix used by assigning a string value to this variable.  
If
-you don't supply one, the value of the variable user-mail-address will be used.
-If the value of feedmail-message-id-suffix contains an \"@\" character, the 
string
-will be used verbatim, else an \"@\" character will be prepended 
automatically."
+You can control the suffix used by assigning a string value to this variable.
+If you don't supply one, the value of the variable `user-mail-address' will be
+used.  If the value of `feedmail-message-id-suffix' contains an \"@\" 
character,
+the string will be used verbatim, else an \"@\" character will be prepended
+automatically."
   :group 'feedmail-headers
-  :type 'string
-)
+  :type '(choice (const nil) string)
+  )
 
 ;; this was suggested in various forms by several people; first was
 ;; Tony DeSimone in Oct 1992; sorry to be so tardy
 (defcustom feedmail-date-generator t
-  "*Specifies the creation of a DATE: header field.
+  "Specifies the creation of a Date: header field.
 
-If nil, nothing is done about DATE:.
+If nil, nothing is done about Date:.
 
-If t, a DATE: header of a predetermined format is produced, but only
-if there is not already a DATE: in the message.  A value of t is
-equivalent to using the function feedmail-default-date-generator.
+If t, a Date: header of a predetermined format is produced, but only
+if there is not already a Date: in the message.  A value of t is
+equivalent to using the function `feedmail-default-date-generator'.
 
 If neither nil nor t, it may be a string, a fiddle-plex, or a function
 which returns either nil, t, a string, or a fiddle-plex (or, in fact,
@@ -655,7 +782,7 @@ feedmail-date-generator
 itself nor the trailing newline.  If a function, it will be called
 with one argument: the possibly-nil name of the file associated with
 the message buffer.  For an explanation of fiddle-plexes, see the
-documentation for the variable feedmail-fiddle-plex-blurb.  In all
+documentation for the variable `feedmail-fiddle-plex-blurb'.  In all
 cases the name element of the fiddle-plex is ignored and is hardwired
 by feedmail to either \"Date\" or \"Resent-Date\".
 
@@ -663,23 +790,23 @@ feedmail-date-generator
 what you're doing.  Study the relevant parts of RFC-822 and RFC-1123.
 Don't make me come up there!
 
-You should let feedmail generate a DATE: for you unless you are sure
+You should let feedmail generate a Date: for you unless you are sure
 that whatever you give your messages to will do it for you (e.g., most
 configurations of sendmail).  Even if the latter case is true, it
 probably won't hurt you to generate your own, and it will then show up
-in the saved message if you use FCC:."
+in the saved message if you use Fcc:."
   :group 'feedmail-headers
-  :type '(choice (const nil) function)
-)
+  :type '(choice (const t) (const nil) function)
+  )
 
 
 (defcustom feedmail-fiddle-headers-upwardly t
-  "*Non-nil means fiddled header fields should be inserted at the top of the 
header.
-Nil means insert them at the bottom.  This is mostly a novelty issue since
+  "Non-nil means fiddled header fields should go at the top of the header.
+nil means insert them at the bottom.  This is mostly a novelty issue since
 the standards define the ordering of header fields to be immaterial and it's
-fairly likely that some MTA along the way will have its own idea of what the
+fairly likely that some MTA/MUA along the way will have its own idea of what 
the
 order should be, regardless of what you specify."
-  :group 'feedmail-header
+  :group 'feedmail-headers
   :type 'boolean
   )
 
@@ -691,14 +818,15 @@ feedmail-fiddle-plex-user-list
 
 feedmail will use this list of fiddle-plexes to manipulate user-specified
 message header fields.  It does this after it has completed all normal
-message header field manipulation and before calling feedmail-last-chance-hook.
+message header field manipulation and before calling 
`feedmail-last-chance-hook'.
 
 For an explanation of fiddle-plexes, see the documentation for the
-variable feedmail-fiddle-plex-blurb.  In contrast to some other fiddle-plex
+variable `feedmail-fiddle-plex-blurb'.  In contrast to some other fiddle-plex
 manipulation functions, in this context, it makes no sense to have an element
 which is nil, t, or a simple string."
-  :group 'feedmail-header
-  :type 'list
+  :group 'feedmail-headers
+  :type '(repeat (choice function)
+                sexp) ; too complex to be described accurately
   )
 
 
@@ -709,77 +837,105 @@ feedmail-enable-spray
 of addressees.  If this variable is non-nil, however, feedmail will
 repeatedly call the same buffer-eating function.  Each time, the list of
 addressees will be just one item from the original list.  This only affects
-the message envelope addresses and doesn't affect what appears in the 
+the message envelope addresses and doesn't affect what appears in the
 message headers except as noted.
 
 Spray mode is usually pointless, and if you can't think of a good reason for
-it, you should avoid it since it is inherently less efficient than normal 
+it, you should avoid it since it is inherently less efficient than normal
 multiple delivery.  One reason to use it is to overcome mis-featured mail
-transports which betray your trust by revealing BCC: addressees in the
+transports which betray your trust by revealing Bcc: addressees in the
 headers of a message.  Another use is to do a crude form of mailmerge, for
-which see feedmail-spray-address-fiddle-plex-list.
+which see `feedmail-spray-address-fiddle-plex-list'.
 
 If one of the calls to the buffer-eating function results in an error,
-what happens next is carelessly defined, so beware."
+what happens next is carelessly defined, so beware.  This should get ironed
+out in some future release, and there could be other API changes for spraying
+as well."
   :group 'feedmail-spray
   :type 'boolean
   )
 
 (defvar feedmail-spray-this-address nil
-  "Do not set or change this variable.  See 
feedmail-spray-address-fiddle-plex-list.")
+  "Do not set this variable, except via 
`feedmail-spray-address-fiddle-plex-list'.")
 
 (defcustom feedmail-spray-address-fiddle-plex-list nil
   "User-supplied specification for a crude form of mailmerge capability.
 When spraying is enabled, feedmail composes a list of envelope addresses.
-In turn, feedmail-spray-this-address is temporarily set to each address
-(stripped of any comments and angle brackets) and calls a function which 
-fiddles message headers according to this variable.  For an overview of
-fiddle-plex data structures, see the documentation for 
feedmail-fiddle-plex-blurb.
+In turn, `feedmail-spray-this-address' is temporarily set to each address
+\(stripped of any comments and angle brackets) and a function is called which
+fiddles message headers according to this variable.  See the documentation for
+`feedmail-fiddle-plex-blurb', for an overview of fiddle-plex data structures.
 
 May be nil, in which case nothing in particular is done about message
 headers for specific addresses.
 
-May be t, in which case a \"TO:\" header is added to the message with
+May be t, in which case a \"To:\" header is added to the message with
 the stripped address as the header contents.  The fiddle-plex operator
-is 'supplement.
+is `supplement'.
 
 May be a string, in which case the string is assumed to be the name of
 a message header field with the stripped address serving as the value.
-The fiddle-plex operator is 'supplement.
+The fiddle-plex operator is `supplement'.
 
 May be a function, in which case it is called with no arguments and is
 expected to return nil, t, a string, another function, or a fiddle-plex.
-The result is used recursively.
+The result is used recursively.  The function may alter the value of the
+variable feedmail-spray-this-address, perhaps to embellish it with a
+human name.  It would be logical in such a case to return as a value a
+string naming a message header like \"TO\" or an appropriately constructed
+fiddle-plex.  For an example, see feedmail-spray-via-bbdb.
 
-May be a list of any combination of the foregoing and fiddle-plexes.  (A
-value for this variable which consists of a single fiddle-plex must be
-nested inside another list to avoid ambiguity.)  If a list, each item
-is acted on in turn as described above.
+May be a list of any combination of the foregoing and/or
+fiddle-plexes.  (A value for this variable which consists of a single
+fiddle-plex must be nested inside another list to avoid ambiguity.)
+If a list, each item is acted on in turn as described above.
 
 For example,
 
-  (setq feedmail-spray-address-fiddle-plex-list 'my-address-embellisher)
+  (setq feedmail-spray-address-fiddle-plex-list \\='feedmail-spray-via-bbdb)
 
 The idea of the example is that, during spray mode, as each message is
 about to be transmitted to an individual address, the function will be
-called and will consult feedmail-spray-this-address to find the
+called and will consult `feedmail-spray-this-address' to find the
 stripped envelope email address (no comments or angle brackets).  The
 function should return an embellished form of the address.
 
 The recipe for sending form letters is:  (1) create a message with all
-addressees on BCC: headers; (2) tell feedmail to remove BCC: headers
-before sending the message; (3) create a function which will embellish 
-stripped addresses, if desired; (4) define 
feedmail-spray-address-fiddle-plex-list
-appropriately; (5) send the message with feedmail-enable-spray set
+addressees on Bcc: headers; (2) tell feedmail to remove Bcc: headers
+before sending the message; (3) create a function which will embellish
+stripped addresses, if desired; (4) define 
`feedmail-spray-address-fiddle-plex-list'
+appropriately; (5) send the message with `feedmail-enable-spray' set
 non-nil; (6) stand back and watch co-workers wonder at how efficient
 you are at accomplishing inherently inefficient things."
   :group 'feedmail-spray
-  :type 'list
+  :type 'sexp ; too complex to be described accurately
   )
 
+;; FIXME this is a macro?
+(declare-function bbdb-search "ext:bbdb-com"
+                 (records &optional name company net notes phone))
+(declare-function bbdb-records "ext:bbdb"
+                 (&optional dont-check-disk already-in-db-buffer))
+(declare-function bbdb-dwim-net-address "ext:bbdb-com" (record &optional net))
+
+(defun feedmail-spray-via-bbdb ()
+  "Example function for use with feedmail spray mode.
+NB: it's up to the user to have the BBDB environment already set up properly
+before using this."
+  (let (net-rec q-net-addy embellish)
+       (setq q-net-addy (concat "^" (regexp-quote feedmail-spray-this-address) 
"$"))
+       (setq net-rec (bbdb-search (bbdb-records) nil nil q-net-addy))
+       (if (and (car net-rec) (not (cdr net-rec)))
+               (setq net-rec (car net-rec))
+         (setq net-rec nil))
+       (if net-rec (setq embellish (bbdb-dwim-net-address net-rec)))
+       (if embellish
+               (list "To" embellish 'supplement)
+         (list "To" feedmail-spray-this-address 'supplement))))
+
 
 (defcustom feedmail-enable-queue nil
-  "*If non-nil, provide for stashing outgoing messages in a queue.
+  "If non-nil, provide for stashing outgoing messages in a queue.
 This is the master on/off switch for feedmail message queuing.
 Queuing is quite handy for laptop-based users.  It's also handy if you
 get a lot of mail and process it more or less sequentially.  For
@@ -799,52 +955,43 @@ feedmail-enable-queue
 directory you want at the prompt.  The right thing will happen.
 
 To transmit all the messages in the queue, invoke the command
-feedmail-run-the-queue or feedmail-run-the-queue-no-prompts."
+`feedmail-run-the-queue' or `feedmail-run-the-queue-no-prompts'."
   :group 'feedmail-queue
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-queue-runner-confirm-global nil
-  "*If non-nil, give a y-or-n confirmation prompt before running the queue.
+  "If non-nil, give a y-or-n confirmation prompt before running the queue.
 Prompt even if the queue is about to be processed as a result of a call to
-feedmail-run-the-queue-no-prompts.  This gives you a way to bail out
+`feedmail-run-the-queue-no-prompts'.  This gives you a way to bail out
 without having to answer no to the individual message prompts."
   :group 'feedmail-queue
   :type 'boolean)
 
 
-;; I provided a default for VMS because someone asked for it (the
-;; normal default doesn't work there), but, puh-lease!, it is a user
-;; definable option, so if you don't like the default, change it to
-;; whatever you want.  I am unable to directly test the VMS goop
-;; provided here by address@hidden (Richard Levitte - VMS Whacker).
 (defcustom feedmail-queue-directory
-  (if (memq system-type '(axp-vms vax-vms))
-      (expand-file-name (concat (getenv "HOME") "[.MAIL.Q]"))
-       (concat (getenv "HOME") "/mail/q"))
-  "*Name of a directory where messages will be queued.
+  (expand-file-name "~/mail/q")
+  "Name of a directory where messages will be queued.
 Directory will be created if necessary.  Should be a string that
-doesn't end with a slash.  Default, except on VMS, is \"$HOME/mail/q\"."
+doesn't end with a slash.  Default is \"~/mail/q\"."
   :group 'feedmail-queue
   :type 'string
-)
+  )
 
 
 (defcustom feedmail-queue-draft-directory
-  (if (memq system-type '(axp-vms vax-vms))
-      (expand-file-name (concat (getenv "HOME") "[.MAIL.DRAFT]"))
-       (concat (getenv "HOME") "/mail/draft"))
-  "*Name of an directory where DRAFT messages will be queued.
+  (expand-file-name "~/mail/draft")
+  "Name of a directory where draft messages will be queued.
 Directory will be created if necessary.  Should be a string that
-doesn't end with a slash.  Default, except on VMS, is \"$HOME/mail/draft\"."
+doesn't end with a slash.  Default is \"~/mail/draft\"."
   :group 'feedmail-queue
   :type 'string
-)
+  )
 
 
 (defcustom feedmail-ask-before-queue t
-  "*If non-nil, feedmail will ask what you want to do with the message.
+  "If non-nil, feedmail will ask what you want to do with the message.
 Default choices for the message action prompt will include sending it
 immediately, putting it in the main queue, putting it in the draft
 queue, or returning to the buffer to continue editing.  Only matters if
@@ -852,66 +999,69 @@ feedmail-ask-before-queue
 without a prompt."
   :group 'feedmail-queue
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-ask-before-queue-prompt "FQM: Message action (q, i, d, e, 
?)? [%s]: "
-  "*A string which will be used for the message action prompt.
+  "A string which will be used for the message action prompt.
 If it contains a \"%s\", that will be replaced with the value of
-feedmail-ask-before-queue-default."
+`feedmail-ask-before-queue-default'."
   :group 'feedmail-queue
   :type 'string
-)
+  )
 
 
 (defcustom feedmail-ask-before-queue-reprompt "FQM: Please type q, i, d, or e; 
or ? for help [%s]: "
-  "*A string which will be used for repompting after invalid input.
+  "A string which will be used for reprompting after invalid input.
 If it contains a \"%s\", that will be replaced with the value of
-feedmail-ask-before-queue-default."
+`feedmail-ask-before-queue-default'."
   :group 'feedmail-queue
   :type 'string
-)
+  )
 
 
 (defcustom feedmail-ask-before-queue-default "queue"
-  "*Meaning if user hits return in response to the message action prompt.
+  "Meaning if user hits return in response to the message action prompt.
 Should be a character or a string; if a string, only the first
 character is significant.  Useful values are those described in
 the help for the message action prompt."
   :group 'feedmail-queue
-  :type '(choice string integer)               ;use integer to get char
-)
+  :type '(choice string integer)       ;use integer to get char
+  )
 
 
 (defvar feedmail-prompt-before-queue-standard-alist
   '((?q . feedmail-message-action-queue)
-       (?Q . feedmail-message-action-queue-strong)
-
-       (?d . feedmail-message-action-draft)
-       (?r . feedmail-message-action-draft)
-       (?D . feedmail-message-action-draft-strong)
-       (?R . feedmail-message-action-draft-strong)
-
-       (?e . feedmail-message-action-edit)
-       (?E . feedmail-message-action-edit)
-       (?\C-g . feedmail-message-action-edit)
-       (?n . feedmail-message-action-edit)
-       (?N . feedmail-message-action-edit)
-
-       (?i . feedmail-message-action-send)
-       (?I . feedmail-message-action-send-strong)
-       (?s . feedmail-message-action-send)
-       (?S . feedmail-message-action-send-strong)
-
-       (?* . feedmail-message-action-toggle-spray)
-
-       (?\C-v . feedmail-message-action-help)
-       (?? . feedmail-message-action-help))
+    (?Q . feedmail-message-action-queue-strong)
+
+    (?d . feedmail-message-action-draft)
+    (?r . feedmail-message-action-draft)
+    (?D . feedmail-message-action-draft-strong)
+    (?R . feedmail-message-action-draft-strong)
+
+    (?e . feedmail-message-action-edit)
+    (?E . feedmail-message-action-edit)
+    (?\C-g . feedmail-message-action-edit)
+    (?n . feedmail-message-action-edit)
+    (?N . feedmail-message-action-edit)
+
+    (?i . feedmail-message-action-send)
+    (?I . feedmail-message-action-send-strong)
+    (?s . feedmail-message-action-send)
+    (?S . feedmail-message-action-send-strong)
+
+    (?* . feedmail-message-action-toggle-spray)
+
+    (?> . feedmail-message-action-scroll-up)
+    (?< . feedmail-message-action-scroll-down)
+    (?  . feedmail-message-action-scroll-up)
+    ;; (?\C-v . feedmail-message-action-help)
+    (?? . feedmail-message-action-help))
   "An alist of choices for the message action prompt.
 All of the values are function names, except help, which is a special
 symbol that calls up help for the prompt (the help describes the
 actions from the standard alist).  To customize your own choices,
-define a similar alist called feedmail-prompt-before-queue-user-alist.
+define a similar alist called `feedmail-prompt-before-queue-user-alist'.
 The actual alist used for message action will be the standard alist
 overlaid with the user-alist.  To neutralize an item in the standard
 alist without providing a replacement, define an appropriate element
@@ -919,9 +1069,9 @@ feedmail-prompt-before-queue-standard-alist
 
 
 (defcustom feedmail-prompt-before-queue-user-alist nil
-  "See feedmail-prompt-before-queue-standard-alist."
+  "See `feedmail-prompt-before-queue-standard-alist'."
   :group 'feedmail-queue
-  :type 'alist
+  :type '(repeat (cons character function))
   )
 
 
@@ -930,50 +1080,56 @@ feedmail-prompt-before-queue-help-supplement
 When the message action prompt is shown, the user can as for verbose help,
 at which point a buffer pops up describing the meaning of possible
 responses to the prompt.  Through various customizations (see, for
-example, feedmail-prompt-before-queue-user-alist), the available responses
+example, `feedmail-prompt-before-queue-user-alist'), the available responses
 and the prompt itself can be changed.  If this variable is set to a string
 value, that string is written to the help buffer after the standard info.
-It may contain embedded line breaks.  It will be printed via princ."
+It may contain embedded line breaks.  It will be printed via `princ'."
   :group 'feedmail-queue
-  :type 'string
+  :type '(choice (const nil) string)
   )
 
 
 (defcustom feedmail-queue-reminder-alist
   '((after-immediate . feedmail-queue-reminder-brief)
-       (after-queue . feedmail-queue-reminder-medium)
-       (after-draft . feedmail-queue-reminder-medium)
-       (after-run . feedmail-queue-reminder-brief)
-       (on-demand . feedmail-run-the-queue-global-prompt))
-  "See feedmail-queue-reminder."
+    (after-queue . feedmail-queue-reminder-medium)
+    (after-draft . feedmail-queue-reminder-medium)
+    (after-run . feedmail-queue-reminder-brief)
+    (on-demand . feedmail-run-the-queue-global-prompt))
+  "See `feedmail-queue-reminder'."
   :group 'feedmail-queue
-  :type 'alist
+  :type '(repeat (cons (choice :tag "Event"
+                              (const on-demand)
+                              (const after-immediate)
+                              (const after-queue)
+                              (const after-draft)
+                              (const after-run))
+                      function))
   )
 
 
 (defcustom feedmail-queue-chatty t
-  "*If non-nil, blat a few status messages and such in the mini-buffer.
+  "If non-nil, blat a few status messages and such in the mini-buffer.
 If nil, just do the work and don't pester people about what's going on.
 In some cases, though, specific options inspire mini-buffer prompting.
 That's not affected by this variable setting.  Also does not control
 reporting of error/abnormal conditions."
   :group 'feedmail-queue
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-queue-chatty-sit-for 2
-  "*Duration of pause after most queue-related messages.
+  "Duration of pause after most queue-related messages.
 After some messages are divulged, it is prudent to pause before
 something else obliterates them.  This value controls the duration of
 the pause."
   :group 'feedmail-queue
   :type 'integer
-)
+  )
 
 
 (defcustom feedmail-queue-run-orderer nil
-  "*If non-nil, name a function which will sort the queued messages.
+  "If non-nil, name a function which will sort the queued messages.
 The function is called during a running of the queue for sending, and
 takes one argument, a list of the files in the queue directory.  It
 may contain the names of non-message files, and it's okay to leave
@@ -983,35 +1139,41 @@ feedmail-queue-run-orderer
 they were placed in the queue."
   :group 'feedmail-queue
   :type '(choice (const nil) function)
-)
+  )
 
 
 (defcustom feedmail-queue-use-send-time-for-date nil
-  "*If non-nil, use send time for the DATE: header value.
+  "If non-nil, use send time for the Date: header value.
 This variable is used by the default date generating function,
 feedmail-default-date-generator.  If nil, the default, the
 last-modified timestamp of the queue file is used to create the
-message DATE: header; if there is no queue file, the current time is
-used."
+message Date: header; if there is no queue file, the current time is
+used. If you are using VM, it might be supplying this header for
+you.  To suppress VM's version
+
+       (setq vm-mail-header-insert-date nil)"
   :group 'feedmail-queue
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-queue-use-send-time-for-message-id nil
-  "*If non-nil, use send time for the MESSAGE-ID: header value.
-This variable is used by the default MESSAGE-ID: generating function,
-feedmail-default-message-id-generator.  If nil, the default, the
+  "If non-nil, use send time for the Message-Id: header value.
+This variable is used by the default Message-Id: generating function,
+`feedmail-default-message-id-generator'.  If nil, the default, the
 last-modified timestamp of the queue file is used to create the
-message MESSAGE-ID: header; if there is no queue file, the current time is
-used."
+message Message-Id: header; if there is no queue file, the current time is
+used.  If you are using VM, it might be supplying this header for
+you.  To suppress VM's version
+
+       (setq vm-mail-header-insert-date nil)"
   :group 'feedmail-queue
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-ask-for-queue-slug nil
-  "*If non-nil, prompt user for part of the queue file name.
+  "If non-nil, prompt user for part of the queue file name.
 The file will automatically get the FQM suffix and an embedded
 sequence number for uniqueness, so don't specify that.  feedmail will
 get rid of all characters other than alphanumeric and hyphen in the
@@ -1024,11 +1186,11 @@ feedmail-ask-for-queue-slug
 based on the subjects of the messages."
   :group 'feedmail-queue
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-queue-slug-maker 'feedmail-queue-subject-slug-maker
-  "*If non-nil, a function which creates part of the queued file name.
+  "If non-nil, a function which creates part of the queued file name.
 Takes a single argument giving the name of the directory into
 which the message will be queued.  The returned string should be just
 the non-directory filename part, without FQM suffix or uniquifying
@@ -1037,15 +1199,31 @@ feedmail-queue-slug-maker
 any."
   :group 'feedmail-queue
   :type '(choice (const nil) function)
+  )
+
+
+(defcustom feedmail-queue-slug-suspect-regexp "[^a-z0-9-]+"
+  "Regular expression for characters/substrings to be replaced.
+When feedmail creates a filename from a subject string, it puts hyphens
+in place of strings which may cause problems in filenames.  By default,
+only alphanumeric and hyphen characters are kept, and all others are
+converted.  In non-ASCII environments, it may be more helpful to
+tweak this regular expression to reflect local or personal language
+conventions.  Substitutions are done repeatedly until the regular expression
+no longer matches to transformed string.  Used by function
+feedmail-tidy-up-slug and indirectly by feedmail-queue-subject-slug-maker."
+  :version "24.1"
+  :group 'feedmail-queue
+  :type 'string
 )
 
 
 (defcustom feedmail-queue-default-file-slug t
-  "*Indicates what to use for subject-less messages when forming a file name.
-When feedmail queues a message, it creates a unique file name.  By default, 
+  "Indicates what to use for subject-less messages when forming a file name.
+When feedmail queues a message, it creates a unique file name.  By default,
 the file name is based in part on the subject of the message being queued.
 If there is no subject, consult this variable.  See documentation for the
-function feedmail-queue-subject-slug-maker.
+function `feedmail-queue-subject-slug-maker'.
 
 If t, an innocuous default is used.
 
@@ -1054,29 +1232,29 @@ feedmail-queue-default-file-slug
 If a function, it is called with no arguments from the buffer containing the 
raw
 text of the message.  It must return a string (which may be empty).
 
-If the symbol 'ask, you will be prompted for a string in the mini-buffer.
+If the symbol `ask', you will be prompted for a string in the mini-buffer.
 Filename completion is available so that you can inspect what's already been
 used, but feedmail will do further manipulation on the string you return, so
 it's not expected to be a complete filename."
   :group 'feedmail-queue
-  :type 'string
-)
+  :type '(choice (const :tag "Default" t) string function (const ask))
+  )
 
 
 (defcustom feedmail-queue-fqm-suffix ".fqm"
-  "*The FQM suffix used to distinguish feedmail queued message files.
+  "The FQM suffix used to distinguish feedmail queued message files.
 You probably want this to be a period followed by some letters and/or
 digits.  The distinction is to be able to tell them from other random
-files that happen to be in the feedmail-queue-directory or
-feedmail-queue-draft-directory. By the way, FQM stands for feedmail
+files that happen to be in the `feedmail-queue-directory' or
+`feedmail-queue-draft-directory'.  By the way, FQM stands for feedmail
 queued message."
   :group 'feedmail-queue
   :type 'string
-)
+  )
 
 
 (defcustom feedmail-nuke-buffer-after-queue nil
-  "*If non-nil, silently kill the buffer after a message is queued.
+  "If non-nil, silently kill the buffer after a message is queued.
 You might like that since a side-effect of queueing the message is
 that its buffer name gets changed to the filename.  That means that
 the buffer won't be reused for the next message you compose.  If you
@@ -1085,185 +1263,258 @@ feedmail-nuke-buffer-after-queue
 message buffers."
   :group 'feedmail-queue
   :type 'boolean
-)
+  )
 
 
 (defcustom feedmail-queue-auto-file-nuke nil
-  "*If non-nil, automatically delete queue files when a message is sent.
+  "If non-nil, automatically delete queue files when a message is sent.
 Normally, feedmail will notice such files when you send a message in
 immediate mode (i.e., not when you're running the queue) and will ask if
 you want to delete them.  Since the answer is usually yes, setting this
-variable to non-nil will tell feedmail to skip the prompt and just delete 
+variable to non-nil will tell feedmail to skip the prompt and just delete
 the file without bothering you."
   :group 'feedmail-queue
   :type 'boolean
+  )
+
+
+(defcustom feedmail-debug nil
+  "If non-nil, blat a debug messages and such in the mini-buffer.
+This is intended as an aid to tracing what's going on but is probably
+of casual real use only to the feedmail developer."
+  :version "24.1"
+  :group 'feedmail-debug
+  :type 'boolean
+)
+
+
+(defcustom feedmail-debug-sit-for 0
+  "Duration of pause after feedmail-debug messages.
+After some messages are divulged, it may be helpful to pause before
+something else obliterates them.  This value controls the duration of
+the pause.  If the value is nil or 0, the sit-for is not done, which
+has the effect of not pausing at all.  Debug messages can be seen after
+the fact in the messages buffer."
+  :version "24.1"
+  :group 'feedmail-debug
+  :type 'integer
 )
 
 
+(defvar feedmail-queue-buffer-file-name nil
+  "If non-nil, has the value normally expected of `buffer-file-name'.
+You are not intended to set this to something in your configuration.  Rather,
+you might programmatically set it to something via a hook or function
+advice or whatever.  You might like to do this if you are using a mail
+composition program that eventually uses sendmail.el's `mail-send'
+function to process the message.  If there is a filename associated
+with the message buffer, `mail-send' will ask you for confirmation.
+There's no trivial way to avoid it.  It's unwise to just set the value
+of `buffer-file-name' to nil because that will defeat feedmail's file
+management features.  Instead, arrange for this variable to be set to
+the value of `buffer-file-name' before setting that to nil.  An easy way
+to do that would be with an advice on `mail-send'.
+
+feedmail will pretend that `buffer-file-name', if nil, has the value
+assigned of `feedmail-queue-buffer-file-name' and carry out its normal
+activities.  feedmail does not restore the non-nil value of
+`buffer-file-name'.  For safe bookkeeping, the user should ensure that
+feedmail-queue-buffer-file-name is restored to nil.
+
+Example advice for mail-send:
+
+    (advice-add 'mail-send :around #'my-feedmail-mail-send-advice)
+    (defun my-feedmail-mail-send-advice (orig-fun &rest args)
+      (let ((feedmail-queue-buffer-file-name buffer-file-name)
+             (buffer-file-name nil))
+        (apply orig-fun args)))")
+
 ;; defvars to make byte-compiler happy(er)
-(defvar feedmail-error-buffer        nil "not a user option variable")
-(defvar feedmail-prepped-text-buffer nil "not a user option variable")
-(defvar feedmail-raw-text-buffer     nil "not a user option variable")
-(defvar feedmail-address-list        nil "not a user option variable")
+(defvar feedmail-error-buffer        nil)
+(defvar feedmail-prepped-text-buffer nil)
+(defvar feedmail-raw-text-buffer     nil)
+(defvar feedmail-address-list        nil)
+
+
+(defvar feedmail-queue-runner-is-active nil
+  "Non-nil means we're inside the logic of the queue-running loop.
+That is, iterating over all messages in the queue to send them.  In
+that case, the value is the name of the queued message file currently
+being processed.  This can be used for differentiating customized code
+for different scenarios.  Users shouldn't set or change this
+variable, but may depend on its value as described here.")
 
 
 (defun feedmail-mail-send-hook-splitter ()
-  "Facilitate dividing mail-send-hook things into queued and immediate cases.
-If you have mail-send-hook functions that should only be called for sending/
+  "Facilitate dividing `mail-send-hook' things into queued and immediate cases.
+If you have `mail-send-hook' functions that should only be called for sending/
 queueing messages or only be called for the sending of queued messages, this is
-for you.  Add this function to mail-send-hook with something like this:
+for you.  Add this function to `mail-send-hook' with something like this:
 
-       (add-hook 'mail-send-hook 'feedmail-mail-send-hook-splitter)
+       (add-hook \\='mail-send-hook \\='feedmail-mail-send-hook-splitter)
 
-Then add the functions you want called to either feedmail-mail-send-hook-queued
-or feedmail-mail-send-hook, as apprpriate.  The distinction is that
-feedmail-mail-send-hook will be called when you send mail from a composition
+Then add the functions you want called to either 
`feedmail-mail-send-hook-queued'
+or `feedmail-mail-send-hook', as appropriate.  The distinction is that
+`feedmail-mail-send-hook' will be called when you send mail from a composition
 buffer (typically by typing C-c C-c), whether the message is sent immediately
-or placed in the queue or drafts directory.  feedmail-mail-send-hook-queued is
+or placed in the queue or drafts directory.  `feedmail-mail-send-hook-queued' 
is
 called when messages are being sent from the queue directory, typically via a
-call to feedmail-run-the-queue."
+call to `feedmail-run-the-queue'."
+  (feedmail-say-debug ">in-> feedmail-mail-send-hook-splitter %s" 
feedmail-queue-runner-is-active)
   (if feedmail-queue-runner-is-active
-         (run-hooks 'feedmail-mail-send-hook-queued)
-       (run-hooks 'feedmail-mail-send-hook))
-)
-
-
-(defvar feedmail-mail-send-hook nil
-  "*See documentation for feedmail-mail-send-hook-splitter.")
+      (run-hooks 'feedmail-mail-send-hook-queued)
+    (run-hooks 'feedmail-mail-send-hook)))
 
+(defcustom feedmail-mail-send-hook nil
+  "Hook run by `feedmail-mail-send-hook-splitter' for immediate mail.
+See documentation of `feedmail-mail-send-hook-splitter' for details."
+  :type 'hook
+  :group 'feedmail)
 
-(defvar feedmail-mail-send-hook-queued nil
-  "*See documentation for feedmail-mail-send-hook-splitter.")
-
+(defcustom feedmail-mail-send-hook-queued nil
+  "Hook run by `feedmail-mail-send-hook-splitter' for queued mail.
+See documentation of `feedmail-mail-send-hook-splitter' for details."
+  :type 'hook
+  :group 'feedmail)
 
 (defun feedmail-confirm-addresses-hook-example ()
-  "An example of a feedmail-last-chance-hook.
+  "An example of a `feedmail-last-chance-hook'.
 It shows the simple addresses and gets a confirmation.  Use as:
- (setq feedmail-last-chance-hook 'feedmail-confirm-addresses-hook-example)."
+ (setq feedmail-last-chance-hook \\='feedmail-confirm-addresses-hook-example)."
   (save-window-excursion
-       (display-buffer (set-buffer (get-buffer-create " F-C-A-H-E")))
-       (erase-buffer)
-       (insert (mapconcat 'identity feedmail-address-list " "))
-       (if (not (y-or-n-p "How do you like them apples? "))
-               (error "FQM: Sending...gave up in last chance hook")
-         )))
-
+    (display-buffer (set-buffer (get-buffer-create " F-C-A-H-E")))
+    (erase-buffer)
+    (insert (mapconcat 'identity feedmail-address-list " "))
+    (if (not (y-or-n-p "How do you like them apples? "))
+       (error "FQM: Sending...gave up in last chance hook"))))
 
 (defcustom feedmail-last-chance-hook nil
-  "*User's last opportunity to modify the message on its way out.
-It has already had all the header prepping from the standard package.
-The next step after running the hook will be to push the buffer into a
-subprocess that mails the mail.  The hook might be interested in
-these: (1) feedmail-prepped-text-buffer contains the header and body
-of the message, ready to go; (2) feedmail-address-list contains a list
+  "User's last opportunity to modify the message on its way out.
+When this hook runs, the current buffer is already the appropriate
+buffer.  It has already had all the header prepping from the standard
+package.  The next step after running the hook will be to save the
+message via Fcc: processing. The hook might be interested in these:
+\(1) `feedmail-prepped-text-buffer' contains the header and body of the
+message, ready to go; (2) `feedmail-address-list' contains a list
 of simplified recipients of addresses which are to be given to the
-subprocess (the hook may change the list); (3) feedmail-error-buffer
+subprocess (the hook may change the list); (3) `feedmail-error-buffer'
 is an empty buffer intended to soak up errors for display to the user.
 If the hook allows interactive activity, the user should not send more
 mail while in the hook since some of the internal buffers will be
-reused and things will get confused."
+reused and things will get confused.  It's not necessary to
+arrange for the undoing of any changes you make to the buffer."
   :group 'feedmail-misc
   :type 'hook
-)
+  )
 
+(defcustom feedmail-queue-express-hook nil
+  "Chance to modify a message being sent directly to a queue.
+Run by feedmail-queue-express-to-queue and feedmail-queue-express-to-draft.
+For example, you might want to run vm-mime-encode-composition to take
+care of attachments.  If you subsequently edit the message buffer, you
+can undo the encoding."
+  :version "24.1"
+  :group 'feedmail-queue
+  :type 'hook
+)
 
 (defcustom feedmail-before-fcc-hook nil
-  "*User's last opportunity to modify the message before FCC action.
+  "User's last opportunity to modify the message before Fcc action.
 It has already had all the header prepping from the standard package.
 The next step after running the hook will be to save the message via
-FCC: processing. The hook might be interested in these: (1)
-feedmail-prepped-text-buffer contains the header and body of the
-message, ready to go; (2) feedmail-address-list contains a list of
+Fcc: processing. The hook might be interested in these: (1)
+`feedmail-prepped-text-buffer' contains the header and body of the
+message, ready to go; (2) `feedmail-address-list' contains a list of
 simplified recipients of addressees to whom the message was sent (3)
-feedmail-error-buffer is an empty buffer intended to soak up errors
+`feedmail-error-buffer' is an empty buffer intended to soak up errors
 for display to the user.  If the hook allows interactive activity, the
 user should not send more mail while in the hook since some of the
 internal buffers will be reused and things will get confused."
   :group 'feedmail-misc
   :type 'hook
-)
+  )
 
 (defcustom feedmail-queue-runner-mode-setter
-  '(lambda (&optional arg) (mail-mode))
-  "*A function to set the proper mode of a message file.  Called when
-the message is read back out of the queue directory with a single
+  (lambda (&optional _) (mail-mode))
+  "A function to set the proper mode of a message file.
+Called when the message is read back out of the queue directory with a single
 argument, the optional argument used in the call to
-feedmail-run-the-queue or feedmail-run-the-queue-no-prompts.
+`feedmail-run-the-queue' or `feedmail-run-the-queue-no-prompts'.
 
-Most people want mail-mode, so the default value is an anonymous
+Most people want `mail-mode', so the default value is an anonymous
 function which is just a wrapper to ignore the supplied argument when
 calling it, but here's your chance to have something different.
-Called with funcall, not call-interactively."
+If you are a VM user, you might like feedmail-vm-mail-mode, though you
+really don't need that (and it's not particularly well-tested).
+
+Called with funcall, not `call-interactively'."
   :group 'feedmail-queue
   :type 'function
-)
+  )
 
 
 (defcustom feedmail-queue-alternative-mail-header-separator nil
-  "*Alternative header demarcation for queued messages.
-If you sometimes get alternative values for mail-header-separator in
+  "Alternative header demarcation for queued messages.
+If you sometimes get alternative values for `mail-header-separator' in
 queued messages, set the value of this variable to whatever it is.
-For example, rmail-resend uses a mail-header-separator value of empty
+For example, `rmail-resend' uses a `mail-header-separator' value of empty
 string (\"\") when you send/queue a message.
 
 When trying to send a queued message, if the value of this variable is
 non-nil, feedmail will first try to send the message using the value
-of mail-header-separator.  If it can't find that, it will temporarily
-set mail-header-separator to the value of
-feedmail-queue-alternative-mail-header-separator and try again."
+of `mail-header-separator'.  If it can't find that, it will temporarily
+set `mail-header-separator' to the value of
+`feedmail-queue-alternative-mail-header-separator' and try again."
   :group 'feedmail-queue
-  :type 'string
-)
+  :type '(choice (const nil) string)
+  )
 
 
-(defcustom feedmail-queue-runner-message-sender 'mail-send-and-exit
-  "*Function to initiate sending a message file.
+(defcustom feedmail-queue-runner-message-sender
+  (lambda (&optional _)
+    ;; `mail-send' is not autoloaded, which is why we need the `require'.
+    (require 'sendmail) (declare-function mail-send "sendmail")
+    (mail-send))
+  "Function to initiate sending a message file.
 Called for each message read back out of the queue directory with a
 single argument, the optional argument used in the call to
-feedmail-run-the-queue or feedmail-run-the-queue-no-prompts.
-Interactively, that argument will be the prefix argument.  Most people
-want mail-send-and-exit (bound to C-c C-c in mail-mode), but here's
-your chance to have something different.  Called with funcall, not
-call-interactively."
+`feedmail-run-the-queue' or `feedmail-run-the-queue-no-prompts'.
+Interactively, that argument will be the prefix argument.
+Most people want `mail-send' (bound to C-c C-s in mail-mode), but here's
+your chance to have something different.  The default value is just a
+wrapper function which discards the optional argument and calls
+mail-send.  If you are a VM user, you might like vm-mail-send, though
+you really don't need that.  Called with funcall, not call-interactively."
+  :version "24.1"                      ; changed default
   :group 'feedmail-queue
   :type 'function
-)
+  )
 
 
 (defcustom feedmail-queue-runner-cleaner-upper
-  '(lambda (fqm-file &optional arg)
-        (delete-file fqm-file)
-        (if (and arg feedmail-queue-chatty) (message "FQM: Nuked %s" 
fqm-file)))
-  "*Function that will be called after a message has been sent.  It's
-not called in the case of errors.  This function is called with two
-arguments, the name of the message queue file for the message just
-sent, and the optional argument used in the call to
-feedmail-run-the-queue or feedmail-run-the-queue-no-prompts.
-Interactively, that argument will be the prefix argument.  In any
-case, the affiliated buffer is killed elsewhere, so don't do that
+  (lambda (fqm-file &optional arg)
+     (delete-file fqm-file)
+     (if arg (feedmail-say-chatter "Nuked %s" fqm-file)))
+  "Function that will be called after a message has been sent.
+Not called in the case of errors.  This function is called with two
+arguments: the name of the message queue file for the message just sent,
+and the optional argument used in the call to `feedmail-run-the-queue'
+or `feedmail-run-the-queue-no-prompts' (prefix arg if interactive).
+In any case, the affiliated buffer is killed elsewhere, so don't do that
 inside this function.  Return value is ignored.
 
 The default action is an anonymous function which gets rid of the file
 from the queue directory.  With a non-nil second argument, a brief
 message is give for each file deleted.  You could replace this
 function, for example, to archive all of your sent messages someplace
-(though there are better ways to get that particular result)."
+\(though there are better ways to get that particular result)."
   :group 'feedmail-queue
   :type 'function
-)
-
-
-(defvar feedmail-queue-runner-is-active nil
-  "*Non-nil means we're inside the logic of the queue-running loop.
-That is, iterating over all messages in the queue to send them.  In
-that case, the value is the name of the queued message file currently
-being processed.  This can be used for differentiating customized code
-for different scenarios.  Users shouldn't set or change this
-variable, but may depend on its value as described here.")
+  )
 
 
 (defvar feedmail-is-a-resend nil
-  "*Non-nil means the the message is a RESEND (in the RFC-822 sense).
+  "Non-nil means the message is a Resend (in the RFC-822 sense).
 This affects the composition of certain headers.  feedmail sets this
 variable as soon as it starts prepping the message text buffer, so any
 user-supplied functions can rely on it.  Users shouldn't set or change this
@@ -1271,66 +1522,108 @@ feedmail-is-a-resend
 
 
 (defcustom feedmail-buffer-eating-function 'feedmail-buffer-to-binmail
-  "*Function used to send the prepped buffer to a subprocess.
+  "Function used to send the prepped buffer to a subprocess.
 The function's three (mandatory) arguments are: (1) the buffer
 containing the prepped message; (2) a buffer where errors should be
 directed; and (3) a list containing the addresses individually as
-strings.  Three popular choices for this are
-feedmail-buffer-to-binmail, feedmail-buffer-to-smtpmail, and
-feedmail-buffer-to-sendmail.  If you use the sendmail form, you
-probably want to set feedmail-nuke-bcc and/or feedmail-nuke-resent-bcc
-to nil.  If you use the binmail form, check the value of
-feedmail-binmail-template."
+strings.  Popular choices for this are `feedmail-buffer-to-binmail',
+`feedmail-buffer-to-smtpmail', `feedmail-buffer-to-sendmail', and
+`feedmail-buffer-to-smtp'.  If you use the sendmail form, you probably
+want to set `feedmail-nuke-bcc' and/or `feedmail-nuke-resent-bcc to nil'.
+If you use the binmail form, check the value of `feedmail-binmail-template'."
   :group 'feedmail-misc
   :type 'function
-)
+  )
 
+(defconst feedmail-binmail-gnulinuxish-template
+  (concat
+   "(echo From "
+   (if (boundp 'user-login-name) user-login-name "feedmail")
+   " ; cat -) | /usr/bin/rmail %s")
+  "Good candidate for GNU/Linux systems and maybe others.
+You may need to modify this if your \"rmail\" is in a different place.
+For example, I hear that in some Debian systems, it's /usr/sbin/rmail.
+See feedmail-binmail-template documentation."
+  )
 
-(defcustom feedmail-binmail-template (if mail-interactive "/bin/mail %s" 
"/bin/rmail %s")
-  "*Command template for the subprocess which will get rid of the mail.
+(defcustom feedmail-binmail-template (if mail-interactive "/bin/mail %s"
+                                      (if (file-exists-p "/bin/rmail")
+                                          "/bin/rmail %s" "/bin/mail %s"))
+  "Command template for the subprocess which will get rid of the mail.
 It can result in any command understandable by /bin/sh.  Might not
-work at all in non-UNIX environments.  The single '%s', if present,
+work at all in non-UNIX environments.  The single `%s', if present,
 gets replaced by the space-separated, simplified list of addressees.
-Used in feedmail-buffer-to-binmail to form the shell command which
-will receive the contents of the prepped buffer as stdin.  If you'd
-like your errors to come back as mail instead of immediately in a
-buffer, try /bin/rmail instead of /bin/mail (this can be accomplished
-by keeping the default nil setting of mail-interactive).  You might
-also like to consult local mail experts for any other interesting
-command line possibilities."
+Used in `feedmail-buffer-to-binmail' to form the shell command which
+will receive the contents of the prepped buffer as stdin.  The default
+value uses /bin/rmail (if it exists) unless `mail-interactive' has been
+set non-nil.
+
+If you'd like your errors to come back as mail instead of immediately
+in a buffer, try /bin/rmail instead of /bin/mail.  If /bin/rmail
+exists, this can be accomplished by keeping the default nil setting of
+`mail-interactive'.  You might also like to consult local mail experts
+for any other interesting command line possibilities.  Some versions
+of UNIX have an rmail program which behaves differently than
+/bin/rmail and complains if feedmail gives it a message on stdin.  If
+you don't know about such things and if there is no local expert to
+consult, stick with /bin/mail or use one of the other buffer eating
+functions.
+
+The above description applies to \"classic\" UNIX /bin/mail and /bin/rmail.
+On most GNU/Linux systems and perhaps other places, /bin/mail behaves
+completely differently and shouldn't be used at all in this template.
+Instead of /bin/rmail, there is a /usr/bin/rmail, and it can be used
+with a wrapper.  The wrapper is necessary because /usr/bin/rmail on such
+systems requires that the first line of the message appearing on standard
+input have a UNIX-style From_ postmark.  If you have such a system, the
+wrapping can be accomplished by setting the value of 
`feedmail-binmail-template'
+to `feedmail-binmail-gnulinuxish-template'.  You should then send some test
+messages to make sure it works as expected."
+  :version "24.1"                      ; changed default
   :group 'feedmail-misc
   :type 'string
-)
+  )
 
 
 ;; feedmail-buffer-to-binmail, feedmail-buffer-to-sendmail, and
-;; feedmail-buffer-to-smptmail are the only things provided for values
+;; feedmail-buffer-to-smtpmail are the only things provided for values
 ;; for the variable feedmail-buffer-eating-function.  It's pretty easy
 ;; to write your own, though.
 (defun feedmail-buffer-to-binmail (prepped errors-to addr-listoid)
   "Function which actually calls /bin/mail as a subprocess.
 Feeds the buffer to it."
+  (feedmail-say-debug ">in-> feedmail-buffer-to-binmail %s" addr-listoid)
   (set-buffer prepped)
   (apply
    'call-process-region
    (append (list (point-min) (point-max) "/bin/sh" nil errors-to nil "-c"
-                                (format feedmail-binmail-template
-                                                (mapconcat 'identity 
addr-listoid " "))))))
+                (format feedmail-binmail-template
+                        (mapconcat 'identity addr-listoid " "))))))
 
 
+(defvar sendmail-program)
+
 (defun feedmail-buffer-to-sendmail (prepped errors-to addr-listoid)
   "Function which actually calls sendmail as a subprocess.
-Feeds the buffer to it.  Probably has some flaws for RESENT-* and other
-complicated cases."
+Feeds the buffer to it.  Probably has some flaws for Resent-* and other
+complicated cases.  Takes addresses from message headers and
+might disappoint you with Bcc: handling.  In case of odd results, consult
+local gurus."
+  (require 'sendmail)
+  (feedmail-say-debug ">in-> feedmail-buffer-to-sendmail %s" addr-listoid)
   (set-buffer prepped)
   (apply 'call-process-region
-                (append (list (point-min) (point-max)
-                                          (if (boundp 'sendmail-program) 
sendmail-program "/usr/lib/sendmail")
-                                          nil errors-to nil "-oi" "-t")
-                                ;; provide envelope "from" to sendmail; 
results will vary
-                                (list "-f" user-mail-address)
-                                ;; These mean "report errors by mail" and 
"deliver in background".
-                                (if (null mail-interactive) '("-oem" 
"-odb")))))
+        (append (list (point-min) (point-max) sendmail-program
+                      nil errors-to nil "-oi" "-t")
+                ;; provide envelope "from" to sendmail; results will vary
+                (if feedmail-sendmail-f-doesnt-sell-me-out
+                    (list "-f" user-mail-address))
+                ;; These mean "report errors by mail" and "deliver in 
background".
+                (if (null mail-interactive) '("-oem" "-odb")))))
+
+(declare-function smtpmail-via-smtp "smtpmail"
+                 (recipient smtpmail-text-buffer &optional ask-for-password))
+(defvar smtpmail-smtp-server)
 
 ;; provided by address@hidden (James A. McLaughlin);
 ;; simplified by WJC after more feedmail development;
@@ -1339,52 +1632,76 @@ feedmail-buffer-to-sendmail
 ;;   Mon 14-Oct-1996; Douglas Gray Stephens
 ;;   modified to insert error for displaying
 (defun feedmail-buffer-to-smtpmail (prepped errors-to addr-listoid)
-  "Function which actually calls smtpmail-via-smtp to send buffer as e-mail."
+  "Function which actually calls `smtpmail-via-smtp' to send buffer as e-mail."
   ;; I'm not sure smtpmail.el is careful about the following
   ;; return value, but it also uses it internally, so I will fear
   ;; no evil.
+  (feedmail-say-debug ">in-> feedmail-buffer-to-smtpmail %s" addr-listoid)
   (require 'smtpmail)
-  (if (not (smtpmail-via-smtp addr-listoid prepped))
+  (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)
+
+;; FLIM's smtp.el pointed out to me by Kenichi Handa <address@hidden>
+(defun feedmail-buffer-to-smtp (prepped errors-to addr-listoid)
+  "Function which actually calls smtp-via-smtp to send buffer as e-mail."
+  (feedmail-say-debug ">in-> feedmail-buffer-to-smtp %s" addr-listoid)
+  (require 'smtp)
+  (if (not (smtp-via-smtp user-mail-address addr-listoid prepped))
          (progn
                (set-buffer errors-to)
-               (insert "Send via smtpmail failed.  Probable SMTP protocol 
error.\n")
+               (insert "Send via smtp 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))))
+                         (tracer (concat "trace.*smtp.*" (regexp-quote 
smtp-server))))
                  (mapcar
-                  '(lambda (buffy)
+                  (lambda (buffy)
                          (if (string-match tracer (buffer-name buffy))
                                  (progn
                                        (insert "SMTP Trace from " (buffer-name 
buffy) "\n---------------")
-                                       (insert-buffer buffy)
+                                       (insert-buffer-substring buffy)
                                        (insert "\n\n"))))
                   (buffer-list))))))
 
-
 ;; just a place to park a docstring
 (defconst feedmail-fiddle-plex-blurb nil
-  "A fiddle-plex is a concise way of specifying how to fiddle with a header 
field.
+  "A fiddle-plex is a concise way of specifying header field fiddling.
 It is a list of up to 4 elements: NAME, VALUE, ACTION, FOLDING.  The element
 VALUE can also be a list sometimes.
 
-NAME is the name of the header field to be fiddled with.  Although case doesn't
-matter in looking for headers, case of NAME is preserved when a header is 
inserted
-via fiddling.  It shouldn't include the trailing colon.
+NAME is the name of the header field to be fiddled with.  Although case
+doesn't matter in looking for headers, case of NAME is preserved when a header
+is inserted via fiddling.  It shouldn't include the trailing colon.
 
 VALUE is either nil, a simple string, a function returning nil or a string, or,
-as described below for ACTION 'combine, a list of up to three values.
+as described below for ACTION `combine', a list of up to three values.
 
 ACTION describes the nature of the fiddling to be done.  Possibilities
-for ACTION (default is 'supplement):
+for ACTION (default is `supplement'):
 
-  'supplement   Leave other like fields as-is, insert this one.
+  `supplement'  Leave other like fields as-is, insert this one.
 
-  'replace      Delete other like fields, if any, and insert this one.
+  `replace'     Delete other like fields, if any, and insert this one.
 
-  'create       Insert this one only if no like field exists.
+  `create'      Insert this one only if no like field exists.
 
-  'combine      Combine aggregate values of like fields with this one.
+  `combine'     Combine aggregate values of like fields with this one.
                 In this case, VALUE has a special form.  It is a list
                 of three items: VAL-PRE, VAL-LIKE, and VAL-POST.
                 VAL-PRE and VAL-POST are strings or nil.  VAL-LIKE may
@@ -1397,8 +1714,8 @@ feedmail-fiddle-plex-blurb
                 fields, or the aggregate value is an empty string,
                 VAL-LIKE is not used.  Else, if VAL-LIKE is a function,
                 it is called with two arguments: NAME and the
-                aggregate like values.  Else, if VAL-LIKE is a string, it is 
-                used as a format string where a single \%s will be
+                aggregate like values.  Else, if VAL-LIKE is a string, it is
+                used as a format string where a single %s will be
                 replaced by the aggregate values of like fields.
 
                 VAL-PRE, the results of using VAL-LIKE, and VAL-POST
@@ -1416,116 +1733,226 @@ feedmail-fiddle-plex-blurb
 
 FOLDING can be nil, in which case VALUE is used as-is.  If FOLDING is
 non-nil, feedmail \"smart filling\" is done on VALUE just before
-insertion.
-")
-
+insertion.")
+
+(declare-function vm-mail "ext:vm" (&optional to subject))
+
+(defun feedmail-vm-mail-mode (&optional _)
+  "Make something like a buffer that has been created via `vm-mail'.
+The optional argument is ignored and is just for argument compatibility with
+`feedmail-queue-runner-mode-setter'.  This function is suitable for being
+applied to a file after you've just read it from disk: for example, a
+feedmail FQM message file from a queue.  You could use something like
+this:
+
+    (add-to-list 'auto-mode-alist \\='(\"\\\\.fqm\\\\\\='\" . 
feedmail-vm-mail-mode))"
+  (feedmail-say-debug ">in-> feedmail-vm-mail-mode")
+  (let ((the-buf (current-buffer)))
+    (vm-mail)
+    (delete-region (point-min) (point-max))
+    (insert-buffer-substring the-buf)
+    (setq buffer-file-name (buffer-file-name the-buf))
+    (set-buffer-modified-p (buffer-modified-p the-buf))
+    ;; For some versions of emacs, saving the message to a queue
+    ;; triggers running the mode function on the buffer, and that
+    ;; leads (through a series of events I don't really understand)
+    ;; to this function being called while the buffer is still
+    ;; marked modified even though it is in the process of being
+    ;; saved.  I guess the function gets called during the renaming
+    ;; that takes place en route to the save.
+    ;;
+    ;; This clearing of the marker probably wastes a buffer copy
+    ;; but it's easy to do and more reliable than figuring out what
+    ;; each variant of emacs does in this strange case.
+    (with-current-buffer the-buf
+      (set-buffer-modified-p nil))
+    (kill-buffer the-buf)
+    ))
 
+;;;###autoload
 (defun feedmail-send-it ()
-  "A function which is a suitable value for send-mail-function.
-To use it, you probably want something like this in your .emacs or
-similar place:
-
-  (setq send-mail-function 'feedmail-send-it)
-  (autoload 'feedmail-send-it \"feedmail\")"
-
+  "Send the current mail buffer using the Feedmail package.
+This is a suitable value for `send-mail-function'.  It can be used
+with various lower-level mechanisms to provide features such as queueing."
+  (feedmail-say-debug ">in-> feedmail-send-it")
+  (save-excursion
+       (let ((bfn-jiggle nil))
+         ;; if buffer-file-name is nil, temporarily use the stashed value
+         (if (and (not buffer-file-name) feedmail-queue-buffer-file-name)
+                 (setq buffer-file-name feedmail-queue-buffer-file-name
+                               bfn-jiggle t))
   ;; avoid matching trouble over slash vs backslash by getting canonical
   (if feedmail-queue-directory
-         (setq feedmail-queue-directory (expand-file-name 
feedmail-queue-directory)))
+      (setq feedmail-queue-directory (expand-file-name 
feedmail-queue-directory)))
   (if feedmail-queue-draft-directory
-         (setq feedmail-queue-draft-directory (expand-file-name 
feedmail-queue-draft-directory)))
-  (if (not feedmail-enable-queue) (feedmail-send-it-immediately)
-       ;; else, queuing is enabled, should we ask about it or just do it?
-       (if feedmail-ask-before-queue
-               (funcall (feedmail-queue-send-edit-prompt))
-         (feedmail-dump-message-to-queue feedmail-queue-directory 
'after-queue))))
-
+      (setq feedmail-queue-draft-directory (expand-file-name 
feedmail-queue-draft-directory)))
+  (if (not feedmail-enable-queue) (feedmail-send-it-immediately-wrapper)
+    ;; else, queuing is enabled, should we ask about it or just do it?
+    (if feedmail-ask-before-queue
+       (funcall (feedmail-queue-send-edit-prompt))
+      (feedmail-dump-message-to-queue feedmail-queue-directory 'after-queue)))
+  ;; put this back
+  (if bfn-jiggle (setq feedmail-queue-buffer-file-name buffer-file-name))
+  )))
 
 (defun feedmail-message-action-send ()
   ;; hooks can make this take a while so clear the prompt
+  (feedmail-say-debug ">in-> feedmail-message-action-send")
   (message "FQM: Immediate send...")
-  (feedmail-send-it-immediately))
+  (feedmail-send-it-immediately-wrapper))
 
 
 ;; From a VM mailing list discussion and some suggestions from Samuel Mikes 
<address@hidden>
 (defun feedmail-queue-express-to-queue ()
-  "*Send message directly to the queue, with a minimum of fuss and bother."
+  "Send message directly to the queue, with a minimum of fuss and bother."
   (interactive)
+  (feedmail-say-debug ">in-> feedmail-queue-express-to-queue")
+  (run-hooks 'feedmail-queue-express-hook)
   (let ((feedmail-enable-queue t)
-               (feedmail-ask-before-queue nil)
-               (feedmail-queue-reminder-alist nil)
-               (feedmail-queue-chatty-sit-for 0))
-       (feedmail-send-it)
-       )
-)
+       (feedmail-ask-before-queue nil)
+       (feedmail-queue-reminder-alist nil)
+       (feedmail-queue-chatty-sit-for 0))
+    (feedmail-send-it)
+    )
+  )
 
 
 (defun feedmail-queue-express-to-draft ()
-  "*Send message directly to the draft queue, with a minimum of fuss and 
bother."
+  "Send message directly to the draft queue, with a minimum of fuss and 
bother."
   (interactive)
+  (feedmail-say-debug ">in-> feedmail-queue-express-to-draft")
   (let ((feedmail-queue-directory feedmail-queue-draft-directory))
-       (feedmail-queue-express-to-queue)
-       )
-)
+    (feedmail-queue-express-to-queue)
+    )
+  )
 
 
 (defun feedmail-message-action-send-strong ()
+  (feedmail-say-debug ">in-> feedmail-message-action-send-strong")
   (let ((feedmail-confirm-outgoing nil)) (feedmail-message-action-send)))
 
 
 (defun feedmail-message-action-edit ()
+  (feedmail-say-debug ">in-> feedmail-message-action-edit")
   (error "FQM: Message not queued; returning to edit"))
 
 
 (defun feedmail-message-action-draft ()
+  (feedmail-say-debug ">in-> feedmail-message-action-draft")
   (feedmail-dump-message-to-queue feedmail-queue-draft-directory 'after-draft))
 
 
 (defun feedmail-message-action-draft-strong ()
+  (feedmail-say-debug ">in-> feedmail-message-action-draft-strong")
   (let ((buffer-file-name nil))
-       (feedmail-message-action-draft)))
+    (feedmail-message-action-draft)))
 
 
 (defun feedmail-message-action-queue ()
+  (feedmail-say-debug ">in-> feedmail-message-action-queue")
   (feedmail-dump-message-to-queue feedmail-queue-directory 'after-queue))
 
 
 (defun feedmail-message-action-queue-strong ()
+  (feedmail-say-debug ">in-> feedmail-message-action-queue-strong")
   (let ((buffer-file-name nil))
-       (feedmail-message-action-queue)))
+    (feedmail-message-action-queue)))
 
 
 (defun feedmail-message-action-toggle-spray ()
+  (feedmail-say-debug ">in-> feedmail-message-action-toggle-spray")
   (let ((feedmail-enable-spray (not feedmail-enable-spray)))
-       (if feedmail-enable-spray
-               (message "FQM: For this message, spray toggled ON")
-         (message "FQM: For this message, spray toggled OFF"))
-       (sit-for 3)
-       ;; recursion, but harmless
-       (feedmail-send-it)))
+    (if feedmail-enable-spray
+       (message "FQM: For this message, spray toggled ON")
+      (message "FQM: For this message, spray toggled OFF"))
+    (sit-for 3)
+    ;; recursion, but harmless
+    (feedmail-send-it)))
+
 
+(defconst feedmail-p-h-b-n "*FQM Help*")
 
 (defun feedmail-message-action-help ()
-       (let ((d-string " "))
-         (if (stringp feedmail-ask-before-queue-default)
-                 (setq d-string feedmail-ask-before-queue-default)
-               (setq d-string  (char-to-string 
feedmail-ask-before-queue-default)))
-         (feedmail-queue-send-edit-prompt-help d-string)
-         ;; recursive, but no worries (it goes deeper on user action)
-         (feedmail-send-it)))
+  (feedmail-say-debug ">in-> feedmail-message-action-help")
+  (let ((d-string " ")
+       (fqm-help (get-buffer feedmail-p-h-b-n)))
+    (if (stringp feedmail-ask-before-queue-default)
+       (setq d-string feedmail-ask-before-queue-default)
+      (setq d-string  (char-to-string feedmail-ask-before-queue-default)))
+       (if (and fqm-help (get-buffer-window fqm-help))
+           (feedmail-scroll-buffer 'up fqm-help)
+         (feedmail-message-action-help-blat d-string))
+    ;; recursive, but no worries (it goes deeper on user action)
+    (feedmail-send-it)))
+
+(defun feedmail-message-action-help-blat (d-string)
+  (feedmail-say-debug ">in-> feedmail-message-action-help-blat")
+  (with-output-to-temp-buffer feedmail-p-h-b-n
+    (princ (substitute-command-keys "\
+You're dispatching a message and feedmail queuing is enabled.
+Typing ? again will normally scroll this help buffer.
+
+Choices:
+   q  QUEUE        for later sending (via feedmail-run-the-queue)
+   Q  QUEUE!       like \"q\", but always make a new file
+   i  IMMEDIATELY  send this (but not the other queued messages)
+   I  IMMEDIATELY! like \"i\", but skip following confirmation prompt
+   d  DRAFT        queue in the draft directory
+   D  DRAFT!       like \"d\", but always make a new file
+   e  EDIT         return to the message edit buffer (don't send or queue)
+   *  SPRAY        toggle spray mode (individual message transmissions)
+   >  SCROLL UP    scroll message up (toward end of message)
+   <  SCROLL DOWN  scroll message down (toward beginning of message)
+   ?  HELP         show or scroll this help buffer
+
+Synonyms:
+   s  SEND         immediately (same as \"i\")
+   S  SEND!        immediately (same as \"I\")
+   r  ROUGH        draft (same as \"d\")
+   R  ROUGH!       draft (same as \"D\")
+   n  NOPE         didn't mean it (same as \"e\")
+   y  YUP          do the default behavior (same as \"C-m\")
+  SPC SCROLL UP    (same as \">\")
+
+The user-configurable default is currently \""))
+       (princ d-string)
+       (princ "\".  For other possibilities,
+see the variable feedmail-prompt-before-queue-user-alist.
+")
+       (and (stringp feedmail-prompt-before-queue-help-supplement)
+                (princ feedmail-prompt-before-queue-help-supplement))
+    (with-current-buffer standard-output
+      (if (fboundp 'help-mode) (help-mode)))))
+
+
+(defun feedmail-message-action-scroll-up ()
+  (feedmail-say-debug ">in-> feedmail-message-action-scroll-up")
+  (feedmail-scroll-buffer 'up)
+  ;; recursive, but no worries (it goes deeper on user action)
+  (feedmail-send-it))
+
+
+(defun feedmail-message-action-scroll-down ()
+  (feedmail-say-debug ">in-> feedmail-message-action-scroll-down")
+  (feedmail-scroll-buffer 'down)
+  ;; recursive, but no worries (it goes deeper on user action)
+  (feedmail-send-it))
 
 
 ;;;###autoload
 (defun feedmail-run-the-queue-no-prompts (&optional arg)
-  "Like feedmail-run-the-queue, but suppress confirmation prompts."
+  "Like `feedmail-run-the-queue', but suppress confirmation prompts."
   (interactive "p")
+  (feedmail-say-debug ">in-> feedmail-run-the-queue-no-prompts")
   (let ((feedmail-confirm-outgoing nil)) (feedmail-run-the-queue arg)))
 
 ;;;###autoload
 (defun feedmail-run-the-queue-global-prompt (&optional arg)
-  "Like feedmail-run-the-queue, but with a global confirmation prompt.
-This is generally most useful if run non-interactively, since you can 
+  "Like `feedmail-run-the-queue', but with a global confirmation prompt.
+This is generally most useful if run non-interactively, since you can
 bail out with an appropriate answer to the global confirmation prompt."
   (interactive "p")
+  (feedmail-say-debug ">in-> feedmail-run-the-queue-global-prompts")
   (let ((feedmail-queue-runner-confirm-global t)) (feedmail-run-the-queue 
arg)))
 
 ;;;###autoload
@@ -1535,131 +1962,124 @@ feedmail-run-the-queue
 messages skipped, and number of non-message things in the queue (commonly
 backup file names and the like)."
   (interactive "p")
+  (feedmail-say-debug ">in-> feedmail-run-the-queue")
   ;; avoid matching trouble over slash vs backslash by getting canonical
   (if feedmail-queue-directory
-         (setq feedmail-queue-directory (expand-file-name 
feedmail-queue-directory)))
+      (setq feedmail-queue-directory (expand-file-name 
feedmail-queue-directory)))
   (if feedmail-queue-draft-directory
-         (setq feedmail-queue-draft-directory (expand-file-name 
feedmail-queue-draft-directory)))
+      (setq feedmail-queue-draft-directory (expand-file-name 
feedmail-queue-draft-directory)))
   (let* ((maybe-file)
-                (qlist (feedmail-look-at-queue-directory 
feedmail-queue-directory))
-                (dlist (feedmail-look-at-queue-directory 
feedmail-queue-draft-directory))
-                (q-cnt (nth 0 qlist))
-                (q-oth (nth 1 qlist))
-                (d-cnt (nth 0 dlist))
-                (d-oth (nth 1 dlist))
-                (messages-sent 0)
-                (messages-skipped 0)
-                (blobby-buffer)
-                (already-buffer)
-                (this-mhsep)
-                (do-the-run t)
-                (list-of-possible-fqms))
-       (if (and (> q-cnt 0) feedmail-queue-runner-confirm-global)
-               (setq do-the-run
-                         (if (fboundp 'y-or-n-p-with-timeout)
-                                 (y-or-n-p-with-timeout (format "FQM: Draft: 
%dm+%d,  Queue: %dm+%d; run the queue? "
-                                                                               
                 d-cnt d-oth q-cnt q-oth)
-                                                                               
 5 nil)
-                               (y-or-n-p (format "FQM: Draft: %dm+%d,  Queue: 
%dm+%d; run the queue? "
-                                                                 d-cnt d-oth 
q-cnt q-oth))
-                               )))
-       (if (not do-the-run)
-               (setq messages-skipped q-cnt)
-         (save-window-excursion
-               (setq list-of-possible-fqms (directory-files 
feedmail-queue-directory t))
-               (if feedmail-queue-run-orderer
-                       (setq list-of-possible-fqms (funcall 
feedmail-queue-run-orderer list-of-possible-fqms)))
-               (mapcar
-                '(lambda (blobby)
-                       (setq maybe-file (expand-file-name blobby 
feedmail-queue-directory))
-                       (cond
-                        ((file-directory-p maybe-file) nil) ; don't care about 
subdirs 
-                        ((feedmail-fqm-p blobby)
-                         (setq blobby-buffer (generate-new-buffer (concat "FQM 
" blobby)))
-                          (setq already-buffer
-                                        (if (fboundp 'find-buffer-visiting) ; 
missing from XEmacs
-                                                (find-buffer-visiting 
maybe-file)
-                                          (get-file-buffer maybe-file)))
-                          (if (and already-buffer (buffer-modified-p 
already-buffer))
-                                  (save-window-excursion
-                                        (display-buffer (set-buffer 
already-buffer))
-                                        (if (fboundp 'y-or-n-p-with-timeout)
-                                                ;; make a guess that the user 
just forgot to save
-                                                (if (y-or-n-p-with-timeout 
(format "FQM: Visiting %s; save before send? " blobby) 10 t)
-                                                        (save-buffer))
-                                          (if (y-or-n-p (format "FQM: Visiting 
%s; save before send? " blobby))
-                                                  (save-buffer))
-                                          )))
-                       
-                          (set-buffer blobby-buffer)
-                          (setq buffer-offer-save nil)
-                          (buffer-disable-undo blobby-buffer)
-                          (insert-file-contents-literally maybe-file)
-                          ;; work around text-vs-binary wierdness and also 
around rmail-resend's creative
-                          ;; manipulation of mail-header-separator
-                          ;;
-                          ;; if we don't find the normal M-H-S, and the 
alternative is defined but also
-                          ;; not found, try reading the file a different way
-                          ;;
-                          ;; if M-H-S not found and (a-M-H-S is nil or not 
found)
-                          (if (and (not (feedmail-find-eoh t))
-                                               (or (not 
feedmail-queue-alternative-mail-header-separator)
-                                                       (not 
-                                                        (let 
((mail-header-separator feedmail-queue-alternative-mail-header-separator))
-                                                          (feedmail-find-eoh 
t)))))
-                                  (let ((file-name-buffer-file-type-alist nil) 
(default-buffer-file-type nil))
-                                        (erase-buffer) (insert-file-contents 
maybe-file))
-                                )
-                          ;; if M-H-S not found and (a-M-H-S is non-nil and is 
found)
-                          ;; temporarily set M-H-S to the value of a-M-H-S
-                          (if (and (not (feedmail-find-eoh t))
-                                               
feedmail-queue-alternative-mail-header-separator
-                                               (let ((mail-header-separator 
feedmail-queue-alternative-mail-header-separator))
-                                                 (feedmail-find-eoh t)))
-                                  (setq this-mhsep 
feedmail-queue-alternative-mail-header-separator)
-                                (setq this-mhsep mail-header-separator))
-                          (funcall feedmail-queue-runner-mode-setter arg)
-                          (condition-case nil                  ; don't give up 
the loop if user skips some
-                                  (let ((feedmail-enable-queue nil)
-                                                (mail-header-separator 
this-mhsep)
-                                                
(feedmail-queue-runner-is-active maybe-file))
-                                        (funcall 
feedmail-queue-runner-message-sender arg)
-                                        (set-buffer blobby-buffer)
-                                        (if (buffer-modified-p) ; still 
modified, means wasn't sent
-                                                (setq messages-skipped (1+ 
messages-skipped))
-                                          (setq messages-sent (1+ 
messages-sent))
-                                          (funcall 
feedmail-queue-runner-cleaner-upper maybe-file arg)
-                                          (if (and already-buffer (not 
(file-exists-p maybe-file)))
-                                                  ;; we have gotten rid of the 
file associated with the
-                                                  ;; buffer, so update the 
buffer's notion of that
-                                                  (save-excursion
-                                                        (set-buffer 
already-buffer)
-                                                        (setq buffer-file-name 
nil)))))
-                                (error (setq messages-skipped (1+ 
messages-skipped))))
-                          (kill-buffer blobby-buffer)
-                          (if feedmail-queue-chatty
-                                  (progn
-                                        (message "FQM: %d to go, %d sent, %d 
skipped (%d other files ignored)"
-                                                         (- q-cnt 
messages-sent messages-skipped)
-                                                         messages-sent 
messages-skipped q-oth)
-                                        (sit-for 
feedmail-queue-chatty-sit-for))))))
-                 list-of-possible-fqms)))
-       (if feedmail-queue-chatty
-               (progn 
-                 (message "FQM: %d sent, %d skipped (%d other files ignored)"
-                                  messages-sent messages-skipped q-oth)
-                 (sit-for feedmail-queue-chatty-sit-for)
-                 (feedmail-queue-reminder 'after-run)
-                 (sit-for feedmail-queue-chatty-sit-for)))
-       (list messages-sent messages-skipped q-oth)))
+        (qlist (feedmail-look-at-queue-directory feedmail-queue-directory))
+        (dlist (feedmail-look-at-queue-directory 
feedmail-queue-draft-directory))
+        (q-cnt (nth 0 qlist))
+        (q-oth (nth 1 qlist))
+        (d-cnt (nth 0 dlist))
+        (d-oth (nth 1 dlist))
+        (messages-sent 0)
+        (messages-skipped 0)
+        (blobby-buffer)
+        (already-buffer)
+        (do-the-run t)
+        (list-of-possible-fqms))
+    (if (and (> q-cnt 0) feedmail-queue-runner-confirm-global)
+       (setq do-the-run
+             (if (fboundp 'y-or-n-p-with-timeout)
+                 (y-or-n-p-with-timeout (format "FQM: Draft: %dm+%d,  Queue: 
%dm+%d; run the queue? "
+                                                d-cnt d-oth q-cnt q-oth)
+                                        5 nil)
+               (y-or-n-p (format "FQM: Draft: %dm+%d,  Queue: %dm+%d; run the 
queue? "
+                                 d-cnt d-oth q-cnt q-oth))
+               )))
+    (if (not do-the-run)
+       (setq messages-skipped q-cnt)
+      (save-window-excursion
+       (setq list-of-possible-fqms (directory-files feedmail-queue-directory 
t))
+       (if feedmail-queue-run-orderer
+           (setq list-of-possible-fqms (funcall feedmail-queue-run-orderer 
list-of-possible-fqms)))
+       (mapc
+        (lambda (blobby)
+           (setq maybe-file (expand-file-name blobby feedmail-queue-directory))
+           (cond
+            ((file-directory-p maybe-file) nil) ; don't care about subdirs
+            ((feedmail-fqm-p blobby)
+             (setq blobby-buffer (generate-new-buffer (concat "FQM " blobby)))
+             (setq already-buffer
+                   (if (fboundp 'find-buffer-visiting) ; missing from XEmacs
+                       (find-buffer-visiting maybe-file)
+                     (get-file-buffer maybe-file)))
+             (if (and already-buffer (buffer-modified-p already-buffer))
+                 (save-window-excursion
+                   (display-buffer (set-buffer already-buffer))
+                   (if (fboundp 'y-or-n-p-with-timeout)
+                       ;; make a guess that the user just forgot to save
+                       (if (y-or-n-p-with-timeout (format "FQM: Visiting %s; 
save before send? " blobby) 10 t)
+                           (save-buffer))
+                     (if (y-or-n-p (format "FQM: Visiting %s; save before 
send? " blobby))
+                         (save-buffer))
+                     )))
+
+             (set-buffer blobby-buffer)
+             (setq buffer-offer-save nil)
+             (buffer-disable-undo blobby-buffer)
+             (insert-file-contents-literally maybe-file)
+             (goto-char (point-min))
+             ;; if at least two line-endings with CRLF, translate the file
+             (if (looking-at ".*\r\n.*\r\n")
+                 (while (search-forward "\r\n" nil t)
+                   (replace-match "\n" nil t)))
+             (funcall feedmail-queue-runner-mode-setter arg)
+             (condition-case signal-stuff ; don't give up the loop if user 
skips some
+                 (let ((feedmail-enable-queue nil)
+                       (feedmail-queue-runner-is-active maybe-file))
+                   ;; if can't find EOH, this is no message!
+                   (unless (feedmail-find-eoh t)
+                     (feedmail-say-chatter "Skipping %s; no 
mail-header-separator" maybe-file)
+                     (error "FQM: you should never see this message"))
+                   (feedmail-say-debug "Prepping %s" maybe-file)
+                   ;; the catch is a way out for users to voluntarily skip 
sending a message
+                   (catch 'skip-me-q (funcall 
feedmail-queue-runner-message-sender arg))
+                   (set-buffer blobby-buffer)
+                   (if (buffer-modified-p) ; still modified, means wasn't sent
+                       (progn
+                         (setq messages-skipped (1+ messages-skipped))
+                         (feedmail-say-chatter "%s wasn't sent by %s" 
maybe-file feedmail-buffer-eating-function))
+                     (setq messages-sent (1+ messages-sent))
+                     (funcall feedmail-queue-runner-cleaner-upper maybe-file 
arg)
+                     (if (and already-buffer (not (file-exists-p maybe-file)))
+                         ;; we have gotten rid of the file associated with the
+                         ;; buffer, so update the buffer's notion of that
+                         (with-current-buffer already-buffer
+                           (setq buffer-file-name nil)))))
+               ;; the handler for the condition-case
+               (error (setq messages-skipped (1+ messages-skipped))
+                      (ding t)
+                      (message "FQM: Trapped `%s', message left in queue." 
(car signal-stuff))
+                      (sit-for 3)
+                      (message "FQM: Trap details: \"%s\""
+                               (mapconcat 'identity (cdr signal-stuff) "\" 
\""))
+                      (sit-for 3)))
+             (kill-buffer blobby-buffer)
+             (feedmail-say-chatter
+              "%d to go, %d sent, %d skipped (%d other files ignored)"
+              (- q-cnt messages-sent messages-skipped)
+              messages-sent messages-skipped q-oth)
+             )))
+        list-of-possible-fqms)))
+    (if feedmail-queue-chatty
+       (progn
+         (feedmail-say-chatter "%d sent, %d skipped (%d other files ignored)"
+                               messages-sent messages-skipped q-oth)
+         (feedmail-queue-reminder 'after-run)
+         (sit-for feedmail-queue-chatty-sit-for)))
+    (list messages-sent messages-skipped q-oth)))
 
 
 ;;;###autoload
 (defun feedmail-queue-reminder (&optional what-event)
   "Perform some kind of reminder activity about queued and draft messages.
 Called with an optional symbol argument which says what kind of event
-is triggering the reminder activity.  The default is 'on-demand, which
-is what you typically would use if you were putting this in your emacs start-up
+is triggering the reminder activity.  The default is `on-demand', which
+is what you typically would use if you were putting this in your Emacs start-up
 or mail hook code.  Other recognized values for WHAT-EVENT (these are passed
 internally by feedmail):
 
@@ -1668,142 +2088,129 @@ feedmail-queue-reminder
    after-draft          (a message has just been placed in the draft directory)
    after-run            (the queue has just been run, possibly sending 
messages)
 
-WHAT-EVENT is used as a key into the table feedmail-queue-reminder-alist.  If
+WHAT-EVENT is used as a key into the table `feedmail-queue-reminder-alist'.  If
 the associated value is a function, it is called without arguments and is 
expected
-to perform the reminder activity.  You can supply your own reminder functions 
-by redefining feedmail-queue-reminder-alist.  If you don't want any reminders, 
-you can set feedmail-queue-reminder-alist to nil."
+to perform the reminder activity.  You can supply your own reminder functions
+by redefining `feedmail-queue-reminder-alist'.  If you don't want any 
reminders,
+you can set `feedmail-queue-reminder-alist' to nil."
   (interactive "p")
+  (feedmail-say-debug ">in-> feedmail-queue-reminder %s" what-event)
   (let ((key (if (and what-event (symbolp what-event)) what-event 'on-demand)) 
entry reminder)
-       (setq entry (assoc key feedmail-queue-reminder-alist))
-       (setq reminder (cdr entry))
-       (if (fboundp reminder) (funcall reminder)))
+    (setq entry (assoc key feedmail-queue-reminder-alist))
+    (setq reminder (cdr entry))
+    (if (fboundp reminder) (funcall reminder)))
   )
 
 
 (defun feedmail-queue-reminder-brief ()
-  "Brief display of draft and queued message counts in modeline."
+  "Brief display of draft and queued message counts in minibuffer."
   (interactive)
+  (feedmail-say-debug ">in-> feedmail-queue-reminder-brief")
   (let (q-cnt d-cnt q-lis d-lis)
-       (setq q-lis (feedmail-look-at-queue-directory feedmail-queue-directory))
-       (setq d-lis (feedmail-look-at-queue-directory 
feedmail-queue-draft-directory))
-       (setq q-cnt (car q-lis))
-       (setq d-cnt (car d-lis))
-       (if (or (> q-cnt 0) (> d-cnt 0))
-               (progn
-                 (message "FQM: [D: %d,  Q: %d]" d-cnt q-cnt))))
+    (setq q-lis (feedmail-look-at-queue-directory feedmail-queue-directory))
+    (setq d-lis (feedmail-look-at-queue-directory 
feedmail-queue-draft-directory))
+    (setq q-cnt (car q-lis))
+    (setq d-cnt (car d-lis))
+    (if (or (> q-cnt 0) (> d-cnt 0))
+       (progn
+         (message "FQM: [D: %d,  Q: %d]" d-cnt q-cnt))))
   )
 
 
 (defun feedmail-queue-reminder-medium ()
-  "Verbose display of draft and queued message counts in modeline."
+  "Verbose display of draft and queued message counts in minibuffer."
   (interactive)
+  (feedmail-say-debug ">in-> feedmail-queue-reminder-medium")
   (let (q-cnt d-cnt q-oth d-oth q-lis d-lis)
-       (setq q-lis (feedmail-look-at-queue-directory feedmail-queue-directory))
-       (setq d-lis (feedmail-look-at-queue-directory 
feedmail-queue-draft-directory))
-       (setq q-cnt (car q-lis))
-       (setq d-cnt (car d-lis))
-       (setq q-oth (nth 1 q-lis))
-       (setq d-oth (nth 1 d-lis))
-       (if (or (> q-cnt 0) (> d-cnt 0))
-               (progn
-                 (message "FQM: Draft: %dm+%d in \"%s\",  Queue: %dm+%d in 
\"%s\""
-                                  d-cnt d-oth (file-name-nondirectory 
feedmail-queue-draft-directory)
-                                  q-cnt q-oth (file-name-nondirectory 
feedmail-queue-directory)))))
+    (setq q-lis (feedmail-look-at-queue-directory feedmail-queue-directory))
+    (setq d-lis (feedmail-look-at-queue-directory 
feedmail-queue-draft-directory))
+    (setq q-cnt (car q-lis))
+    (setq d-cnt (car d-lis))
+    (setq q-oth (nth 1 q-lis))
+    (setq d-oth (nth 1 d-lis))
+    (if (or (> q-cnt 0) (> d-cnt 0))
+       (progn
+         (message "FQM: Draft: %dm+%d in \"%s\",  Queue: %dm+%d in \"%s\""
+                  d-cnt d-oth (file-name-nondirectory 
feedmail-queue-draft-directory)
+                  q-cnt q-oth (file-name-nondirectory 
feedmail-queue-directory)))))
   )
 
 
 (defun feedmail-queue-send-edit-prompt ()
-  "Ask whether to queue, send immediately, or return to editing a message."
+  "Ask whether to queue, send immediately, or return to editing a message, 
etc."
+  (feedmail-say-debug ">in-> feedmail-queue-send-edit-prompt")
+  (feedmail-queue-send-edit-prompt-inner
+   feedmail-ask-before-queue-default
+   feedmail-ask-before-queue-prompt
+   feedmail-ask-before-queue-reprompt
+   'feedmail-message-action-help
+   feedmail-prompt-before-queue-standard-alist
+   feedmail-prompt-before-queue-user-alist
+   ))
+
+(defun feedmail-queue-send-edit-prompt-inner (default prompt reprompt helper
+                                             standard-alist user-alist)
+  (feedmail-say-debug ">in-> feedmail-queue-send-edit-prompt-inner")
   ;; Some implementation ideas here came from the userlock.el code
-  (discard-input)
+  (or defining-kbd-macro (discard-input))
   (save-window-excursion
-       (let ((answer) (d-char) (d-string " "))
-         (if (stringp feedmail-ask-before-queue-default)
-                 (progn
-                       (setq d-char   (string-to-char 
feedmail-ask-before-queue-default))
-                       (setq d-string feedmail-ask-before-queue-default))
-               (setq d-string  (char-to-string 
feedmail-ask-before-queue-default))
-               (setq d-char    feedmail-ask-before-queue-default)
-               )
+    (let ((answer) (d-char) (d-string " "))
+      (if (stringp default)
+         (progn
+           (setq d-char   (string-to-char default)
+                 d-string default))
+       (setq d-string  (char-to-string default))
+       (setq d-char    default)
+       )
       (while (null answer)
-               (message feedmail-ask-before-queue-prompt d-string)
-               (let ((user-sez
-                          (let ((inhibit-quit t) (cursor-in-echo-area t) 
(echo-keystrokes 0))
-                                (read-char-exclusive))))
-                 (if (= user-sez help-char)
-                         (setq answer '(^ . feedmail-message-action-help))
-                       (if (or (eq user-sez ?\C-m) (eq user-sez ?\C-j) (eq 
user-sez ?y))
-                               (setq user-sez d-char))
-                       ;; these char-to-int things because of some 
incomprensible difference
-                       ;; between the two in byte-compiled stuff between 
GNUemacs and XEmacs
-                       ;; (well, I'm sure someone could comprehend it, but I 
say 'uncle')
-                       (setq answer (or (assoc user-sez 
feedmail-prompt-before-queue-user-alist)
-                                                        (and (fboundp 
'char-to-int)
-                                                                 (assoc 
(char-to-int user-sez) feedmail-prompt-before-queue-user-alist))
-                                                        (assoc user-sez 
feedmail-prompt-before-queue-standard-alist)
-                                                        (and (fboundp 
'char-to-int)
-                                                                 (assoc 
(char-to-int user-sez) feedmail-prompt-before-queue-standard-alist))))
-                       (if (or (null answer) (null (cdr answer)))
-                               (progn
-                                 (beep)
-                                 (message feedmail-ask-before-queue-reprompt 
d-string)
-                                 (sit-for 3)))
-                        )))
-         (cdr answer)
-         )))
-
-(defconst feedmail-p-h-b-n "*FQM Help*")
-
-(defun feedmail-queue-send-edit-prompt-help (d-string)
-  (let ((fqm-help (get-buffer feedmail-p-h-b-n)))
-       (if (and fqm-help (get-buffer-window fqm-help))
-               (feedmail-queue-send-edit-prompt-help-later fqm-help d-string)
-         (feedmail-queue-send-edit-prompt-help-first d-string))))
-
-(defun feedmail-queue-send-edit-prompt-help-later (fqm-help d-string)
+       (message prompt d-string)
+       (let ((user-sez
+              (let ((inhibit-quit t) (cursor-in-echo-area t) (echo-keystrokes 
0))
+                (read-char-exclusive))))
+         (if (= user-sez help-char)
+              ;; FIXME: This seems to want to refer to the `helper' argument,
+              ;; but it's quoted so the `helper' arg ends up unused!
+             (setq answer '(^ . helper))
+           (if (or (eq user-sez ?\C-m) (eq user-sez ?\C-j) (eq user-sez ?y))
+               (setq user-sez d-char))
+           ;; these char-to-int things are because of some
+           ;; incomprehensible difference between the two in
+           ;; byte-compiled stuff between Emacs and XEmacs
+           ;; (well, I'm sure someone could comprehend it,
+           ;; but I say 'uncle')
+           (setq answer (or (assoc user-sez user-alist)
+                            (and (fboundp 'char-to-int)
+                                 (assoc (char-to-int user-sez) user-alist))
+                            (assoc user-sez standard-alist)
+                            (and (fboundp 'char-to-int)
+                                 (assoc (char-to-int user-sez) 
standard-alist))))
+           (if (or (null answer) (null (cdr answer)))
+               (progn
+                 (beep)
+                 (message reprompt d-string)
+                 (sit-for 3)))
+           )))
+      (cdr answer)
+      )))
+
+(defun feedmail-scroll-buffer (direction &optional buffy)
   ;; scrolling fun
+  ;; emacs convention is that scroll-up moves text up, window down
+  (feedmail-say-debug ">in-> feedmail-scroll-buffer %s" direction)
   (save-selected-window
-       (let ((signal-error-on-buffer-boundary nil)
-                 (fqm-window (display-buffer fqm-help)))
-         (select-window fqm-window)
-         (if (pos-visible-in-window-p (point-max) fqm-window)
-                 (feedmail-queue-send-edit-prompt-help-first d-string)
-;;               (goto-char (point-min))
-               (scroll-up nil)
-               ))))
-
-(defun feedmail-queue-send-edit-prompt-help-first (d-string)
-  (with-output-to-temp-buffer feedmail-p-h-b-n
-    (princ "You're dispatching a message and feedmail queuing is enabled.
-Typing ? or C-v will normally scroll this help buffer.
+    (let ((signal-error-on-buffer-boundary nil) ;FIXME: Unknown var!?
+         (fqm-window (display-buffer (if buffy buffy (current-buffer)))))
+      (select-window fqm-window)
+      (if (eq direction 'up)
+      (if (pos-visible-in-window-p (point-max) fqm-window)
+         ;; originally just (goto-char (point-min)), but
+         ;; pos-visible-in-window-p seems oblivious to that
+         (scroll-down 999999)
+       (scroll-up))
+      (if (pos-visible-in-window-p (point-min) fqm-window)
+         (scroll-up 999999)
+       (scroll-down))))))
 
-Choices:
-   q  QUEUE        for later sending (via feedmail-run-the-queue)
-   Q  QUEUE!       like \"q\", but always make a new file
-   i  IMMEDIATELY  send this (but not the other queued messages)
-   I  IMMEDIATELY! like \"i\", but skip following confirmation prompt
-   d  DRAFT        queue in the draft directory
-   D  DRAFT!       like \"d\", but always make a new file
-   e  EDIT         return to the message edit buffer (don't send or queue)
-   *  SPRAY        toggle spray mode (individual message transmissions)
-
-Synonyms:
-   s  SEND         immediately (same as \"i\")
-   S  SEND!        immediately (same as \"I\")
-   r  ROUGH        draft (same as \"d\")
-   R  ROUGH!       draft (same as \"R\")
-   n  NOPE         didn't mean it (same as \"e\")
-   y  YUP          do the default behavior (same as \"C-m\")
-
-The user-configurable default is currently \"")
-       (princ d-string)
-       (princ "\".  For other possibilities,
-see the variable feedmail-prompt-before-queue-user-alist.
-")
-       (and (stringp feedmail-prompt-before-queue-help-supplement)
-                (princ feedmail-prompt-before-queue-help-supplement))
-    (save-excursion (set-buffer standard-output) (if (fboundp 'help-mode) 
(help-mode)))))
 
 (defun feedmail-look-at-queue-directory (queue-directory)
   "Find out some things about a queue directory.
@@ -1811,480 +2218,574 @@ feedmail-look-at-queue-directory
 directory, a count of other files in the directory, and a high water
 mark for prefix sequence numbers.  Subdirectories are not included in
 the counts."
+  (feedmail-say-debug ">in-> feedmail-look-at-queue-directory %s" 
queue-directory)
   (let ((q-cnt 0) (q-oth 0) (high-water 0) (blobbet))
-       ;; iterate, counting things we find along the way in the directory
-       (if (file-directory-p queue-directory)
-               (mapcar
-                '(lambda (blobby)
-                       (cond
-                        ((file-directory-p blobby) nil) ; don't care about 
subdirs
-                        ((feedmail-fqm-p blobby)
-                         (setq blobbet (file-name-nondirectory blobby))
-                         (if (string-match "^[0-9][0-9][0-9]-" blobbet)
-                                 (let ((water-mark))
-                                       (setq water-mark (string-to-int 
(substring blobbet 0 3)))
-                                       (if (> water-mark high-water) (setq 
high-water water-mark))))
-                         (setq q-cnt (1+ q-cnt)))
-                        (t (setq q-oth (1+ q-oth)))
-                        ))
-                (directory-files queue-directory t)))
-       (list q-cnt q-oth high-water)))
+    ;; iterate, counting things we find along the way in the directory
+    (if (file-directory-p queue-directory)
+       (mapc
+        (lambda (blobby)
+           (cond
+            ((file-directory-p blobby) nil) ; don't care about subdirs
+            ((feedmail-fqm-p blobby)
+             (setq blobbet (file-name-nondirectory blobby))
+             (if (string-match "^[0-9][0-9][0-9]-" blobbet)
+                 (let ((water-mark))
+                   (setq water-mark (string-to-number (substring blobbet 0 3)))
+                   (if (> water-mark high-water) (setq high-water 
water-mark))))
+             (setq q-cnt (1+ q-cnt)))
+            (t (setq q-oth (1+ q-oth)))
+            ))
+        (directory-files queue-directory t)))
+    (list q-cnt q-oth high-water)))
 
 (defun feedmail-tidy-up-slug (slug)
-  "Utility for mapping out suspect characters in a potential filename"
+  "Utility for mapping out suspect characters in a potential filename."
+  (feedmail-say-debug ">in-> feedmail-tidy-up-slug %s" slug)
   ;; even programmers deserve a break sometimes, so cover nil for them
   (if (null slug) (setq slug ""))
   ;; replace all non-alphanumerics with hyphen for safety
-  (while (string-match "[^a-z0-9-]+" slug) (setq slug (replace-match "-" nil 
nil slug)))
+  (while (string-match feedmail-queue-slug-suspect-regexp slug) (setq slug 
(replace-match "-" nil nil slug)))
   ;; collapse multiple hyphens to one
   (while (string-match "--+" slug) (setq slug (replace-match "-" nil nil 
slug)))
-  ;; for tidyness, peel off leading hyphens
+  ;; for tidiness, peel off leading hyphens
   (if (string-match "^-*" slug) (setq slug (replace-match "" nil nil slug)))
-  ;; for tidyness, peel off trailing hyphens
+  ;; for tidiness, peel off trailing hyphens
   (if (string-match "-*$" slug) (setq slug (replace-match "" nil nil slug)))
   slug
-)  
+  )
 
 (defun feedmail-queue-subject-slug-maker (&optional queue-directory)
   "Create a name for storing the message in the queue.
 Optional argument QUEUE-DIRECTORY specifies into which directory the
-file will be placed.  The name is based on the SUBJECT: header (if
+file will be placed.  The name is based on the Subject: header (if
 there is one).  If there is no subject,
-feedmail-queue-default-file-slug is consulted Special characters are
+`feedmail-queue-default-file-slug' is consulted.  Special characters are
 mapped to mostly alphanumerics for safety."
+  (feedmail-say-debug ">in-> feedmail-queue-subject-slug-maker %s" 
queue-directory)
   (let ((eoh-marker) (case-fold-search t) (subject "") (s-point))
-       (setq eoh-marker (feedmail-find-eoh))
-       (goto-char (point-min))
-       ;; get raw subject value (first line, anyhow)
-       (if (re-search-forward "^SUBJECT:" eoh-marker t)
-               (progn (setq s-point (point))
-                          (end-of-line)
-                          (setq subject (buffer-substring s-point (point)))))
-       (setq subject (feedmail-tidy-up-slug subject))
-       (if (zerop (length subject))
-               (setq subject
-                         (cond
-                          ((stringp feedmail-queue-default-file-slug) 
feedmail-queue-default-file-slug)
-                          ((fboundp feedmail-queue-default-file-slug)
-                               (save-excursion (funcall 
feedmail-queue-default-file-slug)))
-                          ((eq feedmail-queue-default-file-slug 'ask)
-                               (file-name-nondirectory
-                                (read-file-name "FQM: Message filename slug? "
-                                                                
(file-name-as-directory queue-directory) subject nil subject)))
-                          (t "no subject"))
-                         ))
-       (feedmail-tidy-up-slug subject) ;; one more time, with feeling
-       ))
+    (setq eoh-marker (feedmail-find-eoh))
+    (goto-char (point-min))
+    ;; get raw subject value (first line, anyhow)
+    (if (re-search-forward "^Subject:" eoh-marker t)
+       (progn (setq s-point (point))
+              (end-of-line)
+              (setq subject (buffer-substring-no-properties s-point (point)))))
+    (setq subject (feedmail-tidy-up-slug subject))
+    (if (zerop (length subject))
+       (setq subject
+             (cond
+              ((stringp feedmail-queue-default-file-slug) 
feedmail-queue-default-file-slug)
+              ((fboundp feedmail-queue-default-file-slug)
+               (save-excursion (funcall feedmail-queue-default-file-slug)))
+              ((eq feedmail-queue-default-file-slug 'ask)
+               (file-name-nondirectory
+                (read-file-name "FQM: Message filename slug? "
+                                (file-name-as-directory queue-directory) 
subject nil subject)))
+              (t "no subject"))
+             ))
+    ;; one more time, with feeling
+    (feedmail-tidy-up-slug subject)
+    ))
 
 
 (defun feedmail-create-queue-filename (queue-directory)
+  (feedmail-say-debug ">in-> feedmail-create-queue-filename %s" 
queue-directory)
   (let ((slug "wjc"))
-       (cond
-        (feedmail-queue-slug-maker
-         (save-excursion (setq slug (funcall feedmail-queue-slug-maker 
queue-directory))))
-        (feedmail-ask-for-queue-slug
-         (setq slug (file-name-nondirectory
-                                 (read-file-name (concat "FQM: Message 
filename slug? [" slug "]? ")
-                                                                
(file-name-as-directory queue-directory) slug nil slug))))
-         )
-       (setq slug (feedmail-tidy-up-slug slug))
-       (setq slug (format "%03d-%s" (1+ (nth 2 
(feedmail-look-at-queue-directory queue-directory))) slug))
-       (concat
-        (expand-file-name slug queue-directory)
-        feedmail-queue-fqm-suffix)
-       ))
+    (cond
+     (feedmail-queue-slug-maker
+      (save-excursion (setq slug (funcall feedmail-queue-slug-maker 
queue-directory))))
+     (feedmail-ask-for-queue-slug
+      (setq slug (file-name-nondirectory
+                 (read-file-name (concat "FQM: Message filename slug? [" slug 
"]? ")
+                                 (file-name-as-directory queue-directory) slug 
nil slug))))
+     )
+    (setq slug (feedmail-tidy-up-slug slug))
+    (setq slug (format "%03d-%s" (1+ (nth 2 (feedmail-look-at-queue-directory 
queue-directory))) slug))
+    (concat
+     (expand-file-name slug queue-directory)
+     feedmail-queue-fqm-suffix)
+    ))
 
 
 (defun feedmail-dump-message-to-queue (queue-directory what-event)
+  (feedmail-say-debug ">in-> feedmail-dump-message-to-queue %s %s" 
queue-directory what-event)
   (or (file-accessible-directory-p queue-directory)
-         ;; progn to get nil result no matter what
-         (progn (make-directory queue-directory t) nil)
-         (file-accessible-directory-p queue-directory)
-         (error (concat "FQM: Message not queued; trouble with directory " 
queue-directory)))
+      ;; progn to get nil result no matter what
+      (progn (make-directory queue-directory t) nil)
+      (file-accessible-directory-p queue-directory)
+      (error "FQM: Message not queued; trouble with directory %s" 
queue-directory))
   (let ((filename)
-               (is-fqm)
-               (is-in-this-dir)
-               (previous-buffer-file-name buffer-file-name))
-       (if buffer-file-name
-               (progn
-                 (setq is-fqm (feedmail-fqm-p buffer-file-name))
-                 (setq is-in-this-dir (string-equal
-                                                               
(directory-file-name queue-directory)
-                                                               
(directory-file-name (expand-file-name (file-name-directory 
buffer-file-name)))))))
-       ;; if visiting a queued message, just save
-       (if (and is-fqm is-in-this-dir)
-               (setq filename buffer-file-name)
-         (setq filename (feedmail-create-queue-filename queue-directory)))
-       ;; make binary file on DOS/Win95/WinNT, etc
-       (let ((buffer-file-type feedmail-force-binary-write)) (write-file 
filename))
-       ;; convenient for moving from draft to q, for example
-       (if (and previous-buffer-file-name (or (not is-fqm) (not 
is-in-this-dir))
-                        (y-or-n-p (format "FQM: Was previously %s; delete 
that? " previous-buffer-file-name)))
-               (delete-file previous-buffer-file-name))
-       (if feedmail-nuke-buffer-after-queue
-               (let ((a-s-file-name buffer-auto-save-file-name))
-                 ;; be aggressive in nuking auto-save files
-                 (and (kill-buffer (current-buffer))
-                          delete-auto-save-files
-                          (file-exists-p a-s-file-name)
-                          (delete-file a-s-file-name))))
-       (if feedmail-queue-chatty
-               (progn (message (concat "FQM: Queued in " filename))
-                          (sit-for feedmail-queue-chatty-sit-for)))
-       (if feedmail-queue-chatty
-               (progn
-                 (feedmail-queue-reminder what-event)
-                 (sit-for feedmail-queue-chatty-sit-for)))))
+       (is-fqm)
+       (is-in-this-dir)
+       (previous-buffer-file-name buffer-file-name))
+    (if buffer-file-name
+       (progn
+         (setq is-fqm (feedmail-fqm-p buffer-file-name))
+         (setq is-in-this-dir (string-equal
+                               (directory-file-name
+                                (expand-file-name queue-directory))
+                               (directory-file-name (expand-file-name 
(file-name-directory buffer-file-name)))))))
+    ;; if visiting a queued message, just save
+    (if (and is-fqm is-in-this-dir)
+       (setq filename buffer-file-name)
+      (setq filename (feedmail-create-queue-filename queue-directory)))
+    ;; make binary file on DOS/Windows 95/Windows NT, etc
+    (let ((coding-system-for-write
+          (if feedmail-force-binary-write
+              'no-conversion
+            coding-system-for-write)))
+      (write-file filename))
+    ;; convenient for moving from draft to q, for example
+    (if (and previous-buffer-file-name (or (not is-fqm) (not is-in-this-dir))
+            (let (d b s)
+              (setq b (file-name-nondirectory previous-buffer-file-name))
+              (setq d (file-name-directory previous-buffer-file-name))
+              (setq s (substring d (1- (length d))))
+              (setq d (substring d 0 (1- (length d))))
+              (setq d (file-name-nondirectory d))
+              (y-or-n-p (format "FQM: Was previously %s%s%s; delete that? "
+                                d s b))))
+       (delete-file previous-buffer-file-name))
+    (if feedmail-nuke-buffer-after-queue
+       (let ((a-s-file-name buffer-auto-save-file-name))
+         ;; be aggressive in nuking auto-save files
+         (and (kill-buffer (current-buffer))
+              delete-auto-save-files
+              (file-exists-p a-s-file-name)
+              (delete-file a-s-file-name))))
+    (feedmail-say-chatter "Queued in %s" filename)
+    (if feedmail-queue-chatty
+       (progn
+         (feedmail-queue-reminder what-event)
+         (sit-for feedmail-queue-chatty-sit-for)))))
 
 
 ;; from a similar function in mail-utils.el
 (defun feedmail-rfc822-time-zone (time)
+  (feedmail-say-debug ">in-> feedmail-rfc822-time-zone %s" time)
   (let* ((sec (or (car (current-time-zone time)) 0))
-                (absmin (/ (abs sec) 60)))
+        (absmin (/ (abs sec) 60)))
     (format "%c%02d%02d" (if (< sec 0) ?- ?+) (/ absmin 60) (% absmin 60))))
 
 (defun feedmail-rfc822-date (arg-time)
-  (let ((time (if arg-time arg-time (current-time))))
-       (concat
-        (format-time-string "%a, %e %b %Y %T " time)
-        (feedmail-rfc822-time-zone time)
-        )))
+  (feedmail-say-debug ">in-> feedmail-rfc822-date %s" arg-time)
+  (let ((time (if arg-time arg-time (current-time)))
+       (system-time-locale "C"))
+    (concat
+     (format-time-string "%a, %e %b %Y %T " time)
+     (feedmail-rfc822-time-zone time)
+     )))
+
+(defun feedmail-send-it-immediately-wrapper ()
+  "Wrapper to catch skip-me-i"
+  (if (eq 'skip-me-i (catch 'skip-me-i (feedmail-send-it-immediately)))
+      (error "FQM: Sending...abandoned!")))
 
+(declare-function expand-mail-aliases "mailalias" (beg end &optional exclude))
 
 (defun feedmail-send-it-immediately ()
   "Handle immediate sending, including during a queue run."
-  (let* ((feedmail-error-buffer (get-buffer-create " *FQM Outgoing Email 
Errors*"))
-                (feedmail-prepped-text-buffer (get-buffer-create " *FQM 
Outgoing Email Text*"))
-                (feedmail-raw-text-buffer (current-buffer))
-                (feedmail-address-list)
-                (eoh-marker)
-                (bcc-holder)
-                (resent-bcc-holder)
-                (a-re-rtcb  "^RESENT-\\(TO\\|CC\\|BCC\\):")
-                (a-re-rtc   "^RESENT-\\(TO\\|CC\\):")
-                (a-re-rb    "^RESENT-BCC:")
-                (a-re-dtcb  "^\\(TO\\|CC\\|BCC\\):")
-                (a-re-dtc   "^\\(TO\\|CC\\):")
-                (a-re-db    "^BCC:")
-                (mail-header-separator mail-header-separator) ;; to get a 
temporary changable copy
-                )
+  (feedmail-say-debug ">in-> feedmail-send-it-immediately")
+  (let ((feedmail-error-buffer
+         (get-buffer-create " *FQM Outgoing Email Errors*"))
+       (feedmail-prepped-text-buffer
+         (get-buffer-create " *FQM Outgoing Email Text*"))
+       (feedmail-raw-text-buffer (current-buffer))
+       (feedmail-address-list)
+       (eoh-marker)
+       (bcc-holder)
+       (resent-bcc-holder)
+       (a-re-rtcb  "^Resent-\\(To\\|Cc\\|Bcc\\):")
+       (a-re-rtc   "^Resent-\\(To\\|Cc\\):")
+       (a-re-rb    "^Resent-Bcc:")
+       (a-re-dtcb  "^\\(To\\|Cc\\|Bcc\\):")
+       (a-re-dtc   "^\\(To\\|Cc\\):")
+       (a-re-db    "^Bcc:")
+       ;; To get a temporary changeable copy.
+       (mail-header-separator mail-header-separator)
+       )
     (unwind-protect
-               (save-excursion
-                 (set-buffer feedmail-error-buffer) (erase-buffer)
-                 (set-buffer feedmail-prepped-text-buffer) (erase-buffer)
-
-                 ;; jam contents of user-supplied mail buffer into our scratch 
buffer
-                 (insert-buffer feedmail-raw-text-buffer)
-
-                 ;; require one newline at the end.
-                 (goto-char (point-max))
-                 (or (= (preceding-char) ?\n) (insert ?\n))
-
-                 (let ((case-fold-search nil))
-                       ;; Change header-delimiter to be what mailers expect 
(empty line).
-                       (setq eoh-marker (feedmail-find-eoh)) ;; leaves match 
data in place or signals error
-                       (replace-match "\n")
-                       (setq mail-header-separator ""))
-
-                 ;; mail-aliases nil = mail-abbrevs.el
-                 (if (or feedmail-force-expand-mail-aliases
-                                 (and (fboundp 'expand-mail-aliases) 
mail-aliases))
-                         (expand-mail-aliases (point-min) eoh-marker))
-
-                 ;; make it pretty
-                 (if feedmail-fill-to-cc (feedmail-fill-to-cc-function 
eoh-marker))
-                 ;; ignore any blank lines in the header
+       (save-current-buffer
+         (set-buffer feedmail-error-buffer) (erase-buffer)
+         (set-buffer feedmail-prepped-text-buffer) (erase-buffer)
+
+         ;; Jam contents of user-supplied mail buffer into our scratch buffer.
+         (insert-buffer-substring feedmail-raw-text-buffer)
+
+         ;; Require one newline at the end.
+         (goto-char (point-max))
+         (or (= (preceding-char) ?\n) (insert ?\n))
+
+         (let ((case-fold-search nil))
+           ;; Change header-delimiter to be what mailers expect (empty line).
+           ;; leaves match data in place or signals error
+           (feedmail-say-debug "looking for m-h-s \"%s\""
+                               mail-header-separator)
+           (setq eoh-marker (feedmail-find-eoh))
+           (feedmail-say-debug "found m-h-s %s" eoh-marker)
+           (setq mail-header-separator "")
+           (replace-match ""))
+;;       (replace-match "\\1")) ;; might be empty or "\r"
+
+         ;; mail-aliases nil = mail-abbrevs.el
+         (feedmail-say-debug "expanding mail aliases")
+         (if (or feedmail-force-expand-mail-aliases
+                 (and (fboundp 'expand-mail-aliases) mail-aliases))
+             (expand-mail-aliases (point-min) eoh-marker))
+
+         ;; Make it pretty.
+         (if feedmail-fill-to-cc (feedmail-fill-to-cc-function eoh-marker))
+         ;; Ignore any blank lines in the header.
+         (goto-char (point-min))
+         (while (and (re-search-forward "\n\n\n*" eoh-marker t)
+                      (< (point) eoh-marker))
+           (replace-match "\n"))
+
+         (let ((case-fold-search t) (addr-regexp))
+           (goto-char (point-min))
+           ;; There are some RFC-822 combinations/cases missed here,
+           ;; but probably good enough and what users expect.
+           ;;
+           ;; Use resent-* stuff only if there is at least one non-empty one.
+           (setq feedmail-is-a-resend
+                 (re-search-forward
+                  ;; Header name, followed by optional whitespace, followed by
+                  ;; non-whitespace, followed by anything, followed by
+                   ;; newline; the idea is empty Resent-* headers are ignored.
+                  "^\\(Resent-To:\\|Resent-Cc:\\|Resent-Bcc:\\)\\s-*\\S-+.*$"
+                  eoh-marker t))
+           ;; If we say so, gather the Bcc stuff before the main course.
+           (when (eq feedmail-deduce-bcc-where 'first)
+              (setq addr-regexp (if feedmail-is-a-resend a-re-rb a-re-db))
+              (setq feedmail-address-list
+                    (feedmail-deduce-address-list
+                     feedmail-prepped-text-buffer (point-min) eoh-marker
+                     addr-regexp feedmail-address-list)))
+           ;; The main course.
+            (setq addr-regexp
+                  (if (memq feedmail-deduce-bcc-where '(first last))
+                      ;; Handled by first or last cases, so don't get
+                      ;; Bcc stuff.
+                      (if feedmail-is-a-resend a-re-rtc a-re-dtc)
+                    ;; Not handled by first or last cases, so also get
+                    ;; Bcc stuff.
+                    (if feedmail-is-a-resend a-re-rtcb a-re-dtcb)))
+            (setq feedmail-address-list
+                  (feedmail-deduce-address-list
+                   feedmail-prepped-text-buffer (point-min) eoh-marker
+                   addr-regexp feedmail-address-list))
+           ;; If we say so, gather the Bcc stuff after the main course.
+           (when (eq feedmail-deduce-bcc-where 'last)
+              (setq addr-regexp (if feedmail-is-a-resend a-re-rb a-re-db))
+              (setq feedmail-address-list
+                    (feedmail-deduce-address-list
+                     feedmail-prepped-text-buffer (point-min) eoh-marker
+                     addr-regexp feedmail-address-list)))
+           (if (not feedmail-address-list)
+                (error "FQM: Sending...abandoned, no addressees"))
+           ;; Not needed, but meets user expectations.
+           (setq feedmail-address-list (nreverse feedmail-address-list))
+           ;; Find and handle any Bcc fields.
+           (setq bcc-holder
+                  (feedmail-accume-n-nuke-header eoh-marker "^Bcc:"))
+           (setq resent-bcc-holder
+                  (feedmail-accume-n-nuke-header eoh-marker "^Resent-Bcc:"))
+           (when (and bcc-holder (not feedmail-nuke-bcc))
+              (goto-char (point-min))
+              (insert bcc-holder))
+           (when (and resent-bcc-holder (not feedmail-nuke-resent-bcc))
+              (goto-char (point-min))
+              (insert resent-bcc-holder))
+           (goto-char (point-min))
+
+           ;; fiddle about, fiddle about, fiddle about....
+           (feedmail-fiddle-from)
+           (feedmail-fiddle-sender)
+           (feedmail-fiddle-x-mailer)
+           (feedmail-fiddle-message-id
+            (or feedmail-queue-runner-is-active
+                 (buffer-file-name feedmail-raw-text-buffer)))
+           (feedmail-fiddle-date
+            (or feedmail-queue-runner-is-active
+                 (buffer-file-name feedmail-raw-text-buffer)))
+           (feedmail-fiddle-list-of-fiddle-plexes
+             feedmail-fiddle-plex-user-list)
+
+           ;; don't send out a blank headers of various sorts
+           ;; (this loses on continued line with a blank first line)
+           (goto-char (point-min))
+           (and feedmail-nuke-empty-headers ; hey, who's an empty-header?
+                (while (re-search-forward "^[A-Za-z0-9-]+:[ \t]*\n"
+                                           eoh-marker t)
+                  (replace-match ""))))
+
+         (feedmail-say-debug "last chance hook: %s" feedmail-last-chance-hook)
+         (run-hooks 'feedmail-last-chance-hook)
+
+         (save-window-excursion
+         (let ((fcc (feedmail-accume-n-nuke-header eoh-marker "^Fcc:"))
+               (also-file)
+               (confirm (cond
+                         ((eq feedmail-confirm-outgoing 'immediate)
+                          (not feedmail-queue-runner-is-active))
+                         ((eq feedmail-confirm-outgoing 'queued)
+                           feedmail-queue-runner-is-active)
+                         (t feedmail-confirm-outgoing)))
+               (fullframe (cond
+                           ((eq feedmail-display-full-frame 'immediate)
+                            (not feedmail-queue-runner-is-active))
+                           ((eq feedmail-display-full-frame 'queued)
+                             feedmail-queue-runner-is-active)
+                           (t feedmail-display-full-frame))))
+           (if fullframe
+               (progn
+                 (switch-to-buffer feedmail-prepped-text-buffer t)
+                 (delete-other-windows)))
+           (if (or (not confirm)
+                    (feedmail-one-last-look feedmail-prepped-text-buffer))
+               (let ((user-mail-address
+                       (feedmail-envelope-deducer eoh-marker)))
+                 (feedmail-say-debug "give it to buffer-eater")
+                 (feedmail-give-it-to-buffer-eater)
+                 (feedmail-say-debug "gave it to buffer-eater")
+                 (if (and (not feedmail-queue-runner-is-active)
+                           (setq also-file
+                                 (buffer-file-name feedmail-raw-text-buffer)))
+                     (progn
+                        ;; If a file but not running the queue,
+                        ;; offer to delete it
+                       (setq also-file (expand-file-name also-file))
+                       (when (or feedmail-queue-auto-file-nuke
+                                  (y-or-n-p
+                                   (format "FQM: Delete message file %s? "
+                                           also-file)))
+                          ;; If we delete the affiliated file, get rid
+                          ;; of the file name association and make sure we
+                          ;; don't annoy people with a prompt on exit.
+                          (delete-file also-file)
+                          (with-current-buffer feedmail-raw-text-buffer
+                            (setq buffer-offer-save nil)
+                            (setq buffer-file-name nil)))))
                  (goto-char (point-min))
-                 (while (and (re-search-forward "\n\n\n*" eoh-marker t) (< 
(point) eoh-marker))
-                       (replace-match "\n"))
-       
-                 (let ((case-fold-search t) (addr-regexp))
-                       (goto-char (point-min))
-                       ;; there are some RFC-822 combinations/cases missed 
here,
-                       ;; but probably good enough and what users expect
-                       ;;
-                       ;; use resent-* stuff only if there is at least one 
non-empty one
-                       (setq feedmail-is-a-resend
-                                 (re-search-forward
-                                  ;; header name, followed by optional 
whitespace, followed by
-                                  ;; non-whitespace, followed by anything, 
followed by newline;
-                                  ;; the idea is empty RESENT-* headers are 
ignored
-                                  
"^\\(RESENT-TO:\\|RESENT-CC:\\|RESENT-BCC:\\)\\s-*\\S-+.*$"
-                                  eoh-marker t))
-                       ;; if we say so, gather the BCC stuff before the main 
course
-                       (if (eq feedmail-deduce-bcc-where 'first)
-                               (progn (if feedmail-is-a-resend (setq 
addr-regexp a-re-rb) (setq addr-regexp a-re-db))
-                                          (setq feedmail-address-list 
(feedmail-deduce-address-list feedmail-prepped-text-buffer (point-min) 
eoh-marker addr-regexp feedmail-address-list))))
-                       ;; the main course
-                       (if (or (eq feedmail-deduce-bcc-where 'first) (eq 
feedmail-deduce-bcc-where 'last))
-                               ;; handled by first or last cases, so don't get 
BCC stuff
-                               (progn (if feedmail-is-a-resend (setq 
addr-regexp a-re-rtc) (setq addr-regexp a-re-dtc))
-                                          (setq feedmail-address-list 
(feedmail-deduce-address-list feedmail-prepped-text-buffer (point-min) 
eoh-marker addr-regexp feedmail-address-list)))
-                         ;; not handled by first or last cases, so also get 
BCC stuff
-                         (progn (if feedmail-is-a-resend (setq addr-regexp 
a-re-rtcb) (setq addr-regexp a-re-dtcb))
-                                        (setq feedmail-address-list 
(feedmail-deduce-address-list feedmail-prepped-text-buffer (point-min) 
eoh-marker addr-regexp feedmail-address-list))))
-                       ;; if we say so, gather the BCC stuff after the main 
course
-                       (if (eq feedmail-deduce-bcc-where 'last)
-                               (progn (if feedmail-is-a-resend (setq 
addr-regexp a-re-rb) (setq addr-regexp a-re-db))
-                                          (setq feedmail-address-list 
(feedmail-deduce-address-list feedmail-prepped-text-buffer (point-min) 
eoh-marker addr-regexp feedmail-address-list))))
-                       (if (not feedmail-address-list) (error "FQM: 
Sending...abandoned, no addressees"))
-                       ;; not needed, but meets user expectations
-                       (setq feedmail-address-list (nreverse 
feedmail-address-list))
-                       ;; Find and handle any BCC fields.
-                       (setq bcc-holder (feedmail-accume-n-nuke-header 
eoh-marker "^BCC:"))
-                       (setq resent-bcc-holder (feedmail-accume-n-nuke-header 
eoh-marker "^RESENT-BCC:"))
-                       (if (and bcc-holder (not feedmail-nuke-bcc))
-                               (progn (goto-char (point-min))
-                                          (insert bcc-holder)))
-                       (if (and resent-bcc-holder (not 
feedmail-nuke-resent-bcc))
-                               (progn (goto-char (point-min))
-                                          (insert resent-bcc-holder)))
-                       (goto-char (point-min))
-
-                       ;; fiddle about, fiddle about, fiddle about....
-                       (feedmail-fiddle-from)
-                       (feedmail-fiddle-sender)
-                       (feedmail-fiddle-x-mailer)
-                       (feedmail-fiddle-message-id
-                        (or feedmail-queue-runner-is-active (buffer-file-name 
feedmail-raw-text-buffer)))
-                       (feedmail-fiddle-date
-                        (or feedmail-queue-runner-is-active (buffer-file-name 
feedmail-raw-text-buffer)))
-                       (feedmail-fiddle-list-of-fiddle-plexes 
feedmail-fiddle-plex-user-list)
-
-                       ;; don't send out a blank headers of various sorts
-                       ;; (this loses on continued line with a blank first 
line)
-                       (goto-char (point-min))
-                       (and feedmail-nuke-empty-headers ; hey, who's an 
empty-header?
-                                (while (re-search-forward "^[A-Za-z0-9-]+:[ 
\t]*\n" eoh-marker t)
-                                  (replace-match ""))))
-
-                 (run-hooks 'feedmail-last-chance-hook)
-
-                 (let ((fcc (feedmail-accume-n-nuke-header eoh-marker "^FCC:"))
-                               (also-file)
-                               (confirm (cond
-                                                 ((eq 
feedmail-confirm-outgoing 'immediate)
-                                                  (not 
feedmail-queue-runner-is-active))
-                                                 ((eq 
feedmail-confirm-outgoing 'queued) feedmail-queue-runner-is-active)
-                                                 (t 
feedmail-confirm-outgoing))))
-                       (if (or (not confirm) (feedmail-one-last-look 
feedmail-prepped-text-buffer))
-                               (let ((user-mail-address 
(feedmail-envelope-deducer eoh-marker)))
-                                 (feedmail-give-it-to-buffer-eater)
-                                 (if (and (not 
feedmail-queue-runner-is-active) (setq also-file (buffer-file-name 
feedmail-raw-text-buffer)))
-                                         (progn                        ; if a 
file but not running the queue, offer to delete it
-                                               (setq also-file 
(expand-file-name also-file))
-                                               (if (or 
feedmail-queue-auto-file-nuke
-                                                               (y-or-n-p 
(format "FQM: Delete message file %s? " also-file)))
-                                                       (save-excursion
-                                                         ;; if we delete the 
affiliated file, get rid
-                                                         ;; of the file name 
association and make sure we 
-                                                         ;; don't annoy people 
with a prompt on exit
-                                                         (delete-file 
also-file)
-                                                         (set-buffer 
feedmail-raw-text-buffer)
-                                                         (setq 
buffer-offer-save nil)
-                                                         (setq 
buffer-file-name nil)
-                                                         )
-                                                 )))
-                                 (goto-char (point-min))
-                                 ;; re-insert and handle any FCC fields (and, 
optionally, any BCC).
-                                 (if fcc (let ((default-buffer-file-type 
feedmail-force-binary-write))
-                                                       (insert fcc)
-                                                       (if (not 
feedmail-nuke-bcc-in-fcc)
-                                                               (progn (if 
bcc-holder (insert bcc-holder))
-                                                                          (if 
resent-bcc-holder (insert resent-bcc-holder))))
-
-                                                       (run-hooks 
'feedmail-before-fcc-hook)
-
-                                                       (if 
feedmail-nuke-body-in-fcc
-                                                               (progn 
(goto-char eoh-marker)
-                                                                          (if 
(natnump feedmail-nuke-body-in-fcc)
-                                                                               
   (forward-line feedmail-nuke-body-in-fcc))
-                                                                          
(delete-region (point) (point-max))
-                                                                          ))
-                                                       (mail-do-fcc eoh-marker)
-                                                       )))
-                         (error "FQM: Sending...abandoned") ; user bailed out 
of one-last-look
-                         )))                                           ; 
unwind-protect body (save-excursion)
-
-         ;; unwind-protect cleanup forms
-         (kill-buffer feedmail-prepped-text-buffer)
-         (set-buffer feedmail-error-buffer)
-         (if (zerop (buffer-size)) (kill-buffer feedmail-error-buffer)
-               (progn (display-buffer feedmail-error-buffer)
-                          ;; read fast ... the meter is running
-                          (if (and feedmail-queue-runner-is-active 
feedmail-queue-chatty)
-                                  (progn (message "FQM: Sending...failed") 
(ding t) (sit-for 3)))
-                          (error "FQM: Sending...failed")))
-         (set-buffer feedmail-raw-text-buffer))
-       )                                                                       
; let
-  (if (and feedmail-queue-chatty (not feedmail-queue-runner-is-active))
-         (progn
-               (feedmail-queue-reminder 'after-immediate)
-               (sit-for feedmail-queue-chatty-sit-for)))
-  )
+                 ;; Re-insert and handle any Fcc fields (and, optionally,
+                  ;; any Bcc).
+                 (when fcc
+                    (let ((coding-system-for-write
+                          (if (and (memq system-type '(ms-dos windows-nt))
+                                   feedmail-force-binary-write)
+                              'no-conversion
+                            coding-system-for-write)))
+                      (unwind-protect
+                          (progn
+                            (insert fcc)
+                            (unless feedmail-nuke-bcc-in-fcc
+                              (if bcc-holder (insert bcc-holder))
+                              (if resent-bcc-holder
+                                  (insert resent-bcc-holder)))
+
+                            (run-hooks 'feedmail-before-fcc-hook)
+
+                            (when feedmail-nuke-body-in-fcc
+                              (goto-char eoh-marker)
+                              (if (natnump feedmail-nuke-body-in-fcc)
+                                  (forward-line feedmail-nuke-body-in-fcc))
+                              (delete-region (point) (point-max)))
+                            (mail-do-fcc eoh-marker))))))
+             ;; User bailed out of one-last-look.
+             (if feedmail-queue-runner-is-active
+                 (throw 'skip-me-q 'skip-me-q)
+               (throw 'skip-me-i 'skip-me-i))
+             )))) ; unwind-protect body (save-excursion)
+
+      ;; unwind-protect cleanup forms.
+      (kill-buffer feedmail-prepped-text-buffer)
+      (set-buffer feedmail-error-buffer)
+      (if (zerop (buffer-size)) (kill-buffer feedmail-error-buffer)
+        (display-buffer feedmail-error-buffer)
+        ;; Read fast ... the meter is running.
+        (if feedmail-queue-runner-is-active
+            (progn
+              (ding t)
+              (feedmail-say-chatter "Sending...failed")))
+        (error "FQM: Sending...failed"))
+      (set-buffer feedmail-raw-text-buffer))
+    )                                  ; let
+  (when (and feedmail-queue-chatty (not feedmail-queue-runner-is-active))
+    (feedmail-queue-reminder 'after-immediate)
+    (sit-for feedmail-queue-chatty-sit-for)))
 
 
 (defun feedmail-fiddle-header (name value &optional action folding)
   "Internal feedmail function for jamming fields into message header.
 NAME, VALUE, ACTION, and FOLDING are the four elements of a
 fiddle-plex, as described in the documentation for the variable
-feedmail-fiddle-plex-blurb."
+`feedmail-fiddle-plex-blurb'."
+  (feedmail-say-debug ">in-> feedmail-fiddle-header %s %s %s %s"
+                     name value action folding)
   (let ((case-fold-search t)
-               (header-colon (concat (regexp-quote name) ":"))
-               header-regexp eoh-marker has-like ag-like val-like that-point)
-       (setq header-regexp (concat "^" header-colon))
-       (setq eoh-marker (feedmail-find-eoh))
-       (goto-char (point-min))
-       (setq has-like (re-search-forward header-regexp eoh-marker t))
-
-       (if (not action) (setq action 'supplement))
-       (cond
-        ((eq action 'supplement)
-         ;; trim leading/trailing whitespace
-         (if (string-match "\\`[ \t\n]+" value)
-                 (setq value (substring value (match-end 0))))
-         (if (string-match "[ \t\n]+\\'" value)
-                 (setq value (substring value 0 (match-beginning 0))))
-         (if (> (length value) 0)
-                 (progn
-                       (if feedmail-fiddle-headers-upwardly
-                               (goto-char (point-min))
-                         (goto-char eoh-marker))
-                       (setq that-point (point))
-                       (insert name ": " value "\n")
-                       (if folding (feedmail-fill-this-one that-point 
(point))))))
-
-        ((eq action 'replace)
-         (if has-like (feedmail-accume-n-nuke-header eoh-marker header-regexp))
-         (feedmail-fiddle-header name value 'supplement folding))
-
-        ((eq action 'create)
-         (if (not has-like) (feedmail-fiddle-header name value 'supplement 
folding)))
-
-        ((eq action 'combine)
-         (setq val-like (nth 1 value))
-         (setq ag-like (or (feedmail-accume-n-nuke-header eoh-marker 
header-regexp) ""))
-         ;; get rid of initial header name from first instance (front of 
string)
-         (if (string-match (concat header-regexp "[ \t\n]+") ag-like)
-                 (setq ag-like (replace-match "" t t ag-like)))
-         ;; get rid of embedded header names from subsequent instances
-         (while (string-match (concat "\n" header-colon "[ \t\n]+") ag-like)
-               (setq ag-like (replace-match "\n\t" t t ag-like)))
-         ;; trim leading/trailing whitespace
-         (if (string-match "\\`[ \t\n]+" ag-like)
-                 (setq ag-like (substring ag-like (match-end 0))))
-         (if (string-match "[ \t\n]+\\'" ag-like)
-                 (setq ag-like (substring ag-like 0 (match-beginning 0))))
-         ;; if ag-like is not nil and not an empty string, transform it via a 
function
-         ;; call or format operation
-         (if (> (length ag-like) 0)
-                 (setq ag-like
-                               (cond
-                                ((and (symbolp val-like) (fboundp val-like))
-                                 (funcall val-like name ag-like))
-                                ((stringp val-like)
-                                 (format val-like ag-like))
-                                (t nil))))
-         (feedmail-fiddle-header name (concat (nth 0 value) ag-like (nth 2 
value)) 'supplement folding)))
-       ))
+       (header-colon (concat (regexp-quote name) ":"))
+       header-regexp eoh-marker has-like ag-like val-like that-point)
+    (setq header-regexp (concat "^" header-colon))
+    (setq eoh-marker (feedmail-find-eoh))
+    (goto-char (point-min))
+    (setq has-like (re-search-forward header-regexp eoh-marker t))
+
+    (if (not action) (setq action 'supplement))
+    (cond
+     ((eq action 'supplement)
+      ;; trim leading/trailing whitespace
+      (if (string-match "\\`[ \t\n]+" value)
+         (setq value (substring value (match-end 0))))
+      (if (string-match "[ \t\n]+\\'" value)
+         (setq value (substring value 0 (match-beginning 0))))
+      (if (> (length value) 0)
+         (progn
+           (if feedmail-fiddle-headers-upwardly
+               (goto-char (point-min))
+             (goto-char eoh-marker))
+           (setq that-point (point))
+           (insert name ": " value "\n")
+           (if folding (feedmail-fill-this-one that-point (point))))))
+
+     ((eq action 'replace)
+      (if has-like (feedmail-accume-n-nuke-header eoh-marker header-regexp))
+      (feedmail-fiddle-header name value 'supplement folding))
+
+     ((eq action 'create)
+      (if (not has-like) (feedmail-fiddle-header name value 'supplement 
folding)))
+
+     ((eq action 'combine)
+      (setq val-like (nth 1 value))
+      (setq ag-like (or (feedmail-accume-n-nuke-header eoh-marker 
header-regexp) ""))
+      ;; get rid of initial header name from first instance (front of string)
+      (if (string-match (concat header-regexp "[ \t\n]+") ag-like)
+         (setq ag-like (replace-match "" t t ag-like)))
+      ;; get rid of embedded header names from subsequent instances
+      (while (string-match (concat "\n" header-colon "[ \t\n]+") ag-like)
+       (setq ag-like (replace-match "\n\t" t t ag-like)))
+      ;; trim leading/trailing whitespace
+      (if (string-match "\\`[ \t\n]+" ag-like)
+         (setq ag-like (substring ag-like (match-end 0))))
+      (if (string-match "[ \t\n]+\\'" ag-like)
+         (setq ag-like (substring ag-like 0 (match-beginning 0))))
+      ;; if ag-like is not nil and not an empty string, transform it via a 
function
+      ;; call or format operation
+      (if (> (length ag-like) 0)
+         (setq ag-like
+               (cond
+                ((and (symbolp val-like) (fboundp val-like))
+                 (funcall val-like name ag-like))
+                ((stringp val-like)
+                 (format val-like ag-like))
+                (t nil))))
+      (feedmail-fiddle-header name (concat (nth 0 value) ag-like (nth 2 
value)) 'supplement folding)))
+    ))
 
 (defun feedmail-give-it-to-buffer-eater ()
+  (feedmail-say-debug ">in-> feedmail-give-it-to-buffer-eater")
   (save-excursion
-       (if feedmail-enable-spray
-               (mapcar
-                '(lambda (feedmail-spray-this-address)
-                       (let ((spray-buffer (get-buffer-create " *FQM Outgoing 
Email Spray*")))
-                         (save-excursion 
-                               (set-buffer spray-buffer)
-                               (erase-buffer)
-                               ;; not life's most efficient methodology, but 
spraying isn't
-                               ;; an every-5-minutes event either
-                               (insert-buffer feedmail-prepped-text-buffer)
-                               ;; There's a good case to me made that each 
separate transmission of
-                               ;; a message in the spray should have a 
distinct MESSAGE-ID:.  There
-                               ;; is also a less compelling argument in the 
other direction.  I think
-                               ;; they technically should have distinct 
MESSAGE-ID:s, but I doubt that
-                               ;; anyone cares, practically.  If someone 
complains about it, I'll add
-                               ;; it.
-                               (feedmail-fiddle-list-of-spray-fiddle-plexes 
feedmail-spray-address-fiddle-plex-list)
-                               ;; this (let ) is just in case some buffer eater
-                               ;; is cheating and using the global variable 
name instead
-                               ;; of its argument to find the buffer
-                               (let ((feedmail-prepped-text-buffer 
spray-buffer))
-                                 (funcall feedmail-buffer-eating-function
-                                                  feedmail-prepped-text-buffer
-                                                  feedmail-error-buffer
-                                                  (list 
feedmail-spray-this-address))))
-                         (kill-buffer spray-buffer)
-                         ))
-                feedmail-address-list)
-         (funcall feedmail-buffer-eating-function
+    (if feedmail-enable-spray
+       (mapcar
+        (lambda (address)
+          (let ((feedmail-spray-this-address address)
+                 (spray-buffer
+                  (get-buffer-create " *FQM Outgoing Email Spray*")))
+             (with-current-buffer spray-buffer
+               (erase-buffer)
+               ;; not life's most efficient methodology, but spraying isn't
+               ;; an every-5-minutes event either
+               (insert-buffer-substring feedmail-prepped-text-buffer)
+               ;; There's a good case to me made that each separate
+               ;; transmission of a message in the spray should
+               ;; have a distinct Message-Id:.  There is also a less
+               ;; compelling argument in the other direction.
+               ;; I think they technically should have distinct
+               ;; Message-Id:s, but I doubt that anyone cares,
+               ;; practically.  If someone complains about it, I'll
+               ;; add it.
+               (feedmail-fiddle-list-of-spray-fiddle-plexes
+                 feedmail-spray-address-fiddle-plex-list)
+               ;; this (let ) is just in case some buffer eater
+               ;; is cheating and using the global variable name instead
+               ;; of its argument to find the buffer
+               (let ((feedmail-prepped-text-buffer spray-buffer))
+                 (funcall feedmail-buffer-eating-function
                           feedmail-prepped-text-buffer
                           feedmail-error-buffer
-                          feedmail-address-list))))
+                          (list feedmail-spray-this-address))))
+             (kill-buffer spray-buffer)
+             ))
+        feedmail-address-list)
+      (feedmail-say-debug "calling buffer-eater %s"
+                         feedmail-buffer-eating-function)
+      (funcall feedmail-buffer-eating-function
+              feedmail-prepped-text-buffer
+              feedmail-error-buffer
+              feedmail-address-list))))
 
 
 (defun feedmail-envelope-deducer (eoh-marker)
-  "If feedmail-deduce-envelope-from is false, simply return
-user-mail-address.  Else, look for SENDER: or FROM: (or RESENT-*) and
+  "If `feedmail-deduce-envelope-from' is false, simply return 
`user-mail-address'.
+Else, look for Sender: or From: (or Resent-*) and
 return that value."
+  (feedmail-say-debug ">in-> feedmail-envelope-deducer %s" eoh-marker)
   (if (not feedmail-deduce-envelope-from)
-         user-mail-address
-       (let ((from-list))
+      user-mail-address
+    (let ((from-list))
+      (setq from-list
+           (feedmail-deduce-address-list
+            (current-buffer) (point-min) eoh-marker (if feedmail-is-a-resend 
"^Resent-Sender:" "^Sender:")
+            from-list))
+      (if (not from-list)
          (setq from-list
-                       (feedmail-deduce-address-list
-                        (current-buffer) (point-min) eoh-marker (if 
feedmail-is-a-resend "^RESENT-SENDER:" "^SENDER:")
-                        from-list))
-         (if (not from-list)
-                 (setq from-list
-                               (feedmail-deduce-address-list
-                                (current-buffer) (point-min) eoh-marker (if 
feedmail-is-a-resend "^RESENT-FROM:" "^FROM:")
-                                from-list)))
-         (if (and from-list (car from-list)) (car from-list) 
user-mail-address))))
+               (feedmail-deduce-address-list
+                (current-buffer) (point-min) eoh-marker (if 
feedmail-is-a-resend "^Resent-From:" "^From:")
+                from-list)))
+      (if (and from-list (car from-list)) (car from-list) user-mail-address))))
 
 
 (defun feedmail-fiddle-from ()
-  "Fiddle FROM:."
+  "Fiddle From:."
+  (feedmail-say-debug ">in-> feedmail-fiddle-from")
   ;; default is to fall off the end of the list and do nothing
   (cond
    ;; nil means do nothing
    ((eq nil feedmail-from-line) nil)
-   ;; t is the same a using the default computation, so compute it and recurse
+   ;; t is the same as using the default computation, so compute it and recurse
    ;; user-full-name suggested by address@hidden (=Kimball Collins)
    ;; improvement using user-mail-address suggested by
    ;;   address@hidden (Douglas Gray Stephens)
    ((eq t feedmail-from-line)
-       (let ((feedmail-from-line 
-                  (let ((at-stuff
-                                 (if user-mail-address user-mail-address 
(concat (user-login-name) "@" (system-name)))))
-                        (cond
-                         ((eq mail-from-style nil) at-stuff)
-                         ((eq mail-from-style 'parens) (concat at-stuff " (" 
(user-full-name) ")"))
-                         ((eq mail-from-style 'angles) (concat "\"" 
(user-full-name) "\" <" at-stuff ">"))
-                         ))))
-                  (feedmail-fiddle-from)))
+    (let ((feedmail-from-line
+          (cond
+           ((eq mail-from-style nil) user-mail-address)
+           ((eq mail-from-style 'parens) (concat user-mail-address " (" 
(user-full-name) ")"))
+           ((eq mail-from-style 'angles) (concat "\"" (user-full-name) "\" <" 
user-mail-address ">"))
+           )))
+      (feedmail-fiddle-from)))
 
    ;; if it's a string, simply make a fiddle-plex out of it and recurse
    ((stringp feedmail-from-line)
-       (let ((feedmail-from-line (list "ignored" feedmail-from-line 'create)))
-         (feedmail-fiddle-from)))
+    (let ((feedmail-from-line (list "ignored" feedmail-from-line 'create)))
+      (feedmail-fiddle-from)))
 
    ;; if it's a function, call it and recurse with the resulting value
    ((and (symbolp feedmail-from-line) (fboundp feedmail-from-line))
-       (let ((feedmail-from-line (funcall feedmail-from-line)))
-         (feedmail-fiddle-from)))
+    (let ((feedmail-from-line (funcall feedmail-from-line)))
+      (feedmail-fiddle-from)))
 
    ;; if it's a list, it must be a fiddle-plex -- so fiddle, man, fiddle
    ((listp feedmail-from-line)
-       (feedmail-fiddle-header
-        (if feedmail-is-a-resend "Resent-From" "From")
-        (nth 1 feedmail-from-line)      ;; value
-        (nth 2 feedmail-from-line)      ;; action
-        (nth 3 feedmail-from-line)))))  ;; folding
+    (feedmail-fiddle-header
+     (if feedmail-is-a-resend "Resent-From" "From")
+     (nth 1 feedmail-from-line)                ; value
+     (nth 2 feedmail-from-line)                ; action
+     (nth 3 feedmail-from-line)))))    ; folding
 
 
 (defun feedmail-fiddle-sender ()
-  "Fiddle SENDER:."
+  "Fiddle Sender:."
+  (feedmail-say-debug ">in-> feedmail-fiddle-sender")
   ;; default is to fall off the end of the list and do nothing
   (cond
    ;; nil means do nothing
@@ -2294,114 +2795,126 @@ feedmail-fiddle-sender
 
    ;; if it's a string, simply make a fiddle-plex out of it and recurse
    ((stringp feedmail-sender-line)
-       (let ((feedmail-sender-line (list "ignored" feedmail-sender-line 
'create)))
-         (feedmail-fiddle-sender)))
+    (let ((feedmail-sender-line (list "ignored" feedmail-sender-line 'create)))
+      (feedmail-fiddle-sender)))
 
    ;; if it's a function, call it and recurse with the resulting value
    ((and (symbolp feedmail-sender-line) (fboundp feedmail-sender-line))
-       (let ((feedmail-sender-line (funcall feedmail-sender-line)))
-         (feedmail-fiddle-sender)))
+    (let ((feedmail-sender-line (funcall feedmail-sender-line)))
+      (feedmail-fiddle-sender)))
 
    ;; if it's a list, it must be a fiddle-plex -- so fiddle, man, fiddle
    ((listp feedmail-sender-line)
-       (feedmail-fiddle-header
-        (if feedmail-is-a-resend "Resent-Sender" "Sender")
-        (nth 1 feedmail-sender-line)      ;; value
-        (nth 2 feedmail-sender-line)      ;; action
-        (nth 3 feedmail-sender-line)))))  ;; folding
+    (feedmail-fiddle-header
+     (if feedmail-is-a-resend "Resent-Sender" "Sender")
+     (nth 1 feedmail-sender-line)      ; value
+     (nth 2 feedmail-sender-line)      ; action
+     (nth 3 feedmail-sender-line)))))  ; folding
 
 
 (defun feedmail-default-date-generator (maybe-file)
-  "Default function for generating DATE: header contents."
+  "Default function for generating Date: header contents."
+  (feedmail-say-debug ">in-> feedmail-default-date-generator")
+  (when maybe-file
+    (feedmail-say-debug (concat "4 cre " (feedmail-rfc822-date (nth 4 
(file-attributes maybe-file)))))
+    (feedmail-say-debug (concat "5 mod " (feedmail-rfc822-date (nth 5 
(file-attributes maybe-file)))))
+    (feedmail-say-debug (concat "6 sta " (feedmail-rfc822-date (nth 6 
(file-attributes maybe-file))))))
   (let ((date-time))
-       (if (and (not feedmail-queue-use-send-time-for-date) maybe-file)
-               (setq date-time (nth 5 (file-attributes maybe-file))))
-       (feedmail-rfc822-date date-time))
+    (if (and (not feedmail-queue-use-send-time-for-date) maybe-file)
+       (setq date-time (nth 5 (file-attributes maybe-file))))
+    (feedmail-rfc822-date date-time))
   )
 
 
 (defun feedmail-fiddle-date (maybe-file)
-  "Fiddle DATE:.  See documentation of feedmail-date-generator."
+  "Fiddle Date:.  See documentation of `feedmail-date-generator'."
+  (feedmail-say-debug ">in-> feedmail-fiddle-date")
   ;; default is to fall off the end of the list and do nothing
   (cond
    ;; nil means do nothing
    ((eq nil feedmail-date-generator) nil)
    ;; t is the same a using the function feedmail-default-date-generator, so 
let it and recurse
    ((eq t feedmail-date-generator)
-       (let ((feedmail-date-generator (feedmail-default-date-generator 
maybe-file)))
-         (feedmail-fiddle-date maybe-file)))
+    (let ((feedmail-date-generator (feedmail-default-date-generator 
maybe-file)))
+      (feedmail-fiddle-date maybe-file)))
 
    ;; if it's a string, simply make a fiddle-plex out of it and recurse
    ((stringp feedmail-date-generator)
-       (let ((feedmail-date-generator (list "ignored" feedmail-date-generator 
'create)))
-         (feedmail-fiddle-date maybe-file)))
+    (let ((feedmail-date-generator (list "ignored" feedmail-date-generator 
'create)))
+      (feedmail-fiddle-date maybe-file)))
 
    ;; if it's a function, call it and recurse with the resulting value
    ((and (symbolp feedmail-date-generator) (fboundp feedmail-date-generator))
-       (let ((feedmail-date-generator (funcall feedmail-date-generator 
maybe-file)))
-         (feedmail-fiddle-date maybe-file)))
+    (let ((feedmail-date-generator (funcall feedmail-date-generator 
maybe-file)))
+      (feedmail-fiddle-date maybe-file)))
 
    ;; if it's a list, it must be a fiddle-plex -- so fiddle, man, fiddle
    ((listp feedmail-date-generator)
-       (feedmail-fiddle-header
-        (if feedmail-is-a-resend "Resent-Date" "Date")
-        (nth 1 feedmail-date-generator)      ;; value
-        (nth 2 feedmail-date-generator)      ;; action
-        (nth 3 feedmail-date-generator)))))  ;; folding
+    (feedmail-fiddle-header
+     (if feedmail-is-a-resend "Resent-Date" "Date")
+     (nth 1 feedmail-date-generator)   ; value
+     (nth 2 feedmail-date-generator)   ; action
+     (nth 3 feedmail-date-generator))))) ; folding
 
 
 (defun feedmail-default-message-id-generator (maybe-file)
-  "Default function for generating MESSAGE-ID: header contents.
+  "Default function for generating Message-Id: header contents.
 Based on a date and a sort of random number for tie breaking.  Unless
-feedmail-message-id-suffix is defined, uses user-mail-address, so be
-sure it's set."
+`feedmail-message-id-suffix' is defined, uses `user-mail-address', so be
+sure it's set.  If both are nil, creates a quasi-random suffix that is
+probably not appropriate for you."
+  (feedmail-say-debug ">in-> feedmail-default-message-id-generator %s"
+                     maybe-file)
   (let ((date-time)
-               (end-stuff (if feedmail-message-id-suffix 
feedmail-message-id-suffix user-mail-address)))
-       (if (string-match "^\\(.*\\)@" end-stuff)
-               (setq end-stuff
-                         (concat (if (equal (match-beginning 1) (match-end 1)) 
"" "-") end-stuff))
-         (setq end-stuff (concat "@" end-stuff)))
-       (if (and (not feedmail-queue-use-send-time-for-message-id) maybe-file)
-               (setq date-time (nth 5 (file-attributes maybe-file))))
-       (format "<%d-%s%s%s>"
-                       (mod (random) 10000)
-                       (format-time-string "%a%d%b%Y%H%M%S" date-time)
-                       (feedmail-rfc822-time-zone date-time)
-                       end-stuff))
+       (system-time-locale "C")
+       (end-stuff (if feedmail-message-id-suffix feedmail-message-id-suffix 
user-mail-address)))
+    (if (not end-stuff) (setq end-stuff (format "%d.example.com" (random))))
+    (if (string-match "^\\(.*\\)@" end-stuff)
+       (setq end-stuff
+             (concat (if (equal (match-beginning 1) (match-end 1)) "" "-") 
end-stuff))
+      (setq end-stuff (concat "@" end-stuff)))
+    (if (and (not feedmail-queue-use-send-time-for-message-id) maybe-file)
+       (setq date-time (nth 5 (file-attributes maybe-file))))
+    (format "<%d-%s%s%s>"
+           (mod (random) 10000)
+           (format-time-string "%a%d%b%Y%H%M%S" date-time)
+           (feedmail-rfc822-time-zone date-time)
+           end-stuff))
   )
 
 (defun feedmail-fiddle-message-id (maybe-file)
-  "Fiddle MESSAGE-ID:.  See documentation of feedmail-message-id-generator."
+  "Fiddle Message-Id:.  See documentation of `feedmail-message-id-generator'."
+  (feedmail-say-debug ">in-> feedmail-fiddle-message-id %s" maybe-file)
   ;; default is to fall off the end of the list and do nothing
   (cond
    ;; nil means do nothing
    ((eq nil feedmail-message-id-generator) nil)
    ;; t is the same a using the function 
feedmail-default-message-id-generator, so let it and recurse
    ((eq t feedmail-message-id-generator)
-       (let ((feedmail-message-id-generator 
(feedmail-default-message-id-generator maybe-file)))
-         (feedmail-fiddle-message-id maybe-file)))
+    (let ((feedmail-message-id-generator 
(feedmail-default-message-id-generator maybe-file)))
+      (feedmail-fiddle-message-id maybe-file)))
 
    ;; if it's a string, simply make a fiddle-plex out of it and recurse
    ((stringp feedmail-message-id-generator)
-       (let ((feedmail-message-id-generator (list "ignored" 
feedmail-message-id-generator 'create)))
-         (feedmail-fiddle-message-id maybe-file)))
+    (let ((feedmail-message-id-generator (list "ignored" 
feedmail-message-id-generator 'create)))
+      (feedmail-fiddle-message-id maybe-file)))
 
    ;; if it's a function, call it and recurse with the resulting value
    ((and (symbolp feedmail-message-id-generator) (fboundp 
feedmail-message-id-generator))
-       (let ((feedmail-message-id-generator (funcall 
feedmail-message-id-generator maybe-file)))
-         (feedmail-fiddle-message-id maybe-file)))
+    (let ((feedmail-message-id-generator (funcall 
feedmail-message-id-generator maybe-file)))
+      (feedmail-fiddle-message-id maybe-file)))
 
    ;; if it's a list, it must be a fiddle-plex -- so fiddle, man, fiddle
    ((listp feedmail-message-id-generator)
-       (feedmail-fiddle-header
-        (if feedmail-is-a-resend "Resent-Message-ID" "Message-ID")
-        (nth 1 feedmail-message-id-generator)      ;; value
-        (nth 2 feedmail-message-id-generator)      ;; action
-        (nth 3 feedmail-message-id-generator)))))  ;; folding
+    (feedmail-fiddle-header
+     (if feedmail-is-a-resend "Resent-Message-Id" "Message-Id")
+     (nth 1 feedmail-message-id-generator) ; value
+     (nth 2 feedmail-message-id-generator) ; action
+     (nth 3 feedmail-message-id-generator))))) ; folding
 
 
 (defun feedmail-default-x-mailer-generator ()
-  "Default function for generating X-MAILER: header contents."
+  "Default function for generating X-Mailer: header contents."
   (concat
    (let ((case-fold-search t)) (if (string-match "emacs" emacs-version) "" 
"emacs "))
    emacs-version " (via feedmail " feedmail-patch-level
@@ -2412,96 +2925,102 @@ feedmail-default-x-mailer-generator
 
 
 (defun feedmail-fiddle-x-mailer ()
-  "Fiddle X-MAILER:.  See documentation of feedmail-x-mailer-line."
+  "Fiddle X-Mailer:.  See documentation of `feedmail-x-mailer-line'."
+  (feedmail-say-debug ">in-> feedmail-fiddle-x-mailer")
   ;; default is to fall off the end of the list and do nothing
   (cond
+   ;; nil means do nothing
+   ((eq nil feedmail-x-mailer-line) nil)
    ;; t is the same a using the function feedmail-default-x-mailer-generator, 
so let it and recurse
    ((eq t feedmail-x-mailer-line)
-       (let ((feedmail-x-mailer-line (feedmail-default-x-mailer-generator)))
-         (feedmail-fiddle-x-mailer)))
+    (let ((feedmail-x-mailer-line (feedmail-default-x-mailer-generator)))
+      (feedmail-fiddle-x-mailer)))
 
    ;; if it's a string, simply make a fiddle-plex out of it and recurse
    ((stringp feedmail-x-mailer-line)
-       (let ((feedmail-x-mailer-line (list "ignored" (list 
feedmail-x-mailer-line ";\n\t%s") 'combine)))
-         (feedmail-fiddle-x-mailer)))
+    (let ((feedmail-x-mailer-line (list "ignored" (list feedmail-x-mailer-line 
";\n\t%s") 'combine)))
+      (feedmail-fiddle-x-mailer)))
 
    ;; if it's a function, call it and recurse with the resulting value
    ((and (symbolp feedmail-x-mailer-line) (fboundp feedmail-x-mailer-line))
-       (let ((feedmail-x-mailer-line (funcall feedmail-x-mailer-line)))
-         (feedmail-fiddle-x-mailer)))
+    (let ((feedmail-x-mailer-line (funcall feedmail-x-mailer-line)))
+      (feedmail-fiddle-x-mailer)))
 
    ;; if it's a list, it must be a fiddle-plex -- so fiddle, man, fiddle
    ((listp feedmail-x-mailer-line)
-       (feedmail-fiddle-header
-        (if feedmail-is-a-resend "X-Resent-Mailer" "X-Mailer")
-        (nth 1 feedmail-x-mailer-line)      ;; value
-        (nth 2 feedmail-x-mailer-line)      ;; action
-        (nth 3 feedmail-x-mailer-line)))))  ;; folding
+    (feedmail-fiddle-header
+     (if feedmail-is-a-resend "X-Resent-Mailer" "X-Mailer")
+     (nth 1 feedmail-x-mailer-line)    ; value
+     (nth 2 feedmail-x-mailer-line)    ; action
+     (nth 3 feedmail-x-mailer-line)))))        ; folding
 
 
 (defun feedmail-fiddle-spray-address (addy-plex)
-  "Fiddle header for single spray address.  Uses feedmail-spray-this-address."
+  "Fiddle header for single spray address.  Uses 
`feedmail-spray-this-address'."
+  (feedmail-say-debug ">in-> feedmail-fiddle-spray-address %s" addy-plex)
   ;; default is to fall off the end of the list and do nothing
   (cond
    ;; nil means do nothing
    ((eq nil addy-plex) nil)
-   ;; t means the same as using "TO: and unembellished addy
+   ;; t means the same as using "To:" and unembellished addy
    ((eq t addy-plex)
-       (let ((addy-plex (list "To" feedmail-spray-this-address)))
-         (feedmail-fiddle-spray-address addy-plex)))
+    (let ((addy-plex (list "To" feedmail-spray-this-address)))
+      (feedmail-fiddle-spray-address addy-plex)))
 
    ;; if it's a string, simply make a fiddle-plex out of it and recurse, 
assuming
-   ;; the string names a header field (e.g., "TO")
+   ;; the string names a header field (e.g., "To")
    ((stringp addy-plex)
-       (let ((addy-plex (list addy-plex feedmail-spray-this-address)))
-         (feedmail-fiddle-spray-address addy-plex)))
+    (let ((addy-plex (list addy-plex feedmail-spray-this-address)))
+      (feedmail-fiddle-spray-address addy-plex)))
 
    ;; if it's a function, call it and recurse with the resulting value
    ((and (symbolp addy-plex) (fboundp addy-plex))
-       (let ((addy-plex (funcall addy-plex)))
-         (feedmail-fiddle-spray-address addy-plex)))
+    (let ((addy-plex (funcall addy-plex)))
+      (feedmail-fiddle-spray-address addy-plex)))
 
    ;; if it's a list, it must be a fiddle-plex -- so fiddle, man, fiddle
    ((listp addy-plex)
-       (feedmail-fiddle-header
-        (nth 0 addy-plex)      ;; name
-        (nth 1 addy-plex)      ;; value
-        (nth 2 addy-plex)      ;; action
-        (nth 3 addy-plex)))))  ;; folding
+    (feedmail-fiddle-header
+     (nth 0 addy-plex)                 ; name
+     (nth 1 addy-plex)                 ; value
+     (nth 2 addy-plex)                 ; action
+     (nth 3 addy-plex)))))             ; folding
 
 
 (defun feedmail-fiddle-list-of-spray-fiddle-plexes (list-of-fiddle-plexes)
   "Fiddling based on a list of fiddle-plexes for spraying."
+  (feedmail-say-debug ">in-> feedmail-fiddle-list-of-spray-fiddle-plexes")
   ;; default is to fall off the end of the list and do nothing
   (let ((lofp list-of-fiddle-plexes) fp)
-       (if (listp lofp)
-               (while lofp
-                 (setq fp (car lofp))
-                 (setq lofp (cdr lofp))
-                 (feedmail-fiddle-spray-address fp))
-         (feedmail-fiddle-spray-address lofp))))
+    (if (listp lofp)
+       (while lofp
+         (setq fp (car lofp))
+         (setq lofp (cdr lofp))
+         (feedmail-fiddle-spray-address fp))
+      (feedmail-fiddle-spray-address lofp))))
 
 
 (defun feedmail-fiddle-list-of-fiddle-plexes (list-of-fiddle-plexes)
   "Fiddling based on a list of fiddle-plexes.  Values t, nil, and string are 
pointless."
+  (feedmail-say-debug ">in-> feedmail-fiddle-list-of-fiddle-plexes")
   ;; default is to fall off the end of the list and do nothing
   (let ((lofp list-of-fiddle-plexes) fp)
-       (while lofp
-         (setq fp (car lofp))
-         (setq lofp (cdr lofp))
-         (cond
+    (while lofp
+      (setq fp (car lofp))
+      (setq lofp (cdr lofp))
+      (cond
 
-          ;; if it's a function, call it and recurse with the resulting value
-          ((and (symbolp fp) (fboundp fp))
-               (let ((lofp (list (funcall fp)))) 
(feedmail-fiddle-list-of-fiddle-plexes lofp)))
+       ;; if it's a function, call it and recurse with the resulting value
+       ((and (symbolp fp) (fboundp fp))
+       (let ((lofp (list (funcall fp)))) 
(feedmail-fiddle-list-of-fiddle-plexes lofp)))
 
-          ;; if it's a list, it must be a fiddle-plex -- so fiddle, man, fiddle
-          ((listp fp)
-               (feedmail-fiddle-header
-                (nth 0 fp)
-                (nth 1 fp)        ;; value
-                (nth 2 fp)        ;; action
-                (nth 3 fp)))))))  ;; folding
+       ;; if it's a list, it must be a fiddle-plex -- so fiddle, man, fiddle
+       ((listp fp)
+       (feedmail-fiddle-header
+        (nth 0 fp)
+        (nth 1 fp)                     ; value
+        (nth 2 fp)                     ; action
+        (nth 3 fp)))))))               ; folding
 
 
 (defun feedmail-accume-n-nuke-header (header-end header-regexp)
@@ -2509,139 +3028,175 @@ feedmail-accume-n-nuke-header
 There may be multiple such lines, and each may have arbitrarily
 many continuation lines.  Return an accumulation of the deleted
 headers, including the intervening newlines."
+  (feedmail-say-debug ">in-> feedmail-accume-n-nuke-header %s %s"
+                     header-end header-regexp)
   (let ((case-fold-search t) (dropout))
-       (save-excursion
-         (goto-char (point-min))
-         ;; iterate over all matching lines
-         (while (re-search-forward header-regexp header-end t)
-               (forward-line 1)
-               (setq dropout (concat dropout (buffer-substring 
(match-beginning 0) (point))))
-               (delete-region (match-beginning 0) (point))
-               ;; get rid of any continuation lines
-               (while (and (looking-at "^[ \t].*\n") (< (point) header-end))
-                 (forward-line 1)
-                 (setq dropout (concat dropout (buffer-substring 
(match-beginning 0) (point))))
-                 (replace-match ""))))
-       (identity dropout)))
+    (save-excursion
+      (goto-char (point-min))
+      ;; iterate over all matching lines
+      (while (re-search-forward header-regexp header-end t)
+       (forward-line 1)
+       (setq dropout (concat dropout (buffer-substring-no-properties 
(match-beginning 0) (point))))
+       (delete-region (match-beginning 0) (point))
+       ;; get rid of any continuation lines
+       (while (and (looking-at "^[ \t].*\n") (< (point) header-end))
+         (forward-line 1)
+         (setq dropout (concat dropout (buffer-substring-no-properties 
(match-beginning 0) (point))))
+         (replace-match ""))))
+    (identity dropout)))
 
 (defun feedmail-fill-to-cc-function (header-end)
   "Smart filling of address headers (don't be fooled by the name).
 The filling tries to avoid splitting lines except at commas.  This
 avoids, in particular, splitting within parenthesized comments in
-addresses.  Headers filled include FROM:, REPLY-TO:, TO:, CC:, BCC:,
-RESENT-TO:, RESENT-CC:, and RESENT-BCC:."
+addresses.  Headers filled include From:, Reply-To:, To:, Cc:, Bcc:,
+Resent-To:, Resent-Cc:, and Resent-Bcc:."
+  (feedmail-say-debug ">in-> feedmail-fill-to-cc-function")
   (let ((case-fold-search t)
-               this-line
-               this-line-end)
-       (save-excursion
-         (goto-char (point-min))
-         ;; iterate over all TO:/CC:, etc, lines
-         (while
-                 (re-search-forward
-                  
"^\\(FROM:\\|REPLY-TO:\\|TO:\\|CC:\\|BCC:\\|RESENT-TO:\\|RESENT-CC:\\|RESENT-BCC:\\)"
-                  header-end t)
-               (setq this-line (match-beginning 0))
-               ;; replace 0 or more leading spaces with a single space
-               (and (looking-at "[ \t]*") (replace-match " "))
-               (forward-line 1)
-               ;; get any continuation lines
-               (while (and (looking-at "[ \t]+") (< (point) header-end))
-                 (forward-line 1))
-               (setq this-line-end (point-marker))
-               (save-excursion (feedmail-fill-this-one this-line 
this-line-end))
-               ))))
+       this-line
+       this-line-end)
+    (save-excursion
+      (goto-char (point-min))
+      ;; iterate over all To:/Cc:, etc, lines
+      (while
+         (re-search-forward
+          
"^\\(From:\\|Reply-To:\\|To:\\|Cc:\\|Bcc:\\|Resent-To:\\|Resent-Cc:\\|Resent-Bcc:\\)"
+          header-end t)
+       (setq this-line (match-beginning 0))
+       ;; replace 0 or more leading spaces with a single space
+       (and (looking-at "[ \t]*") (replace-match " "))
+       (forward-line 1)
+       ;; get any continuation lines
+       (while (and (looking-at "[ \t]+") (< (point) header-end))
+         (forward-line 1))
+       (setq this-line-end (point-marker))
+       (save-excursion (feedmail-fill-this-one this-line this-line-end))
+       ))))
 
 
 (defun feedmail-fill-this-one (this-line this-line-end)
   "In-place smart filling of the region bounded by the two arguments."
+  (feedmail-say-debug ">in-> feedmail-fill-this-one")
   (let ((fill-prefix "\t")
-               (fill-column feedmail-fill-to-cc-fill-column))
-       ;; The general idea is to break only on commas.  Collapse
-       ;; multiple whitespace to a single blank; change
-       ;; all the blanks to something unprintable; change the
-       ;; commas to blanks; fill the region; change it back.
-       (goto-char this-line)
-       (while (re-search-forward "\\s-+" (1- this-line-end) t)
-         (replace-match " "))
-       
-       (subst-char-in-region this-line this-line-end ?   2 t) ; blank->C-b
-       (subst-char-in-region this-line this-line-end ?, ?  t) ; comma->blank
-       
-       (fill-region-as-paragraph this-line this-line-end)
-       
-       (subst-char-in-region this-line this-line-end ?  ?, t) ; comma<-blank
-       (subst-char-in-region this-line this-line-end  2 ?  t) ; blank<-C-b
-       
-       ;; look out for missing commas before continuation lines
-       (goto-char this-line)
-       (while (re-search-forward "\\([^,]\\)\n\t[ ]*" this-line-end t)
-         (replace-match "\\1,\n\t"))
-       ))
-
-
-(require 'mail-utils)                                  ; pick up 
mail-strip-quoted-names
+       (fill-column feedmail-fill-to-cc-fill-column))
+    ;; The general idea is to break only on commas.  Collapse
+    ;; multiple whitespace to a single blank; change
+    ;; all the blanks to something unprintable; change the
+    ;; commas to blanks; fill the region; change it back.
+    (goto-char this-line)
+    (while (re-search-forward "\\s-+" (1- this-line-end) t)
+      (replace-match " "))
+
+    (subst-char-in-region this-line this-line-end ?   2 t) ; blank->C-b
+    (subst-char-in-region this-line this-line-end ?, ?  t) ; comma->blank
+
+    (fill-region-as-paragraph this-line this-line-end)
+
+    (subst-char-in-region this-line this-line-end ?  ?, t) ; comma<-blank
+    (subst-char-in-region this-line this-line-end  2 ?  t) ; blank<-C-b
+
+    ;; look out for missing commas before continuation lines
+    (goto-char this-line)
+    (while (re-search-forward "\\([^,]\\)\n\t[ ]*" this-line-end t)
+      (replace-match "\\1,\n\t"))
+    ))
+
+
 (defun feedmail-deduce-address-list (message-buffer header-start header-end 
addr-regexp address-list)
   "Get address list with all comments and other excitement trimmed.
 Addresses are collected only from headers whose names match the fourth
-argument Returns a list of strings.  Duplicate addresses will have
+argument.  Returns a list of strings.  Duplicate addresses will have
 been weeded out."
+  (feedmail-say-debug ">in-> feedmail-deduce-address-list %s %s" addr-regexp 
address-list)
   (let ((simple-address)
-               (address-blob)
-               (this-line)
-               (this-line-end))
-       (unwind-protect
-               (save-excursion
-                 (set-buffer (get-buffer-create " *FQM scratch*")) 
(erase-buffer)
-                 (insert-buffer-substring message-buffer header-start 
header-end)
-                 (goto-char (point-min))
-                 (let ((case-fold-search t))
-                       (while (re-search-forward addr-regexp (point-max) t)
-                         (replace-match "")
-                         (setq this-line (match-beginning 0))
-                         (forward-line 1)
-                         ;; get any continuation lines
-                         (while (and (looking-at "^[ \t]+") (< (point) 
(point-max)))
-                               (forward-line 1))
-                         (setq this-line-end (point-marker))
-                         ;; only keep if we don't have it already
-                         (setq address-blob
-                                       (mail-strip-quoted-names 
(buffer-substring this-line this-line-end)))
-                         (while (string-match "\\([, \t\n\r]*\\)\\([^, 
\t\n\r]+\\)" address-blob)
-                               (setq simple-address (substring address-blob 
(match-beginning 2) (match-end 2)))
-                               (setq address-blob (replace-match "" t t 
address-blob))
-                               (if (not (member simple-address address-list))
-                                       (add-to-list 'address-list 
simple-address)))
-                         ))
-                 (kill-buffer nil)))
-       (identity address-list)))
-
-
-(defun feedmail-one-last-look (feedmail-prepped-text-buffer)
+       (address-blob)
+       (this-line)
+       (this-line-end))
+    (unwind-protect
+       (with-current-buffer (get-buffer-create " *FQM scratch*")
+          (erase-buffer)
+         (insert-buffer-substring message-buffer header-start header-end)
+         (goto-char (point-min))
+         (let ((case-fold-search t))
+           (while (re-search-forward addr-regexp (point-max) t)
+             (replace-match "")
+             (setq this-line (match-beginning 0))
+             (forward-line 1)
+             ;; get any continuation lines
+             (while (and (looking-at "^[ \t]+") (< (point) (point-max)))
+               (forward-line 1))
+             (setq this-line-end (point-marker))
+             ;; only keep if we don't have it already
+             (setq address-blob
+                   (mail-strip-quoted-names (buffer-substring-no-properties 
this-line this-line-end)))
+             (while (string-match "\\([, \t\n\r]*\\)\\([^, \t\n\r]+\\)" 
address-blob)
+               (setq simple-address (substring address-blob (match-beginning 
2) (match-end 2)))
+               (setq address-blob (replace-match "" t t address-blob))
+               (if (not (member simple-address address-list))
+                   (push simple-address address-list)))
+             ))
+         (kill-buffer nil)))
+    (identity address-list)))
+
+
+(defun feedmail-one-last-look (buffer)
   "Offer the user one last chance to give it up."
+  (feedmail-say-debug ">in-> feedmail-one-last-look")
   (save-excursion
-       (save-window-excursion
-         (switch-to-buffer feedmail-prepped-text-buffer)
-         (if (and (fboundp 'y-or-n-p-with-timeout) (numberp 
feedmail-confirm-outgoing-timeout))
-                 (y-or-n-p-with-timeout
-                  "FQM: Send this email? "
-                  (abs feedmail-confirm-outgoing-timeout)
-                  (> feedmail-confirm-outgoing-timeout 0))
-               (y-or-n-p "FQM: Send this email? "))
-         )))
+    ;; FIXME: switch-to-buffer may fail or pop up a new frame
+    ;; (in minibuffer-only frames, for example) and save-window-excursion
+    ;; won't delete the newly created frame upon exit!
+    (save-window-excursion
+      (switch-to-buffer buffer)
+      (if (and (fboundp 'y-or-n-p-with-timeout)
+               (numberp feedmail-confirm-outgoing-timeout))
+         (y-or-n-p-with-timeout
+          "FQM: Send this email? "
+          (abs feedmail-confirm-outgoing-timeout)
+          (> feedmail-confirm-outgoing-timeout 0))
+       (y-or-n-p "FQM: Send this email? "))
+      )))
 
 (defun feedmail-fqm-p (might-be)
   "Internal; does filename end with FQM suffix?"
+  (feedmail-say-debug ">in-> feedmail-fqm-p %s" might-be)
   (string-match (concat (regexp-quote feedmail-queue-fqm-suffix) "$") 
might-be))
 
+(defun feedmail-say-debug (format &optional a1 a2 a3 a4 a5 a6 a7 a8 a9)
+  "Internal; emits debug messages in standard format."
+  (when feedmail-debug
+    (funcall 'message (concat "FQM DB: " format) a1 a2 a3 a4 a5 a6 a7 a8 a9)
+    (and feedmail-debug-sit-for (not (= 0 feedmail-debug-sit-for))
+        (sit-for feedmail-debug-sit-for))))
+
+(defun feedmail-say-chatter (format &optional a1 a2 a3 a4 a5 a6 a7 a8 a9)
+  "Internal; emits queue chatter messages in standard format."
+  (when feedmail-queue-chatty
+    (funcall 'message (concat "FQM: " format) a1 a2 a3 a4 a5 a6 a7 a8 a9)
+    (and feedmail-queue-chatty-sit-for (not (= 0 
feedmail-queue-chatty-sit-for))
+        (sit-for feedmail-queue-chatty-sit-for))))
 
 (defun feedmail-find-eoh (&optional noerror)
   "Internal; finds the end of message header fields, returns mark just before 
it"
+  ;; all this funny business with line endings is to account for CRLF
+  ;; weirdness that I don't think I'll ever figure out
+  (feedmail-say-debug ">in-> feedmail-find-eoh %s" noerror)
+  (let ((mhs mail-header-separator)
+               (alt-mhs feedmail-queue-alternative-mail-header-separator)
+               r-mhs r-alt-mhs)
+       (setq r-mhs (concat "^" (regexp-quote mhs) "$"))
+       (setq r-alt-mhs (concat "^" (regexp-quote (or alt-mhs "")) "$"))
   (save-excursion
-       (goto-char (point-min))
-       (if (re-search-forward (concat "^" (regexp-quote mail-header-separator) 
"\n") nil noerror)
-               (progn
-                 (forward-line -1)
-                 (point-marker)))))
+    (goto-char (point-min))
+    (if (or (re-search-forward r-mhs nil t)
+             (and alt-mhs (re-search-forward r-alt-mhs nil t)))
+       (progn
+         (beginning-of-line)
+         (point-marker))
+      (if noerror
+         nil
+       (error "FQM: Can't find message-header-separator or alternate"))))))
 
 (provide 'feedmail)
+
 ;;; feedmail.el ends here



reply via email to

[Prev in Thread] Current Thread [Next in Thread]