*** empty log message ***

[gnus] / lisp / gnus-uu.el
diff --git a/lisp/gnus-uu.el b/lisp/gnus-uu.el

index 627c9d6..eeac01a 100644 (file)
--- a/lisp/gnus-uu.el
+++ b/lisp/gnus-uu.el
@@ -1,5 +1,5 @@
  ;;; gnus-uu.el --- extract (uu)encoded files in Gnus
  ;;; gnus-uu.el --- extract (uu)encoded files in Gnus
-;; Copyright (C) 1985,86,87,93,94,95,96 Free Software Foundation, Inc.
+;; Copyright (C) 1985,86,87,93,94,95,96,97 Free Software Foundation, Inc.
  
  ;; Author: Lars Magne Ingebrigtsen <larsi@ifi.uio.no>
  ;; Created: 2 Oct 1993
  
  ;; Author: Lars Magne Ingebrigtsen <larsi@ifi.uio.no>
  ;; Created: 2 Oct 1993
@@ -24,21 +24,42 @@
  
  ;;; Commentary:
  
  
  ;;; Commentary:
  
-;;; Code: 
+;;; Code:
+
+(eval-when-compile (require 'cl))
  
  (require 'gnus)
  
  (require 'gnus)
+(require 'gnus-art)
+(require 'message)
  (require 'gnus-msg)
  (require 'gnus-msg)
-(eval-when-compile (require 'cl))
+
+(defgroup gnus-extract nil
+  "Extracting encoded files."
+  :prefix "gnus-uu-"
+  :group 'gnus)
+
+(defgroup gnus-extract-view nil
+  "Viewwing extracted files."
+  :group 'gnus-extract)
+
+(defgroup gnus-extract-archive nil
+  "Extracting encoded archives."
+  :group 'gnus-extract)
+
+(defgroup gnus-extract-post nil
+  "Extracting encoded archives."
+  :prefix "gnus-uu-post"
+  :group 'gnus-extract)
  
  ;; Default viewing action rules
  
  
  ;; Default viewing action rules
  
-(defvar gnus-uu-default-view-rules 
+(defcustom gnus-uu-default-view-rules
    '(("\\.te?xt$\\|\\.doc$\\|read.*me\\|\\.c?$\\|\\.h$\\|\\.bat$\\|\\.asm$\\|makefile" "cat %s | sed s/\r//g")
      ("\\.pas$" "cat %s | sed s/\r//g")
      ("\\.[1-9]$" "groff -mandoc -Tascii %s | sed s/\b.//g")
      ("\\.\\(jpe?g\\|gif\\|tiff?\\|p[pgb]m\\|xwd\\|xbm\\|pcx\\)$" "xv")
      ("\\.tga$" "tgatoppm %s | xv -")
    '(("\\.te?xt$\\|\\.doc$\\|read.*me\\|\\.c?$\\|\\.h$\\|\\.bat$\\|\\.asm$\\|makefile" "cat %s | sed s/\r//g")
      ("\\.pas$" "cat %s | sed s/\r//g")
      ("\\.[1-9]$" "groff -mandoc -Tascii %s | sed s/\b.//g")
      ("\\.\\(jpe?g\\|gif\\|tiff?\\|p[pgb]m\\|xwd\\|xbm\\|pcx\\)$" "xv")
      ("\\.tga$" "tgatoppm %s | xv -")
-    ("\\.\\(wav\\|aiff\\|hcom\\|u[blw]\\|s[bfw]\\|voc\\|smp\\)$" 
+    ("\\.\\(wav\\|aiff\\|hcom\\|u[blw]\\|s[bfw]\\|voc\\|smp\\)$"
       "sox -v .5 %s -t .au -u - > /dev/audio")
      ("\\.au$" "cat %s > /dev/audio")
      ("\\.midi?$" "playmidi -f")
       "sox -v .5 %s -t .au -u - > /dev/audio")
      ("\\.au$" "cat %s > /dev/audio")
      ("\\.midi?$" "playmidi -f")
@@ -48,9 +69,9 @@
      ("\\.html$" "xmosaic")
      ("\\.mpe?g$" "mpeg_play")
      ("\\.\\(flc\\|fli\\|rle\\|iff\\|pfx\\|avi\\|sme\\|rpza\\|dl\\|qt\\|rsrc\\|mov\\)$" "xanim")
      ("\\.html$" "xmosaic")
      ("\\.mpe?g$" "mpeg_play")
      ("\\.\\(flc\\|fli\\|rle\\|iff\\|pfx\\|avi\\|sme\\|rpza\\|dl\\|qt\\|rsrc\\|mov\\)$" "xanim")
-    ("\\.\\(tar\\|arj\\|zip\\|zoo\\|arc\\|gz\\|Z\\|lzh\\|ar\\|lha\\)$" 
+    ("\\.\\(tar\\|arj\\|zip\\|zoo\\|arc\\|gz\\|Z\\|lzh\\|ar\\|lha\\)$"
       "gnus-uu-archive"))
       "gnus-uu-archive"))
-  "*Default actions to be taken when the user asks to view a file.  
+  "Default actions to be taken when the user asks to view a file.
  To change the behaviour, you can either edit this variable or set
  `gnus-uu-user-view-rules' to something useful.
  
  To change the behaviour, you can either edit this variable or set
  `gnus-uu-user-view-rules' to something useful.
  
@@ -61,38 +82,44 @@ following in your .emacs file:
  
    (setq gnus-uu-user-view-rules '((\"jpg$\\\\|gif$\" \"xli\")))
  
  
    (setq gnus-uu-user-view-rules '((\"jpg$\\\\|gif$\" \"xli\")))
  
-Both these variables are lists of lists with two string elements. The
-first string is a regular expression. If the file name matches this
+Both these variables are lists of lists with two string elements.  The
+first string is a regular expression.  If the file name matches this
  regular expression, the command in the second string is executed with
  the file as an argument.
  
  If the command string contains \"%s\", the file name will be inserted
  regular expression, the command in the second string is executed with
  the file as an argument.
  
  If the command string contains \"%s\", the file name will be inserted
-at that point in the command string. If there's no \"%s\" in the
+at that point in the command string.  If there's no \"%s\" in the
  command string, the file name will be appended to the command string
  before executing.
  
  There are several user variables to tailor the behaviour of gnus-uu to
  command string, the file name will be appended to the command string
  before executing.
  
  There are several user variables to tailor the behaviour of gnus-uu to
-your needs. First we have `gnus-uu-user-view-rules', which is the
+your needs.  First we have `gnus-uu-user-view-rules', which is the
  variable gnus-uu first consults when trying to decide how to view a
  variable gnus-uu first consults when trying to decide how to view a
-file. If this variable contains no matches, gnus-uu examines the
-default rule variable provided in this package. If gnus-uu finds no
+file.  If this variable contains no matches, gnus-uu examines the
+default rule variable provided in this package.  If gnus-uu finds no
  match here, it uses `gnus-uu-user-view-rules-end' to try to make a
  match here, it uses `gnus-uu-user-view-rules-end' to try to make a
-match.")
-
-(defvar gnus-uu-user-view-rules nil 
-  "*Variable detailing what actions are to be taken to view a file.
-See the documentation on the `gnus-uu-default-view-rules' variable for 
-details.")
-
-(defvar gnus-uu-user-view-rules-end 
+match."
+  :group 'gnus-extract-view
+  :type '(repeat (group regexp (string :tag "Command"))))
+
+(defcustom gnus-uu-user-view-rules nil
+  "What actions are to be taken to view a file.
+See the documentation on the `gnus-uu-default-view-rules' variable for
+details."
+  :group 'gnus-extract-view
+  :type '(repeat (group regexp (string :tag "Command"))))
+
+(defcustom gnus-uu-user-view-rules-end
    '(("" "file"))
    '(("" "file"))
-  "*Variable saying what actions are to be taken if no rule matched the file name.
-See the documentation on the `gnus-uu-default-view-rules' variable for 
-details.")
+  "What actions are to be taken if no rule matched the file name.
+See the documentation on the `gnus-uu-default-view-rules' variable for
+details."
+  :group 'gnus-extract-view
+  :type '(repeat (group regexp (string :tag "Command"))))
  
  ;; Default unpacking commands
  
  
  ;; Default unpacking commands
  
-(defvar gnus-uu-default-archive-rules 
+(defcustom gnus-uu-default-archive-rules
    '(("\\.tar$" "tar xf")
      ("\\.zip$" "unzip -o")
      ("\\.ar$" "ar x")
    '(("\\.tar$" "tar xf")
      ("\\.zip$" "unzip -o")
      ("\\.ar$" "ar x")
@@ -101,38 +128,49 @@ details.")
      ("\\.\\(lzh\\|lha\\)$" "lha x")
      ("\\.Z$" "uncompress")
      ("\\.gz$" "gunzip")
      ("\\.\\(lzh\\|lha\\)$" "lha x")
      ("\\.Z$" "uncompress")
      ("\\.gz$" "gunzip")
-    ("\\.arc$" "arc -x")))
+    ("\\.arc$" "arc -x"))
+  "See `gnus-uu-user-archive-rules'."
+  :group 'gnus-extract-archive
+  :type '(repeat (group regexp (string :tag "Command"))))
  
  
-(defvar gnus-uu-destructive-archivers 
+(defvar gnus-uu-destructive-archivers
    (list "uncompress" "gunzip"))
  
    (list "uncompress" "gunzip"))
  
-(defvar gnus-uu-user-archive-rules nil
-  "*A list that can be set to override the default archive unpacking commands.
+(defcustom gnus-uu-user-archive-rules nil
+  "A list that can be set to override the default archive unpacking commands.
  To use, for instance, 'untar' to unpack tar files and 'zip -x' to
  unpack zip files, say the following:
  To use, for instance, 'untar' to unpack tar files and 'zip -x' to
  unpack zip files, say the following:
-  (setq gnus-uu-user-archive-rules 
+  (setq gnus-uu-user-archive-rules
      '((\"\\\\.tar$\" \"untar\")
      '((\"\\\\.tar$\" \"untar\")
-      (\"\\\\.zip$\" \"zip -x\")))")
+      (\"\\\\.zip$\" \"zip -x\")))"
+  :group 'gnus-extract-archive
+  :type '(repeat (group regexp (string :tag "Command"))))
  
  
-(defvar gnus-uu-ignore-files-by-name nil
+(defcustom gnus-uu-ignore-files-by-name nil
    "*A regular expression saying what files should not be viewed based on name.
    "*A regular expression saying what files should not be viewed based on name.
-If, for instance, you want gnus-uu to ignore all .au and .wav files, 
+If, for instance, you want gnus-uu to ignore all .au and .wav files,
  you could say something like
  
    (setq gnus-uu-ignore-files-by-name \"\\\\.au$\\\\|\\\\.wav$\")
  
  Note that this variable can be used in conjunction with the
  you could say something like
  
    (setq gnus-uu-ignore-files-by-name \"\\\\.au$\\\\|\\\\.wav$\")
  
  Note that this variable can be used in conjunction with the
-`gnus-uu-ignore-files-by-type' variable.")
+`gnus-uu-ignore-files-by-type' variable."
+  :group 'gnus-extract
+  :type '(choice (const :tag "off" nil)
+                (regexp :format "%v")))
  
  
-(defvar gnus-uu-ignore-files-by-type nil
+(defcustom gnus-uu-ignore-files-by-type nil
    "*A regular expression saying what files that shouldn't be viewed, based on MIME file type.
    "*A regular expression saying what files that shouldn't be viewed, based on MIME file type.
-If, for instance, you want gnus-uu to ignore all audio files and all mpegs, 
+If, for instance, you want gnus-uu to ignore all audio files and all mpegs,
  you could say something like
  
    (setq gnus-uu-ignore-files-by-type \"audio/\\\\|video/mpeg\")
  
  Note that this variable can be used in conjunction with the
  you could say something like
  
    (setq gnus-uu-ignore-files-by-type \"audio/\\\\|video/mpeg\")
  
  Note that this variable can be used in conjunction with the
-`gnus-uu-ignore-files-by-name' variable.")
+`gnus-uu-ignore-files-by-name' variable."
+  :group 'gnus-extract
+  :type '(choice (const :tag "off" nil)
+                (regexp :format "%v")))
  
  ;; Pseudo-MIME support
  
  
  ;; Pseudo-MIME support
  
@@ -175,72 +213,111 @@ Note that this variable can be used in conjunction with the
      ("\\.rsrc$" "video/rsrc")
      ("\\..*$" "unknown/unknown")))
  
      ("\\.rsrc$" "video/rsrc")
      ("\\..*$" "unknown/unknown")))
  
-;; Various variables users may set 
+;; Various variables users may set
  
  
-(defvar gnus-uu-tmp-dir "/tmp/" 
+(defcustom gnus-uu-tmp-dir "/tmp/"
    "*Variable saying where gnus-uu is to do its work.
    "*Variable saying where gnus-uu is to do its work.
-Default is \"/tmp/\".")
+Default is \"/tmp/\"."
+  :group 'gnus-extract
+  :type 'directory)
  
  
-(defvar gnus-uu-do-not-unpack-archives nil 
-  "*Non-nil means that gnus-uu won't peek inside archives looking for files to display. 
-Default is nil.")
+(defcustom gnus-uu-do-not-unpack-archives nil
+  "*Non-nil means that gnus-uu won't peek inside archives looking for files to display.
+Default is nil."
+  :group 'gnus-extract-archive
+  :type 'boolean)
  
  
-(defvar gnus-uu-ignore-default-view-rules nil
+(defcustom gnus-uu-ignore-default-view-rules nil
    "*Non-nil means that gnus-uu will ignore the default viewing rules.
    "*Non-nil means that gnus-uu will ignore the default viewing rules.
-Only the user viewing rules will be consulted. Default is nil.")
+Only the user viewing rules will be consulted.  Default is nil."
+  :group 'gnus-extract-view
+  :type 'boolean)
  
  
-(defvar gnus-uu-grabbed-file-functions nil
-  "*Functions run on each file after successful decoding.
+(defcustom gnus-uu-grabbed-file-functions nil
+  "Functions run on each file after successful decoding.
  They will be called with the name of the file as the argument.
  They will be called with the name of the file as the argument.
-Likely functions you can use in this list are `gnus-uu-grab-view' 
-and `gnus-uu-grab-move'.")
-
-(defvar gnus-uu-ignore-default-archive-rules nil 
-  "*Non-nil means that gnus-uu will ignore the default archive unpacking commands.  
-Only the user unpacking commands will be consulted. Default is nil.")
-
-(defvar gnus-uu-kill-carriage-return t
+Likely functions you can use in this list are `gnus-uu-grab-view'
+and `gnus-uu-grab-move'."
+  :group 'gnus-extract
+  :options '(gnus-uu-grab-view gnus-uu-grab-move)
+  :type 'hook)
+
+(defcustom gnus-uu-ignore-default-archive-rules nil
+  "*Non-nil means that gnus-uu will ignore the default archive unpacking commands.
+Only the user unpacking commands will be consulted.  Default is nil."
+  :group 'gnus-extract-archive
+  :type 'boolean)
+
+(defcustom gnus-uu-kill-carriage-return t
    "*Non-nil means that gnus-uu will strip all carriage returns from articles.
    "*Non-nil means that gnus-uu will strip all carriage returns from articles.
-Default is t.")
+Default is t."
+  :group 'gnus-extract
+  :type 'boolean)
  
  
-(defvar gnus-uu-view-with-metamail nil
+(defcustom gnus-uu-view-with-metamail nil
    "*Non-nil means that files will be viewed with metamail.
  The gnus-uu viewing functions will be ignored and gnus-uu will try
    "*Non-nil means that files will be viewed with metamail.
  The gnus-uu viewing functions will be ignored and gnus-uu will try
-to guess at a content-type based on file name suffixes. Default
-it nil.")
-
-(defvar gnus-uu-unmark-articles-not-decoded nil
-  "*Non-nil means that gnus-uu will mark articles that were unsuccessfully decoded as unread. 
-Default is nil.")
-
-(defvar gnus-uu-correct-stripped-uucode nil
-  "*Non-nil means that gnus-uu will *try* to fix uuencoded files that have had trailing spaces deleted. 
-Default is nil.")
-
-(defvar gnus-uu-save-in-digest nil
+to guess at a content-type based on file name suffixes.  Default
+it nil."
+  :group 'gnus-extract
+  :type 'boolean)
+
+(defcustom gnus-uu-unmark-articles-not-decoded nil
+  "*Non-nil means that gnus-uu will mark articles that were unsuccessfully decoded as unread.
+Default is nil."
+  :group 'gnus-extract
+  :type 'boolean)
+
+(defcustom gnus-uu-correct-stripped-uucode nil
+  "*Non-nil means that gnus-uu will *try* to fix uuencoded files that have had trailing spaces deleted.
+Default is nil."
+  :group 'gnus-extract
+  :type 'boolean)
+
+(defcustom gnus-uu-save-in-digest nil
    "*Non-nil means that gnus-uu, when asked to save without decoding, will save in digests.
    "*Non-nil means that gnus-uu, when asked to save without decoding, will save in digests.
-If this variable is nil, gnus-uu will just save everything in a 
-file without any embellishments. The digesting almost conforms to RFC1153 -
-no easy way to specify any meaningful volume and issue numbers were found, 
-so I simply dropped them.")
-
-(defvar gnus-uu-digest-headers 
+If this variable is nil, gnus-uu will just save everything in a
+file without any embellishments.  The digesting almost conforms to RFC1153 -
+no easy way to specify any meaningful volume and issue numbers were found,
+so I simply dropped them."
+  :group 'gnus-extract
+  :type 'boolean)
+
+(defcustom gnus-uu-pre-uudecode-hook nil
+  "Hook run before sending a message to uudecode."
+  :group 'gnus-extract
+  :type 'hook)
+
+(defcustom gnus-uu-digest-headers
    '("^Date:" "^From:" "^To:" "^Cc:" "^Subject:" "^Message-ID:" "^Keywords:"
      "^Summary:" "^References:")
    '("^Date:" "^From:" "^To:" "^Cc:" "^Subject:" "^Message-ID:" "^Keywords:"
      "^Summary:" "^References:")
-  "*List of regexps to match headers included in digested messages.
-The headers will be included in the sequence they are matched.")
-
-(defvar gnus-uu-save-separate-articles nil
-  "*Non-nil means that gnus-uu will save articles in separate files.")
+  "List of regexps to match headers included in digested messages.
+The headers will be included in the sequence they are matched."
+  :group 'gnus-extract
+  :type '(repeat regexp))
+
+(defcustom gnus-uu-save-separate-articles nil
+  "*Non-nil means that gnus-uu will save articles in separate files."
+  :group 'gnus-extract
+  :type 'boolean)
+
+(defcustom gnus-uu-be-dangerous 'ask
+  "*Specifies what to do if unusual situations arise during decoding.
+If nil, be as conservative as possible.  If t, ignore things that
+didn't work, and overwrite existing files.  Otherwise, ask each time."
+  :group 'gnus-extract
+  :type '(choice (const :tag "conservative" nil)
+                (const :tag "ask" ask)
+                (const :tag "liberal" t)))
  
  ;; Internal variables
  
  (defvar gnus-uu-saved-article-name nil)
  
  
  ;; Internal variables
  
  (defvar gnus-uu-saved-article-name nil)
  
-(defconst gnus-uu-begin-string "^begin[ \t]+[0-7][0-7][0-7][ \t]+\\(.*\\)$")
-(defconst gnus-uu-end-string "^end[ \t]*$")
+(defvar gnus-uu-begin-string "^begin[ \t]+[0-7][0-7][0-7][ \t]+\\(.*\\)$")
+(defvar gnus-uu-end-string "^end[ \t]*$")
  
  
-(defconst gnus-uu-body-line "^M")
+(defvar gnus-uu-body-line "^M")
  (let ((i 61))
    (while (> (setq i (1- i)) 0)
      (setq gnus-uu-body-line (concat gnus-uu-body-line "[^a-z]")))
  (let ((i 61))
    (while (> (setq i (1- i)) 0)
      (setq gnus-uu-body-line (concat gnus-uu-body-line "[^a-z]")))
@@ -248,58 +325,60 @@ The headers will be included in the sequence they are matched.")
  
  ;"^M.............................................................?$"
  
  
  ;"^M.............................................................?$"
  
-(defconst gnus-uu-shar-begin-string "^#! */bin/sh")
+(defvar gnus-uu-shar-begin-string "^#! */bin/sh")
  
  (defvar gnus-uu-shar-file-name nil)
  
  (defvar gnus-uu-shar-file-name nil)
-(defconst gnus-uu-shar-name-marker "begin [0-7][0-7][0-7][ \t]+\\(\\(\\w\\|\\.\\)*\\b\\)")
+(defvar gnus-uu-shar-name-marker "begin [0-7][0-7][0-7][ \t]+\\(\\(\\w\\|\\.\\)*\\b\\)")
  
  
-(defconst gnus-uu-postscript-begin-string "^%!PS-")
-(defconst gnus-uu-postscript-end-string "^%%EOF$")
+(defvar gnus-uu-postscript-begin-string "^%!PS-")
+(defvar gnus-uu-postscript-end-string "^%%EOF$")
  
  (defvar gnus-uu-file-name nil)
  
  (defvar gnus-uu-file-name nil)
-(defconst gnus-uu-uudecode-process nil)
+(defvar gnus-uu-uudecode-process nil)
  (defvar gnus-uu-binhex-article-name nil)
  
  (defvar gnus-uu-work-dir nil)
  
  (defvar gnus-uu-binhex-article-name nil)
  
  (defvar gnus-uu-work-dir nil)
  
-(defconst gnus-uu-output-buffer-name " *Gnus UU Output*")
+(defvar gnus-uu-output-buffer-name " *Gnus UU Output*")
  
  (defvar gnus-uu-default-dir gnus-article-save-directory)
  (defvar gnus-uu-digest-from-subject nil)
  
  ;; Keymaps
  
  
  (defvar gnus-uu-default-dir gnus-article-save-directory)
  (defvar gnus-uu-digest-from-subject nil)
  
  ;; Keymaps
  
-(gnus-define-keys 
- (gnus-uu-mark-map "P" gnus-summary-mark-map)
- "p" gnus-summary-mark-as-processable
- "u" gnus-summary-unmark-as-processable
- "U" gnus-summary-unmark-all-processable
- "v" gnus-uu-mark-over
- "s" gnus-uu-mark-series
- "r" gnus-uu-mark-region
- "R" gnus-uu-mark-by-regexp
- "t" gnus-uu-mark-thread
- "T" gnus-uu-unmark-thread
- "a" gnus-uu-mark-all
- "b" gnus-uu-mark-buffer
- "S" gnus-uu-mark-sparse)
-
-(gnus-define-keys 
- (gnus-uu-extract-map "X" gnus-summary-mode-map)
- ;;"x" gnus-uu-extract-any
- ;;"m" gnus-uu-extract-mime
- "u" gnus-uu-decode-uu
- "U" gnus-uu-decode-uu-and-save
- "s" gnus-uu-decode-unshar
- "S" gnus-uu-decode-unshar-and-save
- "o" gnus-uu-decode-save
- "O" gnus-uu-decode-save
- "b" gnus-uu-decode-binhex
- "B" gnus-uu-decode-binhex
- "p" gnus-uu-decode-postscript
- "P" gnus-uu-decode-postscript-and-save)
-
-(gnus-define-keys 
+(gnus-define-keys (gnus-uu-mark-map "P" gnus-summary-mark-map)
+  "p" gnus-summary-mark-as-processable
+  "u" gnus-summary-unmark-as-processable
+  "U" gnus-summary-unmark-all-processable
+  "v" gnus-uu-mark-over
+  "s" gnus-uu-mark-series
+  "r" gnus-uu-mark-region
+  "R" gnus-uu-mark-by-regexp
+  "t" gnus-uu-mark-thread
+  "T" gnus-uu-unmark-thread
+  "a" gnus-uu-mark-all
+  "b" gnus-uu-mark-buffer
+  "S" gnus-uu-mark-sparse
+  "k" gnus-summary-kill-process-mark
+  "y" gnus-summary-yank-process-mark
+  "w" gnus-summary-save-process-mark
+  "i" gnus-uu-invert-processable)
+
+(gnus-define-keys (gnus-uu-extract-map "X" gnus-summary-mode-map)
+  ;;"x" gnus-uu-extract-any
+  ;;"m" gnus-uu-extract-mime
+  "u" gnus-uu-decode-uu
+  "U" gnus-uu-decode-uu-and-save
+  "s" gnus-uu-decode-unshar
+  "S" gnus-uu-decode-unshar-and-save
+  "o" gnus-uu-decode-save
+  "O" gnus-uu-decode-save
+  "b" gnus-uu-decode-binhex
+  "B" gnus-uu-decode-binhex
+  "p" gnus-uu-decode-postscript
+  "P" gnus-uu-decode-postscript-and-save)
+
+(gnus-define-keys
   (gnus-uu-extract-view-map "v" gnus-uu-extract-map)
   "u" gnus-uu-decode-uu-view
   "U" gnus-uu-decode-uu-and-save-view
   (gnus-uu-extract-view-map "v" gnus-uu-extract-map)
   "u" gnus-uu-decode-uu-view
   "U" gnus-uu-decode-uu-and-save-view
@@ -317,7 +396,7 @@ The headers will be included in the sequence they are matched.")
  
  (defun gnus-uu-decode-uu (&optional n)
    "Uudecodes the current article."
  
  (defun gnus-uu-decode-uu (&optional n)
    "Uudecodes the current article."
-  (interactive "P") 
+  (interactive "P")
    (gnus-uu-decode-with-method 'gnus-uu-uustrip-article n))
  
  (defun gnus-uu-decode-uu-and-save (n dir)
    (gnus-uu-decode-with-method 'gnus-uu-uustrip-article n))
  
  (defun gnus-uu-decode-uu-and-save (n dir)
@@ -349,7 +428,7 @@ The headers will be included in the sequence they are matched.")
    "Saves the current article."
    (interactive
     (list current-prefix-arg
    "Saves the current article."
    (interactive
     (list current-prefix-arg
-        (read-file-name 
+        (read-file-name
           (if gnus-uu-save-separate-articles
               "Save articles is dir: "
             "Save articles in file: ")
           (if gnus-uu-save-separate-articles
               "Save articles is dir: "
             "Save articles in file: ")
@@ -366,12 +445,12 @@ The headers will be included in the sequence they are matched.")
           (read-file-name "Unbinhex and save in dir: "
                           gnus-uu-default-dir
                           gnus-uu-default-dir))))
           (read-file-name "Unbinhex and save in dir: "
                           gnus-uu-default-dir
                           gnus-uu-default-dir))))
-  (setq gnus-uu-binhex-article-name 
+  (setq gnus-uu-binhex-article-name
         (make-temp-name (concat gnus-uu-work-dir "binhex")))
    (gnus-uu-decode-with-method 'gnus-uu-binhex-article n dir))
  
  (defun gnus-uu-decode-uu-view (&optional n)
         (make-temp-name (concat gnus-uu-work-dir "binhex")))
    (gnus-uu-decode-with-method 'gnus-uu-binhex-article n dir))
  
  (defun gnus-uu-decode-uu-view (&optional n)
-  "Uudecodes and views the current article."    
+  "Uudecodes and views the current article."
    (interactive "P")
    (let ((gnus-view-pseudos (or gnus-view-pseudos 'automatic)))
      (gnus-uu-decode-uu n)))
    (interactive "P")
    (let ((gnus-view-pseudos (or gnus-view-pseudos 'automatic)))
      (gnus-uu-decode-uu n)))
@@ -419,7 +498,7 @@ The headers will be included in the sequence they are matched.")
     (list current-prefix-arg
          (read-file-name "Unbinhex, view and save in dir: "
                          gnus-uu-default-dir gnus-uu-default-dir)))
     (list current-prefix-arg
          (read-file-name "Unbinhex, view and save in dir: "
                          gnus-uu-default-dir gnus-uu-default-dir)))
-  (setq gnus-uu-binhex-article-name 
+  (setq gnus-uu-binhex-article-name
         (make-temp-name (concat gnus-uu-work-dir "binhex")))
    (let ((gnus-view-pseudos (or gnus-view-pseudos 'automatic)))
      (gnus-uu-decode-binhex n file)))
         (make-temp-name (concat gnus-uu-work-dir "binhex")))
    (let ((gnus-view-pseudos (or gnus-view-pseudos 'automatic)))
      (gnus-uu-decode-binhex n file)))
@@ -431,43 +510,45 @@ The headers will be included in the sequence they are matched.")
    "Digests and forwards all articles in this series."
    (interactive "P")
    (let ((gnus-uu-save-in-digest t)
    "Digests and forwards all articles in this series."
    (interactive "P")
    (let ((gnus-uu-save-in-digest t)
-       (file (make-temp-name (concat gnus-uu-tmp-dir "forward")))
-       buf subject from)
-    (setq gnus-uu-digest-from-subject nil)
-    (gnus-uu-decode-save n file)
-    (setq buf (switch-to-buffer (get-buffer-create " *gnus-uu-forward*")))
-    (gnus-add-current-to-buffer-list)
-    (erase-buffer)
-    (delete-other-windows)
-    (insert-file file)
-    (let ((fs gnus-uu-digest-from-subject))
-      (if (not fs)
-         ()
-       (setq from (caar fs)
-             subject (gnus-simplify-subject-fuzzy (cdar fs))
-             fs (cdr fs))
-       (while (and fs (or from subject))
-         (and from
-              (or (string= from (caar fs))
-                  (setq from nil)))
-         (and subject
-              (or (string= (gnus-simplify-subject-fuzzy (cdar fs))
-                           subject)
-                  (setq subject nil)))
-         (setq fs (cdr fs))))
-      (or subject (setq subject "Digested Articles"))
-      (or from (setq from "Various")))
-    (goto-char (point-min))
-    (and (re-search-forward "^Subject: ")
-        (progn
-          (delete-region (point) (gnus-point-at-eol))
-          (insert subject)))
-    (goto-char (point-min))
-    (and (re-search-forward "^From: ")
-        (progn
-          (delete-region (point) (gnus-point-at-eol))
-          (insert from)))
-    (message-forward post)
+       (file (make-temp-name (nnheader-concat gnus-uu-tmp-dir "forward")))
+       buf subject from newsgroups)
+    (gnus-setup-message 'forward
+      (setq gnus-uu-digest-from-subject nil)
+      (gnus-uu-decode-save n file)
+      (setq buf (switch-to-buffer (get-buffer-create " *gnus-uu-forward*")))
+      (gnus-add-current-to-buffer-list)
+      (erase-buffer)
+      (insert-file file)
+      (let ((fs gnus-uu-digest-from-subject))
+       (when fs
+         (setq from (caar fs)
+               subject (gnus-simplify-subject-fuzzy (cdar fs))
+               fs (cdr fs))
+         (while (and fs (or from subject))
+           (when from
+             (unless (string= from (caar fs))
+               (setq from nil)))
+           (when subject
+             (unless (string= (gnus-simplify-subject-fuzzy (cdar fs))
+                              subject)
+               (setq subject nil)))
+           (setq fs (cdr fs))))
+       (unless subject
+         (setq subject "Digested Articles"))
+       (unless from
+         (setq from
+               (if (gnus-news-group-p gnus-newsgroup-name)
+                   gnus-newsgroup-name
+                 "Various"))))
+      (goto-char (point-min))
+      (when (re-search-forward "^Subject: ")
+       (delete-region (point) (gnus-point-at-eol))
+       (insert subject))
+      (goto-char (point-min))
+      (when (re-search-forward "^From: ")
+       (delete-region (point) (gnus-point-at-eol))
+       (insert from))
+      (message-forward post))
      (delete-file file)
      (kill-buffer buf)
      (setq gnus-uu-digest-from-subject nil)))
      (delete-file file)
      (kill-buffer buf)
      (setq gnus-uu-digest-from-subject nil)))
@@ -482,7 +563,6 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-mark-by-regexp (regexp &optional unmark)
    "Ask for a regular expression and set the process mark on all articles that match."
    (interactive (list (read-from-minibuffer "Mark (regexp): ")))
  (defun gnus-uu-mark-by-regexp (regexp &optional unmark)
    "Ask for a regular expression and set the process mark on all articles that match."
    (interactive (list (read-from-minibuffer "Mark (regexp): ")))
-  (gnus-set-global-variables)
    (let ((articles (gnus-uu-find-articles-matching regexp)))
      (while articles
        (if unmark
    (let ((articles (gnus-uu-find-articles-matching regexp)))
      (while articles
        (if unmark
@@ -499,7 +579,6 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-mark-series ()
    "Mark the current series with the process mark."
    (interactive)
  (defun gnus-uu-mark-series ()
    "Mark the current series with the process mark."
    (interactive)
-  (gnus-set-global-variables)
    (let ((articles (gnus-uu-find-articles-matching)))
      (while articles
        (gnus-summary-set-process-mark (car articles))
    (let ((articles (gnus-uu-find-articles-matching)))
      (while articles
        (gnus-summary-set-process-mark (car articles))
@@ -510,7 +589,6 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-mark-region (beg end &optional unmark)
    "Set the process mark on all articles between point and mark."
    (interactive "r")
  (defun gnus-uu-mark-region (beg end &optional unmark)
    "Set the process mark on all articles between point and mark."
    (interactive "r")
-  (gnus-set-global-variables)
    (save-excursion
      (goto-char beg)
      (while (< (point) end)
    (save-excursion
      (goto-char beg)
      (while (< (point) end)
@@ -529,16 +607,15 @@ The headers will be included in the sequence they are matched.")
    "Set the process mark on all articles in the buffer."
    (interactive)
    (gnus-uu-mark-region (point-min) (point-max)))
    "Set the process mark on all articles in the buffer."
    (interactive)
    (gnus-uu-mark-region (point-min) (point-max)))
-      
+
  (defun gnus-uu-unmark-buffer ()
    "Remove the process mark on all articles in the buffer."
    (interactive)
    (gnus-uu-mark-region (point-min) (point-max) t))
  (defun gnus-uu-unmark-buffer ()
    "Remove the process mark on all articles in the buffer."
    (interactive)
    (gnus-uu-mark-region (point-min) (point-max) t))
-      
+
  (defun gnus-uu-mark-thread ()
    "Marks all articles downwards in this thread."
    (interactive)
  (defun gnus-uu-mark-thread ()
    "Marks all articles downwards in this thread."
    (interactive)
-  (gnus-set-global-variables)
    (let ((level (gnus-summary-thread-level)))
      (while (and (gnus-summary-set-process-mark (gnus-summary-article-number))
                 (zerop (gnus-summary-next-subject 1))
    (let ((level (gnus-summary-thread-level)))
      (while (and (gnus-summary-set-process-mark (gnus-summary-article-number))
                 (zerop (gnus-summary-next-subject 1))
@@ -548,7 +625,6 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-unmark-thread ()
    "Unmarks all articles downwards in this thread."
    (interactive)
  (defun gnus-uu-unmark-thread ()
    "Unmarks all articles downwards in this thread."
    (interactive)
-  (gnus-set-global-variables)
    (let ((level (gnus-summary-thread-level)))
      (while (and (gnus-summary-remove-process-mark
                  (gnus-summary-article-number))
    (let ((level (gnus-summary-thread-level)))
      (while (and (gnus-summary-remove-process-mark
                  (gnus-summary-article-number))
@@ -556,6 +632,19 @@ The headers will be included in the sequence they are matched.")
                 (> (gnus-summary-thread-level) level))))
    (gnus-summary-position-point))
  
                 (> (gnus-summary-thread-level) level))))
    (gnus-summary-position-point))
  
+(defun gnus-uu-invert-processable ()
+  "Invert the list of process-marked articles."
+  (interactive)
+  (let ((data gnus-newsgroup-data)
+       d number)
+    (save-excursion
+      (while data
+       (if (memq (setq number (gnus-data-number (pop data)))
+                 gnus-newsgroup-processable)
+           (gnus-summary-remove-process-mark number)
+         (gnus-summary-set-process-mark number)))))
+  (gnus-summary-position-point))
+
  (defun gnus-uu-mark-over (&optional score)
    "Mark all articles with a score over SCORE (the prefix.)"
    (interactive "P")
  (defun gnus-uu-mark-over (&optional score)
    "Mark all articles with a score over SCORE (the prefix.)"
    (interactive "P")
@@ -574,17 +663,17 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-mark-sparse ()
    "Mark all series that have some articles marked."
    (interactive)
  (defun gnus-uu-mark-sparse ()
    "Mark all series that have some articles marked."
    (interactive)
-  (gnus-set-global-variables)
    (let ((marked (nreverse gnus-newsgroup-processable))
         subject articles total headers)
    (let ((marked (nreverse gnus-newsgroup-processable))
         subject articles total headers)
-    (or marked (error "No articles marked with the process mark"))
+    (unless marked
+      (error "No articles marked with the process mark"))
      (setq gnus-newsgroup-processable nil)
      (save-excursion
        (while marked
      (setq gnus-newsgroup-processable nil)
      (save-excursion
        (while marked
-       (and (vectorp (setq headers 
+       (and (vectorp (setq headers
                             (gnus-summary-article-header (car marked))))
              (setq subject (mail-header-subject headers)
                             (gnus-summary-article-header (car marked))))
              (setq subject (mail-header-subject headers)
-                  articles (gnus-uu-find-articles-matching 
+                  articles (gnus-uu-find-articles-matching
                              (gnus-uu-reginize-string subject))
                    total (nconc total articles)))
         (while articles
                              (gnus-uu-reginize-string subject))
                    total (nconc total articles)))
         (while articles
@@ -598,7 +687,6 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-mark-all ()
    "Mark all articles in \"series\" order."
    (interactive)
  (defun gnus-uu-mark-all ()
    "Mark all articles in \"series\" order."
    (interactive)
-  (gnus-set-global-variables)
    (setq gnus-newsgroup-processable nil)
    (save-excursion
      (let ((data gnus-newsgroup-data)
    (setq gnus-newsgroup-processable nil)
    (save-excursion
      (let ((data gnus-newsgroup-data)
@@ -612,7 +700,7 @@ The headers will be included in the sequence they are matched.")
         (setq data (cdr data)))))
    (gnus-summary-position-point))
  
         (setq data (cdr data)))))
    (gnus-summary-position-point))
  
-;; All PostScript functions written by Erik Selberg <speed@cs.washington.edu>. 
+;; All PostScript functions written by Erik Selberg <speed@cs.washington.edu>.
  
  (defun gnus-uu-decode-postscript (&optional n)
    "Gets postscript of the current article."
  
  (defun gnus-uu-decode-postscript (&optional n)
    "Gets postscript of the current article."
@@ -633,7 +721,7 @@ The headers will be included in the sequence they are matched.")
           (read-file-name "Save in dir: "
                           gnus-uu-default-dir
                           gnus-uu-default-dir t))))
           (read-file-name "Save in dir: "
                           gnus-uu-default-dir
                           gnus-uu-default-dir t))))
-  (gnus-uu-decode-with-method 'gnus-uu-decode-postscript-article 
+  (gnus-uu-decode-with-method 'gnus-uu-decode-postscript-article
                               n dir nil nil t))
  
  (defun gnus-uu-decode-postscript-and-save-view (n dir)
                               n dir nil nil t))
  
  (defun gnus-uu-decode-postscript-and-save-view (n dir)
@@ -649,10 +737,11 @@ The headers will be included in the sequence they are matched.")
  
  ;; Internal functions.
  
  
  ;; Internal functions.
  
-(defun gnus-uu-decode-with-method (method n &optional save not-insert 
+(defun gnus-uu-decode-with-method (method n &optional save not-insert
                                           scan cdir)
    (gnus-uu-initialize scan)
                                           scan cdir)
    (gnus-uu-initialize scan)
-  (if save (setq gnus-uu-default-dir save))
+  (when save
+    (setq gnus-uu-default-dir save))
    ;; Create the directory we save to.
    (when (and scan cdir save
              (not (file-exists-p save)))
    ;; Create the directory we save to.
    (when (and scan cdir save
              (not (file-exists-p save)))
@@ -661,9 +750,11 @@ The headers will be included in the sequence they are matched.")
         files)
      (setq files (gnus-uu-grab-articles articles method t))
      (let ((gnus-current-article (car articles)))
         files)
      (setq files (gnus-uu-grab-articles articles method t))
      (let ((gnus-current-article (car articles)))
-      (and scan (setq files (gnus-uu-scan-directory gnus-uu-work-dir))))
-    (and save (gnus-uu-save-files files save))
-    (if (eq gnus-uu-do-not-unpack-archives nil)
+      (when scan
+       (setq files (gnus-uu-scan-directory gnus-uu-work-dir))))
+    (when save
+      (gnus-uu-save-files files save))
+    (when (eq gnus-uu-do-not-unpack-archives nil)
        (setq files (gnus-uu-unpack-files files)))
      (setq files (nreverse (gnus-uu-get-actions files)))
      (or not-insert (not gnus-insert-pseudo-articles)
        (setq files (gnus-uu-unpack-files files)))
      (setq files (nreverse (gnus-uu-get-actions files)))
      (or not-insert (not gnus-insert-pseudo-articles)
@@ -680,7 +771,7 @@ The headers will be included in the sequence they are matched.")
               out)
         (when (file-directory-p file)
           (setq out (nconc (gnus-uu-scan-directory file t) out)))))
               out)
         (when (file-directory-p file)
           (setq out (nconc (gnus-uu-scan-directory file t) out)))))
-    (if rec 
+    (if rec
         out
        (nreverse out))))
  
         out
        (nreverse out))))
  
@@ -694,11 +785,13 @@ The headers will be included in the sequence they are matched.")
         (string-match reg file)
         (setq fromdir (substring file (match-end 0)))
         (if (file-directory-p file)
         (string-match reg file)
         (setq fromdir (substring file (match-end 0)))
         (if (file-directory-p file)
-           (unless (file-exists-p (concat dir fromdir))
-             (make-directory (concat dir fromdir) t))
+           (gnus-make-directory (concat dir fromdir))
           (setq to-file (concat dir fromdir))
           (when (or (not (file-exists-p to-file))
           (setq to-file (concat dir fromdir))
           (when (or (not (file-exists-p to-file))
-                   (gnus-y-or-n-p (format "%s exists; overwrite? " to-file)))
+                   (eq gnus-uu-be-dangerous t)
+                   (and gnus-uu-be-dangerous
+                        (gnus-y-or-n-p (format "%s exists; overwrite? "
+                                               to-file))))
             (copy-file file to-file t t)))))
      (gnus-message 5 "Saved %d file%s" len (if (= len 1) "" "s"))))
  
             (copy-file file to-file t t)))))
      (gnus-message 5 "Saved %d file%s" len (if (= len 1) "" "s"))))
  
@@ -707,49 +800,48 @@ The headers will be included in the sequence they are matched.")
  
  ;; Function called by gnus-uu-grab-articles to treat each article.
  (defun gnus-uu-save-article (buffer in-state)
  
  ;; Function called by gnus-uu-grab-articles to treat each article.
  (defun gnus-uu-save-article (buffer in-state)
-  (cond 
+  (cond
     (gnus-uu-save-separate-articles
      (save-excursion
        (set-buffer buffer)
     (gnus-uu-save-separate-articles
      (save-excursion
        (set-buffer buffer)
-      (write-region 1 (point-max) (concat gnus-uu-saved-article-name 
-                                         gnus-current-article))
+      (gnus-write-buffer
+       (concat gnus-uu-saved-article-name gnus-current-article))
        (cond ((eq in-state 'first) (list gnus-uu-saved-article-name 'begin))
        (cond ((eq in-state 'first) (list gnus-uu-saved-article-name 'begin))
-           ((eq in-state 'first-and-last) (list gnus-uu-saved-article-name 
+           ((eq in-state 'first-and-last) (list gnus-uu-saved-article-name
                                                  'begin 'end))
             ((eq in-state 'last) (list 'end))
             (t (list 'middle)))))
     ((not gnus-uu-save-in-digest)
      (save-excursion
        (set-buffer buffer)
                                                  'begin 'end))
             ((eq in-state 'last) (list 'end))
             (t (list 'middle)))))
     ((not gnus-uu-save-in-digest)
      (save-excursion
        (set-buffer buffer)
-      (write-region 1 (point-max) gnus-uu-saved-article-name t)
+      (write-region (point-min) (point-max) gnus-uu-saved-article-name t)
        (cond ((eq in-state 'first) (list gnus-uu-saved-article-name 'begin))
        (cond ((eq in-state 'first) (list gnus-uu-saved-article-name 'begin))
-           ((eq in-state 'first-and-last) (list gnus-uu-saved-article-name 
+           ((eq in-state 'first-and-last) (list gnus-uu-saved-article-name
                                                  'begin 'end))
             ((eq in-state 'last) (list 'end))
             (t (list 'middle)))))
     (t
      (let ((header (gnus-summary-article-header)))
                                                  'begin 'end))
             ((eq in-state 'last) (list 'end))
             (t (list 'middle)))))
     (t
      (let ((header (gnus-summary-article-header)))
-      (setq gnus-uu-digest-from-subject
-           (cons (cons (mail-header-from header)
-                       (mail-header-subject header))
-                 gnus-uu-digest-from-subject)))
+      (push (cons (mail-header-from header)
+                 (mail-header-subject header))
+           gnus-uu-digest-from-subject))
      (let ((name (file-name-nondirectory gnus-uu-saved-article-name))
           (delim (concat "^" (make-string 30 ?-) "$"))
           beg subj headers headline sorthead body end-string state)
      (let ((name (file-name-nondirectory gnus-uu-saved-article-name))
           (delim (concat "^" (make-string 30 ?-) "$"))
           beg subj headers headline sorthead body end-string state)
-      (if (or (eq in-state 'first) 
+      (if (or (eq in-state 'first)
               (eq in-state 'first-and-last))
               (eq in-state 'first-and-last))
-         (progn 
+         (progn
             (setq state (list 'begin))
             (save-excursion (set-buffer (get-buffer-create "*gnus-uu-body*"))
                             (erase-buffer))
             (setq state (list 'begin))
             (save-excursion (set-buffer (get-buffer-create "*gnus-uu-body*"))
                             (erase-buffer))
-           (save-excursion 
+           (save-excursion
               (set-buffer (get-buffer-create "*gnus-uu-pre*"))
               (erase-buffer)
               (set-buffer (get-buffer-create "*gnus-uu-pre*"))
               (erase-buffer)
-             (insert (format 
+             (insert (format
                        "Date: %s\nFrom: %s\nSubject: %s Digest\n\nTopics:\n"
                        (current-time-string) name name))))
                        "Date: %s\nFrom: %s\nSubject: %s Digest\n\nTopics:\n"
                        (current-time-string) name name))))
-       (if (not (eq in-state 'end))
-           (setq state (list 'middle))))
+       (when (not (eq in-state 'end))
+         (setq state (list 'middle))))
        (save-excursion
         (set-buffer (get-buffer "*gnus-uu-body*"))
         (goto-char (setq beg (point-max)))
        (save-excursion
         (set-buffer (get-buffer "*gnus-uu-body*"))
         (goto-char (setq beg (point-max)))
@@ -765,10 +857,10 @@ The headers will be included in the sequence they are matched.")
             (re-search-forward "\n\n")
             ;; Quote all 30-dash lines.
             (save-excursion
             (re-search-forward "\n\n")
             ;; Quote all 30-dash lines.
             (save-excursion
-             (while (re-search-forward delim nil t)
+             (while (re-search-forward "^-" nil t)
                 (beginning-of-line)
                 (delete-char 1)
                 (beginning-of-line)
                 (delete-char 1)
-               (insert " ")))
+               (insert "- ")))
             (setq body (buffer-substring (1- (point)) (point-max)))
             (narrow-to-region (point-min) (point))
             (if (not (setq headers gnus-uu-digest-headers))
             (setq body (buffer-substring (1- (point)) (point-max)))
             (narrow-to-region (point-min) (point))
             (if (not (setq headers gnus-uu-digest-headers))
@@ -778,9 +870,9 @@ The headers will be included in the sequence they are matched.")
                 (setq headers (cdr headers))
                 (goto-char (point-min))
                 (while (re-search-forward headline nil t)
                 (setq headers (cdr headers))
                 (goto-char (point-min))
                 (while (re-search-forward headline nil t)
-                 (setq sorthead 
+                 (setq sorthead
                         (concat sorthead
                         (concat sorthead
-                               (buffer-substring 
+                               (buffer-substring
                                  (match-beginning 0)
                                  (or (and (re-search-forward "^[^ \t]" nil t)
                                           (1- (point)))
                                  (match-beginning 0)
                                  (or (and (re-search-forward "^[^ \t]" nil t)
                                           (1- (point)))
@@ -790,41 +882,40 @@ The headers will be included in the sequence they are matched.")
         (insert body) (goto-char (point-max))
         (insert (concat "\n" (make-string 30 ?-) "\n\n"))
         (goto-char beg)
         (insert body) (goto-char (point-max))
         (insert (concat "\n" (make-string 30 ?-) "\n\n"))
         (goto-char beg)
-       (if (re-search-forward "^Subject: \\(.*\\)$" nil t)
-           (progn
-             (setq subj (buffer-substring (match-beginning 1) (match-end 1)))
-             (save-excursion 
-               (set-buffer (get-buffer "*gnus-uu-pre*"))
-               (insert (format "   %s\n" subj))))))
-      (if (or (eq in-state 'last)
-             (eq in-state 'first-and-last))
-         (progn
-           (save-excursion
-             (set-buffer (get-buffer "*gnus-uu-pre*"))
-             (insert (format "\n\n%s\n\n" (make-string 70 ?-)))
-             (write-region 1 (point-max) gnus-uu-saved-article-name))
-           (save-excursion
-             (set-buffer (get-buffer "*gnus-uu-body*"))
-             (goto-char (point-max))
-             (insert 
-              (concat (setq end-string (format "End of %s Digest" name)) 
-                      "\n"))
-             (insert (concat (make-string (length end-string) ?*) "\n"))
-             (write-region 1 (point-max) gnus-uu-saved-article-name t))
-           (kill-buffer (get-buffer "*gnus-uu-pre*"))
-           (kill-buffer (get-buffer "*gnus-uu-body*"))
-           (setq state (cons 'end state))))
+       (when (re-search-forward "^Subject: \\(.*\\)$" nil t)
+         (setq subj (buffer-substring (match-beginning 1) (match-end 1)))
+         (save-excursion
+           (set-buffer (get-buffer "*gnus-uu-pre*"))
+           (insert (format "   %s\n" subj)))))
+      (when (or (eq in-state 'last)
+               (eq in-state 'first-and-last))
+       (save-excursion
+         (set-buffer (get-buffer "*gnus-uu-pre*"))
+         (insert (format "\n\n%s\n\n" (make-string 70 ?-)))
+         (gnus-write-buffer gnus-uu-saved-article-name))
+       (save-excursion
+         (set-buffer (get-buffer "*gnus-uu-body*"))
+         (goto-char (point-max))
+         (insert
+          (concat (setq end-string (format "End of %s Digest" name))
+                  "\n"))
+         (insert (concat (make-string (length end-string) ?*) "\n"))
+         (write-region
+          (point-min) (point-max) gnus-uu-saved-article-name t))
+       (kill-buffer (get-buffer "*gnus-uu-pre*"))
+       (kill-buffer (get-buffer "*gnus-uu-body*"))
+       (push 'end state))
        (if (memq 'begin state)
           (cons gnus-uu-saved-article-name state)
         state)))))
  
        (if (memq 'begin state)
           (cons gnus-uu-saved-article-name state)
         state)))))
  
-;; Binhex treatment - not very advanced. 
+;; Binhex treatment - not very advanced.
  
  
-(defconst gnus-uu-binhex-body-line 
+(defvar gnus-uu-binhex-body-line
    "^[^:]...............................................................$")
    "^[^:]...............................................................$")
-(defconst gnus-uu-binhex-begin-line 
+(defvar gnus-uu-binhex-begin-line
    "^:...............................................................$")
    "^:...............................................................$")
-(defconst gnus-uu-binhex-end-line
+(defvar gnus-uu-binhex-end-line
    ":$")
  
  (defun gnus-uu-binhex-article (buffer in-state)
    ":$")
  
  (defun gnus-uu-binhex-article (buffer in-state)
@@ -833,9 +924,9 @@ The headers will be included in the sequence they are matched.")
        (set-buffer buffer)
        (widen)
        (goto-char (point-min))
        (set-buffer buffer)
        (widen)
        (goto-char (point-min))
-      (if (not (re-search-forward gnus-uu-binhex-begin-line nil t))
-         (if (not (re-search-forward gnus-uu-binhex-body-line nil t))
-             (setq state (list 'wrong-type))))
+      (when (not (re-search-forward gnus-uu-binhex-begin-line nil t))
+       (when (not (re-search-forward gnus-uu-binhex-body-line nil t))
+         (setq state (list 'wrong-type))))
  
        (if (memq 'wrong-type state)
           ()
  
        (if (memq 'wrong-type state)
           ()
@@ -847,16 +938,17 @@ The headers will be included in the sequence they are matched.")
               (write-region 1 1 gnus-uu-binhex-article-name))
           (setq state (list 'middle)))
         (goto-char (point-max))
               (write-region 1 1 gnus-uu-binhex-article-name))
           (setq state (list 'middle)))
         (goto-char (point-max))
-       (re-search-backward (concat gnus-uu-binhex-body-line "\\|" 
-                                   gnus-uu-binhex-end-line) nil t)
-       (if (looking-at gnus-uu-binhex-end-line)
-           (setq state (if (memq 'begin state)
-                           (cons 'end state)
-                         (list 'end))))
+       (re-search-backward (concat gnus-uu-binhex-body-line "\\|"
+                                   gnus-uu-binhex-end-line)
+                           nil t)
+       (when (looking-at gnus-uu-binhex-end-line)
+         (setq state (if (memq 'begin state)
+                         (cons 'end state)
+                       (list 'end))))
         (beginning-of-line)
         (forward-line 1)
         (beginning-of-line)
         (forward-line 1)
-       (if (file-exists-p gnus-uu-binhex-article-name)
-           (append-to-file start-char (point) gnus-uu-binhex-article-name))))
+       (when (file-exists-p gnus-uu-binhex-article-name)
+         (append-to-file start-char (point) gnus-uu-binhex-article-name))))
      (if (memq 'begin state)
         (cons gnus-uu-binhex-article-name state)
        state)))
      (if (memq 'begin state)
         (cons gnus-uu-binhex-article-name state)
        state)))
@@ -883,7 +975,7 @@ The headers will be included in the sequence they are matched.")
           (write-region (point-min) (point-max) file-name)
           (setq state (list file-name 'begin 'end)))))
      state))
           (write-region (point-min) (point-max) file-name)
           (setq state (list file-name 'begin 'end)))))
      state))
-      
+
  
  ;; Find actions.
  
  
  ;; Find actions.
  
@@ -892,7 +984,7 @@ The headers will be included in the sequence they are matched.")
         action name)
      (while files
        (setq name (cdr (assq 'name (car files))))
         action name)
      (while files
        (setq name (cdr (assq 'name (car files))))
-      (and 
+      (and
         (setq action (gnus-uu-get-action name))
         (setcar files (nconc (list (if (string= action "gnus-uu-archive")
                                       (cons 'action "file")
         (setq action (gnus-uu-get-action name))
         (setcar files (nconc (list (if (string= action "gnus-uu-archive")
                                       (cons 'action "file")
@@ -905,20 +997,20 @@ The headers will be included in the sequence they are matched.")
  
  (defun gnus-uu-get-action (file-name)
    (let (action)
  
  (defun gnus-uu-get-action (file-name)
    (let (action)
-    (setq action 
-         (gnus-uu-choose-action 
+    (setq action
+         (gnus-uu-choose-action
            file-name
            file-name
-          (append 
+          (append
             gnus-uu-user-view-rules
             gnus-uu-user-view-rules
-           (if gnus-uu-ignore-default-view-rules 
-               nil 
+           (if gnus-uu-ignore-default-view-rules
+               nil
               gnus-uu-default-view-rules)
             gnus-uu-user-view-rules-end)))
               gnus-uu-default-view-rules)
             gnus-uu-user-view-rules-end)))
-    (if (and (not (string= (or action "") "gnus-uu-archive")) 
-            gnus-uu-view-with-metamail)
-       (if (setq action 
+    (when (and (not (string= (or action "") "gnus-uu-archive"))
+              gnus-uu-view-with-metamail)
+      (when (setq action
                   (gnus-uu-choose-action file-name gnus-uu-ext-to-mime-list))
                   (gnus-uu-choose-action file-name gnus-uu-ext-to-mime-list))
-           (setq action (format "metamail -d -b -c \"%s\"" action))))
+       (setq action (format "metamail -d -b -c \"%s\"" action))))
      action))
  
  
      action))
  
  
@@ -929,7 +1021,7 @@ The headers will be included in the sequence they are matched.")
    ;; ignores any leading "version numbers" thingies that they use in
    ;; the comp.binaries groups, and either replaces anything that looks
    ;; like "2/3" with "[0-9]+/[0-9]+" or, if it can't find something
    ;; ignores any leading "version numbers" thingies that they use in
    ;; the comp.binaries groups, and either replaces anything that looks
    ;; like "2/3" with "[0-9]+/[0-9]+" or, if it can't find something
-  ;; like that, replaces the last two numbers with "[0-9]+". This, in
+  ;; like that, replaces the last two numbers with "[0-9]+".  This, in
    ;; my experience, should get most postings of a series.
    (let ((count 2)
         (vernum "v[0-9]+[a-z][0-9]+:")
    ;; my experience, should get most postings of a series.
    (let ((count 2)
         (vernum "v[0-9]+[a-z][0-9]+:")
@@ -943,10 +1035,9 @@ The headers will be included in the sequence they are matched.")
  
        (setq case-fold-search nil)
        (goto-char (point-min))
  
        (setq case-fold-search nil)
        (goto-char (point-min))
-      (if (looking-at vernum)
-         (progn
-           (replace-match vernum t t)
-           (setq beg (length vernum))))
+      (when (looking-at vernum)
+       (replace-match vernum t t)
+       (setq beg (length vernum)))
  
        (goto-char beg)
        (if (re-search-forward "[ \t]*[0-9]+/[0-9]+" nil t)
  
        (goto-char beg)
        (if (re-search-forward "[ \t]*[0-9]+/[0-9]+" nil t)
@@ -957,14 +1048,9 @@ The headers will be included in the sequence they are matched.")
             (replace-match "[0-9]+ of [0-9]+")
  
           (end-of-line)
             (replace-match "[0-9]+ of [0-9]+")
  
           (end-of-line)
-         (while (and (re-search-backward "[0-9]" nil t) (> count 0))
-            (while (and 
-                   (looking-at "[0-9]") 
-                   (< 1 (goto-char (1- (point))))))
-            (re-search-forward "[0-9]+" nil t)
-           (replace-match "[0-9]+")
-           (backward-char 5)
-           (setq count (1- count)))))
+          (if (re-search-backward "\\([^0-9]\\)[0-9]+\\([^0-9]+\\)[0-9]+"
+                                  nil t)
+              (replace-match "\\1[0-9]+\\2[0-9]+" t nil nil nil))))
  
        (goto-char beg)
        (while (re-search-forward "[ \t]+" nil t)
  
        (goto-char beg)
        (while (re-search-forward "[ \t]+" nil t)
@@ -976,18 +1062,19 @@ The headers will be included in the sequence they are matched.")
    ;; If N is non-nil, the article numbers of the N next articles
    ;; will be returned.
    ;; If any articles have been marked as processable, they will be
    ;; If N is non-nil, the article numbers of the N next articles
    ;; will be returned.
    ;; If any articles have been marked as processable, they will be
-  ;; returned. 
+  ;; returned.
    ;; Failing that, articles that have subjects that are part of the
    ;; same "series" as the current will be returned.
    (let (articles)
    ;; Failing that, articles that have subjects that are part of the
    ;; same "series" as the current will be returned.
    (let (articles)
-    (cond 
+    (cond
       (n
       (n
+      (setq n (prefix-numeric-value n))
        (let ((backward (< n 0))
             (n (abs n)))
         (save-excursion
           (while (and (> n 0)
        (let ((backward (< n 0))
             (n (abs n)))
         (save-excursion
           (while (and (> n 0)
-                     (setq articles (cons (gnus-summary-article-number) 
-                                          articles))
+                     (push (gnus-summary-article-number)
+                           articles)
                       (gnus-summary-search-forward nil nil backward))
             (setq n (1- n))))
         (nreverse articles)))
                       (gnus-summary-search-forward nil nil backward))
             (setq n (1- n))))
         (nreverse articles)))
@@ -999,13 +1086,13 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-string< (l1 l2)
    (string< (car l1) (car l2)))
  
  (defun gnus-uu-string< (l1 l2)
    (string< (car l1) (car l2)))
  
-(defun gnus-uu-find-articles-matching 
+(defun gnus-uu-find-articles-matching
    (&optional subject only-unread do-not-translate)
    ;; Finds all articles that matches the regexp SUBJECT.  If it is
    (&optional subject only-unread do-not-translate)
    ;; Finds all articles that matches the regexp SUBJECT.  If it is
-  ;; nil, the current article name will be used. If ONLY-UNREAD is
-  ;; non-nil, only unread articles are chosen. If DO-NOT-TRANSLATE is
+  ;; nil, the current article name will be used.  If ONLY-UNREAD is
+  ;; non-nil, only unread articles are chosen.  If DO-NOT-TRANSLATE is
    ;; non-nil, article names are not equalized before sorting.
    ;; non-nil, article names are not equalized before sorting.
-  (let ((subject (or subject 
+  (let ((subject (or subject
                      (gnus-uu-reginize-string (gnus-summary-article-subject))))
         list-of-subjects)
      (save-excursion
                      (gnus-uu-reginize-string (gnus-summary-article-subject))))
         list-of-subjects)
      (save-excursion
@@ -1025,24 +1112,23 @@ The headers will be included in the sequence they are matched.")
                      (= mark gnus-dormant-mark))
                  (setq subj (mail-header-subject (gnus-data-header d)))
                  (string-match subject subj)
                      (= mark gnus-dormant-mark))
                  (setq subj (mail-header-subject (gnus-data-header d)))
                  (string-match subject subj)
-                (setq list-of-subjects 
-                      (cons (cons subj (gnus-data-number d))
-                            list-of-subjects)))))
+                (push (cons subj (gnus-data-number d))
+                      list-of-subjects))))
  
         ;; Expand numbers, sort, and return the list of article
         ;; numbers.
  
         ;; Expand numbers, sort, and return the list of article
         ;; numbers.
-       (mapcar (lambda (sub) (cdr sub)) 
-               (sort (gnus-uu-expand-numbers 
+       (mapcar (lambda (sub) (cdr sub))
+               (sort (gnus-uu-expand-numbers
                        list-of-subjects
                        list-of-subjects
-                      (not do-not-translate)) 
+                      (not do-not-translate))
                       'gnus-uu-string<))))))
  
  (defun gnus-uu-expand-numbers (string-list &optional translate)
    ;; Takes a list of strings and "expands" all numbers in all the
    ;; strings.  That is, this function makes all numbers equal length by
                       'gnus-uu-string<))))))
  
  (defun gnus-uu-expand-numbers (string-list &optional translate)
    ;; Takes a list of strings and "expands" all numbers in all the
    ;; strings.  That is, this function makes all numbers equal length by
-  ;; prepending lots of zeroes before each number. This is to ease later
+  ;; prepending lots of zeroes before each number.  This is to ease later
    ;; sorting to find out what sequence the articles are supposed to be
    ;; sorting to find out what sequence the articles are supposed to be
-  ;; decoded in. Returns the list of expanded strings.
+  ;; decoded in.  Returns the list of expanded strings.
    (let ((out-list string-list)
         string)
      (save-excursion
    (let ((out-list string-list)
         string)
      (save-excursion
@@ -1057,15 +1143,15 @@ The headers will be included in the sequence they are matched.")
           (replace-match " "))
         ;; Translate all characters to "a".
         (goto-char (point-min))
           (replace-match " "))
         ;; Translate all characters to "a".
         (goto-char (point-min))
-       (if translate 
-           (while (re-search-forward "[A-Za-z]" nil t)
-             (replace-match "a" t t)))
+       (when translate
+         (while (re-search-forward "[A-Za-z]" nil t)
+           (replace-match "a" t t)))
         ;; Expand numbers.
         (goto-char (point-min))
         (while (re-search-forward "[0-9]+" nil t)
         ;; Expand numbers.
         (goto-char (point-min))
         (while (re-search-forward "[0-9]+" nil t)
-         (replace-match  
-          (format "%06d" 
-                  (string-to-int (buffer-substring 
+         (replace-match
+          (format "%06d"
+                  (string-to-int (buffer-substring
                                    (match-beginning 0) (match-end 0))))))
         (setq string (buffer-substring 1 (point-max)))
         (setcar (car string-list) string)
                                    (match-beginning 0) (match-end 0))))))
         (setq string (buffer-substring 1 (point-max)))
         (setcar (car string-list) string)
@@ -1078,14 +1164,14 @@ The headers will be included in the sequence they are matched.")
  ;; to apply to each article.
  ;;
  ;; The function to be called should take two parameters.  The first
  ;; to apply to each article.
  ;;
  ;; The function to be called should take two parameters.  The first
-;; parameter is the article buffer. The function should leave the
-;; result, if any, in this buffer. Most treatment functions will just
+;; parameter is the article buffer.  The function should leave the
+;; result, if any, in this buffer.  Most treatment functions will just
  ;; generate files...
  ;;
  ;; The second parameter is the state of the list of articles, and can
  ;; have four values: `first', `middle', `last' and `first-and-last'.
  ;;
  ;; generate files...
  ;;
  ;; The second parameter is the state of the list of articles, and can
  ;; have four values: `first', `middle', `last' and `first-and-last'.
  ;;
-;; The function should return a list. The list may contain the
+;; The function should return a list.  The list may contain the
  ;; following symbols:
  ;; `error' if an error occurred
  ;; `begin' if the beginning of an encoded file has been received
  ;; following symbols:
  ;; `error' if an error occurred
  ;; `begin' if the beginning of an encoded file has been received
@@ -1104,30 +1190,30 @@ The headers will be included in the sequence they are matched.")
      (if (not (and gnus-uu-has-been-grabbed
                   gnus-uu-unmark-articles-not-decoded))
         ()
      (if (not (and gnus-uu-has-been-grabbed
                   gnus-uu-unmark-articles-not-decoded))
         ()
-      (if dont-unmark-last-article
-         (progn
-           (setq art (car gnus-uu-has-been-grabbed))
-           (setq gnus-uu-has-been-grabbed (cdr gnus-uu-has-been-grabbed))))
+      (when dont-unmark-last-article
+       (setq art (car gnus-uu-has-been-grabbed))
+       (setq gnus-uu-has-been-grabbed (cdr gnus-uu-has-been-grabbed)))
        (while gnus-uu-has-been-grabbed
         (gnus-summary-tick-article (car gnus-uu-has-been-grabbed) t)
         (setq gnus-uu-has-been-grabbed (cdr gnus-uu-has-been-grabbed)))
        (while gnus-uu-has-been-grabbed
         (gnus-summary-tick-article (car gnus-uu-has-been-grabbed) t)
         (setq gnus-uu-has-been-grabbed (cdr gnus-uu-has-been-grabbed)))
-      (if dont-unmark-last-article
-         (setq gnus-uu-has-been-grabbed (list art))))))
+      (when dont-unmark-last-article
+       (setq gnus-uu-has-been-grabbed (list art))))))
  
  ;; This function takes a list of articles and a function to apply to
  
  ;; This function takes a list of articles and a function to apply to
-;; each article grabbed. 
-;; 
+;; each article grabbed.
+;;
  ;; This function returns a list of files decoded if the grabbing and
  ;; the process-function has been successful and nil otherwise.
  ;; This function returns a list of files decoded if the grabbing and
  ;; the process-function has been successful and nil otherwise.
-(defun gnus-uu-grab-articles (articles process-function 
+(defun gnus-uu-grab-articles (articles process-function
                                        &optional sloppy limit no-errors)
                                        &optional sloppy limit no-errors)
-  (let ((state 'first) 
+  (let ((state 'first)
+       (gnus-asynchronous nil)
         has-been-begin article result-file result-files process-state
         gnus-summary-display-article-function
         gnus-article-display-hook gnus-article-prepare-hook
         article-series files)
         has-been-begin article result-file result-files process-state
         gnus-summary-display-article-function
         gnus-article-display-hook gnus-article-prepare-hook
         article-series files)
- 
-    (while (and articles 
+
+    (while (and articles
                 (not (memq 'error process-state))
                 (or sloppy
                     (not (memq 'end process-state))))
                 (not (memq 'error process-state))
                 (or sloppy
                     (not (memq 'end process-state))))
@@ -1135,40 +1221,45 @@ The headers will be included in the sequence they are matched.")
        (setq article (pop articles))
        (push article article-series)
  
        (setq article (pop articles))
        (push article article-series)
  
-      (unless articles 
+      (unless articles
         (if (eq state 'first)
             (setq state 'first-and-last)
           (setq state 'last)))
  
        (let ((part (gnus-uu-part-number article)))
         (if (eq state 'first)
             (setq state 'first-and-last)
           (setq state 'last)))
  
        (let ((part (gnus-uu-part-number article)))
-       (gnus-message 6 "Getting article %d%s..." 
+       (gnus-message 6 "Getting article %d%s..."
                       article (if (string= part "") "" (concat ", " part))))
        (gnus-summary-display-article article)
                       article (if (string= part "") "" (concat ", " part))))
        (gnus-summary-display-article article)
-      
+
        ;; Push the article to the processing function.
        (save-excursion
         (set-buffer gnus-original-article-buffer)
         (let ((buffer-read-only nil))
           (save-excursion
             (set-buffer gnus-summary-buffer)
        ;; Push the article to the processing function.
        (save-excursion
         (set-buffer gnus-original-article-buffer)
         (let ((buffer-read-only nil))
           (save-excursion
             (set-buffer gnus-summary-buffer)
-           (setq process-state 
+           (setq process-state
                   (funcall process-function
                            gnus-original-article-buffer state)))))
  
        (gnus-summary-remove-process-mark article)
  
                   (funcall process-function
                            gnus-original-article-buffer state)))))
  
        (gnus-summary-remove-process-mark article)
  
-      ;; If this is the beginning of a decoded file, we push it 
+      ;; If this is the beginning of a decoded file, we push it
        ;; on to a list.
        (when (or (memq 'begin process-state)
        ;; on to a list.
        (when (or (memq 'begin process-state)
-               (and (or (eq state 'first) 
+               (and (or (eq state 'first)
                          (eq state 'first-and-last))
                      (memq 'ok process-state)))
                          (eq state 'first-and-last))
                      (memq 'ok process-state)))
-       (if has-been-begin
-           ;; If there is a `result-file' here, that means that the
-           ;; file was unsuccessfully decoded, so we delete it.
-           (when (and result-file 
-                      (file-exists-p result-file)) 
-             (delete-file result-file)))
+       (when has-been-begin
+         ;; If there is a `result-file' here, that means that the
+         ;; file was unsuccessfully decoded, so we delete it.
+         (when (and result-file
+                    (file-exists-p result-file)
+                    (not gnus-uu-be-dangerous)
+                    (or (eq gnus-uu-be-dangerous t)
+                        (gnus-y-or-n-p
+                         (format "Delete unsuccessfully decoded file %s"
+                                 result-file))))
+           (delete-file result-file)))
         (when (memq 'begin process-state)
           (setq result-file (car process-state)))
         (setq has-been-begin t))
         (when (memq 'begin process-state)
           (setq result-file (car process-state)))
         (setq has-been-begin t))
@@ -1192,6 +1283,7 @@ The headers will be included in the sequence they are matched.")
               (setq funcs (list funcs)))
             (while funcs
               (funcall (pop funcs) result-file))))
               (setq funcs (list funcs)))
             (while funcs
               (funcall (pop funcs) result-file))))
+       (setq result-file nil)
         ;; Check whether we have decoded enough articles.
         (and limit (= (length result-files) limit)
              (setq articles nil)))
         ;; Check whether we have decoded enough articles.
         (and limit (= (length result-files) limit)
              (setq articles nil)))
@@ -1201,11 +1293,14 @@ The headers will be included in the sequence they are matched.")
        ;; the partially decoded file.
        (and (or (eq state 'last) (eq state 'first-and-last))
            (not (memq 'end process-state))
        ;; the partially decoded file.
        (and (or (eq state 'last) (eq state 'first-and-last))
            (not (memq 'end process-state))
-          result-file 
+          result-file
            (file-exists-p result-file)
            (file-exists-p result-file)
+          (not gnus-uu-be-dangerous)
+          (or (eq gnus-uu-be-dangerous t)
+              (gnus-y-or-n-p (format "Delete incomplete file %s? " result-file)))
            (delete-file result-file))
  
            (delete-file result-file))
  
-      ;; If this was a file of the wrong sort, then 
+      ;; If this was a file of the wrong sort, then
        (when (and (or (memq 'wrong-type process-state)
                      (memq 'error process-state))
                  gnus-uu-unmark-articles-not-decoded)
        (when (and (or (memq 'wrong-type process-state)
                      (memq 'error process-state))
                  gnus-uu-unmark-articles-not-decoded)
@@ -1230,7 +1325,7 @@ The headers will be included in the sequence they are matched.")
         (gnus-message 2 "Wrong type file"))
         ((memq 'error process-state)
         (gnus-message 2 "An error occurred during decoding"))
         (gnus-message 2 "Wrong type file"))
         ((memq 'error process-state)
         (gnus-message 2 "An error occurred during decoding"))
-       ((not (or (memq 'ok process-state) 
+       ((not (or (memq 'ok process-state)
                  (memq 'end process-state)))
         (gnus-message 2 "End of articles reached before end of file")))
        ;; Make unsuccessfully decoded articles unread.
                  (memq 'end process-state)))
         (gnus-message 2 "End of articles reached before end of file")))
        ;; Make unsuccessfully decoded articles unread.
@@ -1261,7 +1356,7 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-part-number (article)
    (let* ((header (gnus-summary-article-header article))
          (subject (and header (mail-header-subject header))))
  (defun gnus-uu-part-number (article)
    (let* ((header (gnus-summary-article-header article))
          (subject (and header (mail-header-subject header))))
-    (if (and subject 
+    (if (and subject
              (string-match "[0-9]+ */[0-9]+\\|[0-9]+ * of *[0-9]+" subject))
         (match-string 0 subject)
        "")))
              (string-match "[0-9]+ */[0-9]+\\|[0-9]+ * of *[0-9]+" subject))
         (match-string 0 subject)
        "")))
@@ -1274,7 +1369,7 @@ The headers will be included in the sequence they are matched.")
    (save-excursion
      (set-buffer process-buffer)
      (let ((state (list 'wrong-type))
    (save-excursion
      (set-buffer process-buffer)
      (let ((state (list 'wrong-type))
-         process-connection-type case-fold-search buffer-read-only 
+         process-connection-type case-fold-search buffer-read-only
           files start-char)
        (goto-char (point-min))
  
           files start-char)
        (goto-char (point-min))
  
@@ -1293,12 +1388,13 @@ The headers will be included in the sequence they are matched.")
  
         (if (not (looking-at gnus-uu-begin-string))
             (setq state (list 'middle))
  
         (if (not (looking-at gnus-uu-begin-string))
             (setq state (list 'middle))
-         ;; This is the beginning of an uuencoded article.
+         ;; This is the beginning of a uuencoded article.
           ;; We replace certain characters that could make things messy.
           ;; We replace certain characters that could make things messy.
-         (setq gnus-uu-file-name 
+         (setq gnus-uu-file-name
                 (let ((nnheader-file-name-translation-alist
                        '((?/ . ?,) (? . ?_) (?* . ?_) (?$ . ?_))))
                   (nnheader-translate-file-chars (match-string 1))))
                 (let ((nnheader-file-name-translation-alist
                        '((?/ . ?,) (? . ?_) (?* . ?_) (?$ . ?_))))
                   (nnheader-translate-file-chars (match-string 1))))
+          (replace-match (concat "begin 644 " gnus-uu-file-name) t t)
  
           ;; Remove any non gnus-uu-body-line right after start.
           (forward-line 1)
  
           ;; Remove any non gnus-uu-body-line right after start.
           (forward-line 1)
@@ -1308,7 +1404,7 @@ The headers will be included in the sequence they are matched.")
  
           ;; If a process is running, we kill it.
           (when (and gnus-uu-uudecode-process
  
           ;; If a process is running, we kill it.
           (when (and gnus-uu-uudecode-process
-                    (memq (process-status gnus-uu-uudecode-process) 
+                    (memq (process-status gnus-uu-uudecode-process)
                            '(run stop)))
             (delete-process gnus-uu-uudecode-process)
             (gnus-uu-unmark-list-of-grabbed t))
                            '(run stop)))
             (delete-process gnus-uu-uudecode-process)
             (gnus-uu-unmark-list-of-grabbed t))
@@ -1319,24 +1415,24 @@ The headers will be included in the sequence they are matched.")
                 (progn
                   (cd gnus-uu-work-dir)
                   (setq gnus-uu-uudecode-process
                 (progn
                   (cd gnus-uu-work-dir)
                   (setq gnus-uu-uudecode-process
-                       (start-process 
-                        "*uudecode*" 
+                       (start-process
+                        "*uudecode*"
                          (get-buffer-create gnus-uu-output-buffer-name)
                          shell-file-name shell-command-switch
                          (format "cd %s %s uudecode" gnus-uu-work-dir
                                  gnus-shell-command-separator))))
               (cd cdir)))
                          (get-buffer-create gnus-uu-output-buffer-name)
                          shell-file-name shell-command-switch
                          (format "cd %s %s uudecode" gnus-uu-work-dir
                                  gnus-shell-command-separator))))
               (cd cdir)))
-         (set-process-sentinel 
+         (set-process-sentinel
            gnus-uu-uudecode-process 'gnus-uu-uudecode-sentinel)
           (setq state (list 'begin))
           (push (concat gnus-uu-work-dir gnus-uu-file-name) files))
            gnus-uu-uudecode-process 'gnus-uu-uudecode-sentinel)
           (setq state (list 'begin))
           (push (concat gnus-uu-work-dir gnus-uu-file-name) files))
-       
+
         ;; We look for the end of the thing to be decoded.
         (if (re-search-forward gnus-uu-end-string nil t)
         ;; We look for the end of the thing to be decoded.
         (if (re-search-forward gnus-uu-end-string nil t)
-           (setq state (cons 'end state))
+           (push 'end state)
           (goto-char (point-max))
           (re-search-backward gnus-uu-body-line nil t))
           (goto-char (point-max))
           (re-search-backward gnus-uu-body-line nil t))
-        
+
         (forward-line 1)
  
         (when gnus-uu-uudecode-process
         (forward-line 1)
  
         (when gnus-uu-uudecode-process
@@ -1344,13 +1440,14 @@ The headers will be included in the sequence they are matched.")
             ;; Try to correct mishandled uucode.
             (when gnus-uu-correct-stripped-uucode
               (gnus-uu-check-correct-stripped-uucode start-char (point)))
             ;; Try to correct mishandled uucode.
             (when gnus-uu-correct-stripped-uucode
               (gnus-uu-check-correct-stripped-uucode start-char (point)))
+           (run-hooks 'gnus-uu-pre-uudecode-hook)
  
             ;; Send the text to the process.
             (condition-case nil
                 (process-send-region
                  gnus-uu-uudecode-process start-char (point))
  
             ;; Send the text to the process.
             (condition-case nil
                 (process-send-region
                  gnus-uu-uudecode-process start-char (point))
-             (error 
-              (progn 
+             (error
+              (progn
                  (delete-process gnus-uu-uudecode-process)
                  (gnus-message 2 "gnus-uu: Couldn't uudecode")
                  (setq state (list 'wrong-type)))))
                  (delete-process gnus-uu-uudecode-process)
                  (gnus-message 2 "gnus-uu: Couldn't uudecode")
                  (setq state (list 'wrong-type)))))
@@ -1358,9 +1455,8 @@ The headers will be included in the sequence they are matched.")
             (if (memq 'end state)
                 (progn
                   ;; Send an EOF, just in case.
             (if (memq 'end state)
                 (progn
                   ;; Send an EOF, just in case.
-                 (condition-case ()
-                     (process-send-eof gnus-uu-uudecode-process)
-                   (error nil))
+                 (ignore-errors
+                   (process-send-eof gnus-uu-uudecode-process))
                   (while (memq (process-status gnus-uu-uudecode-process)
                                '(open run))
                     (accept-process-output gnus-uu-uudecode-process 1)))
                   (while (memq (process-status gnus-uu-uudecode-process)
                                '(open run))
                     (accept-process-output gnus-uu-uudecode-process 1)))
@@ -1385,10 +1481,12 @@ The headers will be included in the sequence they are matched.")
           (setq state (list 'wrong-type))
         (beginning-of-line)
         (setq start-char (point))
           (setq state (list 'wrong-type))
         (beginning-of-line)
         (setq start-char (point))
-       (call-process-region 
-        start-char (point-max) shell-file-name nil 
-        (get-buffer-create gnus-uu-output-buffer-name) nil 
-        shell-command-switch (concat "cd " gnus-uu-work-dir " ; sh"))))
+       (call-process-region
+        start-char (point-max) shell-file-name nil
+        (get-buffer-create gnus-uu-output-buffer-name) nil
+        shell-command-switch
+        (concat "cd " gnus-uu-work-dir " "
+                gnus-shell-command-separator  " sh"))))
      state))
  
  ;; Returns the name of what the shar file is going to unpack.
      state))
  
  ;; Returns the name of what the shar file is going to unpack.
@@ -1396,8 +1494,8 @@ The headers will be included in the sequence they are matched.")
    (let ((oldpoint (point))
         res)
      (goto-char (point-min))
    (let ((oldpoint (point))
         res)
      (goto-char (point-min))
-    (if (re-search-forward gnus-uu-shar-name-marker nil t)
-       (setq res (buffer-substring (match-beginning 1) (match-end 1))))
+    (when (re-search-forward gnus-uu-shar-name-marker nil t)
+      (setq res (buffer-substring (match-beginning 1) (match-end 1))))
      (goto-char oldpoint)
      res))
  
      (goto-char oldpoint)
      res))
  
@@ -1408,41 +1506,42 @@ The headers will be included in the sequence they are matched.")
    (let ((action-list (copy-sequence file-action-list))
         (case-fold-search t)
         rule action)
    (let ((action-list (copy-sequence file-action-list))
         (case-fold-search t)
         rule action)
-    (and 
-     (or no-ignore 
-        (and (not 
-              (and gnus-uu-ignore-files-by-name
-                   (string-match gnus-uu-ignore-files-by-name file-name)))
-             (not 
-              (and gnus-uu-ignore-files-by-type
-                   (string-match gnus-uu-ignore-files-by-type 
-                                 (or (gnus-uu-choose-action 
-                                      file-name gnus-uu-ext-to-mime-list t) 
-                                     ""))))))
+    (and
+     (unless no-ignore
+       (and (not
+            (and gnus-uu-ignore-files-by-name
+                 (string-match gnus-uu-ignore-files-by-name file-name)))
+           (not
+            (and gnus-uu-ignore-files-by-type
+                 (string-match gnus-uu-ignore-files-by-type
+                               (or (gnus-uu-choose-action
+                                    file-name gnus-uu-ext-to-mime-list t)
+                                   ""))))))
       (while (not (or (eq action-list ()) action))
         (setq rule (car action-list))
         (setq action-list (cdr action-list))
       (while (not (or (eq action-list ()) action))
         (setq rule (car action-list))
         (setq action-list (cdr action-list))
-       (if (string-match (car rule) file-name)
-          (setq action (cadr rule)))))
+       (when (string-match (car rule) file-name)
+        (setq action (cadr rule)))))
      action))
  
  (defun gnus-uu-treat-archive (file-path)
      action))
  
  (defun gnus-uu-treat-archive (file-path)
-  ;; Unpacks an archive. Returns t if unpacking is successful.
+  ;; Unpacks an archive.  Returns t if unpacking is successful.
    (let ((did-unpack t)
         action command dir)
    (let ((did-unpack t)
         action command dir)
-    (setq action (gnus-uu-choose-action 
+    (setq action (gnus-uu-choose-action
                   file-path (append gnus-uu-user-archive-rules
                                     (if gnus-uu-ignore-default-archive-rules
                                         nil
                                       gnus-uu-default-archive-rules))))
  
                   file-path (append gnus-uu-user-archive-rules
                                     (if gnus-uu-ignore-default-archive-rules
                                         nil
                                       gnus-uu-default-archive-rules))))
  
-    (if (not action) (error "No unpackers for the file %s" file-path))
+    (when (not action)
+      (error "No unpackers for the file %s" file-path))
  
      (string-match "/[^/]*$" file-path)
      (setq dir (substring file-path 0 (match-beginning 0)))
  
  
      (string-match "/[^/]*$" file-path)
      (setq dir (substring file-path 0 (match-beginning 0)))
  
-    (if (member action gnus-uu-destructive-archivers)
-       (copy-file file-path (concat file-path "~") t))
+    (when (member action gnus-uu-destructive-archivers)
+      (copy-file file-path (concat file-path "~") t))
  
      (setq command (format "cd %s ; %s" dir (gnus-uu-command action file-path)))
  
  
      (setq command (format "cd %s ; %s" dir (gnus-uu-command action file-path)))
  
@@ -1452,15 +1551,15 @@ The headers will be included in the sequence they are matched.")
  
      (gnus-message 5 "Unpacking: %s..." (gnus-uu-command action file-path))
  
  
      (gnus-message 5 "Unpacking: %s..." (gnus-uu-command action file-path))
  
-    (if (= 0 (call-process shell-file-name nil 
+    (if (= 0 (call-process shell-file-name nil
                            (get-buffer-create gnus-uu-output-buffer-name)
                            nil shell-command-switch command))
         (message "")
        (gnus-message 2 "Error during unpacking of archive")
        (setq did-unpack nil))
  
                            (get-buffer-create gnus-uu-output-buffer-name)
                            nil shell-command-switch command))
         (message "")
        (gnus-message 2 "Error during unpacking of archive")
        (setq did-unpack nil))
  
-    (if (member action gnus-uu-destructive-archivers)
-       (rename-file (concat file-path "~") file-path t))
+    (when (member action gnus-uu-destructive-archivers)
+      (rename-file (concat file-path "~") file-path t))
  
      did-unpack))
  
  
      did-unpack))
  
@@ -1470,35 +1569,34 @@ The headers will be included in the sequence they are matched.")
      (while dirs
        (if (file-directory-p (setq file (car dirs)))
           (setq files (append files (gnus-uu-dir-files file)))
      (while dirs
        (if (file-directory-p (setq file (car dirs)))
           (setq files (append files (gnus-uu-dir-files file)))
-       (setq files (cons file files)))
+       (push file files))
        (setq dirs (cdr dirs)))
      files))
  
  (defun gnus-uu-unpack-files (files &optional ignore)
        (setq dirs (cdr dirs)))
      files))
  
  (defun gnus-uu-unpack-files (files &optional ignore)
-  ;; Go through FILES and look for files to unpack. 
+  ;; Go through FILES and look for files to unpack.
    (let* ((totfiles (gnus-uu-ls-r gnus-uu-work-dir))
          (ofiles files)
          file did-unpack)
      (while files
        (setq file (cdr (assq 'name (car files))))
    (let* ((totfiles (gnus-uu-ls-r gnus-uu-work-dir))
          (ofiles files)
          file did-unpack)
      (while files
        (setq file (cdr (assq 'name (car files))))
-      (if (and (not (member file ignore))
-              (equal (gnus-uu-get-action (file-name-nondirectory file))
-                     "gnus-uu-archive"))
-         (progn
-           (setq did-unpack (cons file did-unpack))
-           (or (gnus-uu-treat-archive file)
-               (gnus-message 2 "Error during unpacking of %s" file))
-           (let* ((newfiles (gnus-uu-ls-r gnus-uu-work-dir))
-                  (nfiles newfiles))
-             (while nfiles
-               (or (member (car nfiles) totfiles)
-                   (setq ofiles (cons (list (cons 'name (car nfiles))
-                                            (cons 'original file))
-                                      ofiles)))
-               (setq nfiles (cdr nfiles)))
-             (setq totfiles newfiles))))
+      (when (and (not (member file ignore))
+                (equal (gnus-uu-get-action (file-name-nondirectory file))
+                       "gnus-uu-archive"))
+       (push file did-unpack)
+       (unless (gnus-uu-treat-archive file)
+         (gnus-message 2 "Error during unpacking of %s" file))
+       (let* ((newfiles (gnus-uu-ls-r gnus-uu-work-dir))
+              (nfiles newfiles))
+         (while nfiles
+           (unless (member (car nfiles) totfiles)
+             (push (list (cons 'name (car nfiles))
+                         (cons 'original file))
+                   ofiles))
+           (setq nfiles (cdr nfiles)))
+         (setq totfiles newfiles)))
        (setq files (cdr files)))
        (setq files (cdr files)))
-    (if did-unpack 
+    (if did-unpack
         (gnus-uu-unpack-files ofiles (append did-unpack ignore))
        ofiles)))
  
         (gnus-uu-unpack-files ofiles (append did-unpack ignore))
        ofiles)))
  
@@ -1506,10 +1604,9 @@ The headers will be included in the sequence they are matched.")
    (let* ((files (gnus-uu-directory-files dir t))
          (ofiles files))
      (while files
    (let* ((files (gnus-uu-directory-files dir t))
          (ofiles files))
      (while files
-      (if (file-directory-p (car files))
-         (progn
-           (setq ofiles (delete (car files) ofiles))
-           (setq ofiles (append ofiles (gnus-uu-ls-r (car files))))))
+      (when (file-directory-p (car files))
+       (setq ofiles (delete (car files) ofiles))
+       (setq ofiles (append ofiles (gnus-uu-ls-r (car files)))))
        (setq files (cdr files)))
      ofiles))
  
        (setq files (cdr files)))
      ofiles))
  
@@ -1521,8 +1618,8 @@ The headers will be included in the sequence they are matched.")
      (while files
        (setq file (car files))
        (setq files (cdr files))
      (while files
        (setq file (car files))
        (setq files (cdr files))
-      (or (member (file-name-nondirectory file) '("." ".."))
-         (setq out (cons file out))))
+      (unless (member (file-name-nondirectory file) '("." ".."))
+       (push file out)))
      (setq out (nreverse out))
      out))
  
      (setq out (nreverse out))
      out))
  
@@ -1538,25 +1635,25 @@ The headers will be included in the sequence they are matched.")
               (goto-char start)
               (while (not (eobp))
                 (progn
               (goto-char start)
               (while (not (eobp))
                 (progn
-                 (if (looking-at "\n") (replace-match ""))
+                 (when (looking-at "\n")
+                   (replace-match ""))
                   (forward-line 1))))
                   (forward-line 1))))
-           
+
           (while (not (eobp))
           (while (not (eobp))
-           (if (looking-at (concat gnus-uu-begin-string "\\|" 
+           (if (looking-at (concat gnus-uu-begin-string "\\|"
                                     gnus-uu-end-string))
                 ()
                                     gnus-uu-end-string))
                 ()
-             (if (not found)
-                 (progn
-                   (beginning-of-line)
-                   (setq beg (point))
-                   (end-of-line)
-                   (setq length (- (point) beg))))
+             (when (not found)
+               (beginning-of-line)
+               (setq beg (point))
+               (end-of-line)
+               (setq length (- (point) beg)))
               (setq found t)
               (beginning-of-line)
               (setq beg (point))
               (end-of-line)
               (setq found t)
               (beginning-of-line)
               (setq beg (point))
               (end-of-line)
-             (if (not (= length (- (point) beg)))
-                 (insert (make-string (- length (- (point) beg)) ? ))))
+             (when (not (= length (- (point) beg)))
+               (insert (make-string (- length (- (point) beg)) ? ))))
             (forward-line 1)))))))
  
  (defvar gnus-uu-tmp-alist nil)
             (forward-line 1)))))))
  
  (defvar gnus-uu-tmp-alist nil)
@@ -1564,28 +1661,27 @@ The headers will be included in the sequence they are matched.")
  (defun gnus-uu-initialize (&optional scan)
    (let (entry)
      (if (and (not scan)
  (defun gnus-uu-initialize (&optional scan)
    (let (entry)
      (if (and (not scan)
-            (if (setq entry (assoc gnus-newsgroup-name gnus-uu-tmp-alist))
-                (if (file-exists-p (cdr entry))
-                    (setq gnus-uu-work-dir (cdr entry))
-                  (setq gnus-uu-tmp-alist (delq entry gnus-uu-tmp-alist))
-                  nil)))
+            (when (setq entry (assoc gnus-newsgroup-name gnus-uu-tmp-alist))
+              (if (file-exists-p (cdr entry))
+                  (setq gnus-uu-work-dir (cdr entry))
+                (setq gnus-uu-tmp-alist (delq entry gnus-uu-tmp-alist))
+                nil)))
         t
         t
-      (setq gnus-uu-tmp-dir (file-name-as-directory 
+      (setq gnus-uu-tmp-dir (file-name-as-directory
                              (expand-file-name gnus-uu-tmp-dir)))
        (if (not (file-directory-p gnus-uu-tmp-dir))
           (error "Temp directory %s doesn't exist" gnus-uu-tmp-dir)
                              (expand-file-name gnus-uu-tmp-dir)))
        (if (not (file-directory-p gnus-uu-tmp-dir))
           (error "Temp directory %s doesn't exist" gnus-uu-tmp-dir)
-       (if (not (file-writable-p gnus-uu-tmp-dir))
-           (error "Temp directory %s can't be written to" 
-                  gnus-uu-tmp-dir)))
+       (when (not (file-writable-p gnus-uu-tmp-dir))
+         (error "Temp directory %s can't be written to"
+                gnus-uu-tmp-dir)))
  
  
-      (setq gnus-uu-work-dir 
+      (setq gnus-uu-work-dir
             (make-temp-name (concat gnus-uu-tmp-dir "gnus")))
             (make-temp-name (concat gnus-uu-tmp-dir "gnus")))
-      (if (not (file-directory-p gnus-uu-work-dir)) 
-         (gnus-make-directory gnus-uu-work-dir))
+      (gnus-make-directory gnus-uu-work-dir)
        (set-file-modes gnus-uu-work-dir 448)
        (setq gnus-uu-work-dir (file-name-as-directory gnus-uu-work-dir))
        (set-file-modes gnus-uu-work-dir 448)
        (setq gnus-uu-work-dir (file-name-as-directory gnus-uu-work-dir))
-      (setq gnus-uu-tmp-alist (cons (cons gnus-newsgroup-name gnus-uu-work-dir)
-                                   gnus-uu-tmp-alist)))))
+      (push (cons gnus-newsgroup-name gnus-uu-work-dir)
+           gnus-uu-tmp-alist))))
  
  
  ;; Kills the temporary uu buffers, kills any processes, etc.
  
  
  ;; Kills the temporary uu buffers, kills any processes, etc.
@@ -1595,23 +1691,29 @@ The headers will be included in the sequence they are matched.")
          (memq (process-status (or gnus-uu-uudecode-process "nevair"))
                '(stop run))
          (delete-process gnus-uu-uudecode-process))
          (memq (process-status (or gnus-uu-uudecode-process "nevair"))
                '(stop run))
          (delete-process gnus-uu-uudecode-process))
-    (and (setq buf (get-buffer gnus-uu-output-buffer-name))
-        (kill-buffer buf))))
-
-;; Inputs an action and a file and returns a full command, putting
-;; quotes round the file name and escaping any quotes in the file name.
+    (when (setq buf (get-buffer gnus-uu-output-buffer-name))
+      (kill-buffer buf))))
+
+(defun gnus-quote-arg-for-sh-or-csh (arg)
+  (let ((pos 0) new-pos accum)
+    ;; *** bug: we don't handle newline characters properly
+    (while (setq new-pos (string-match "[!`\"$\\& \t{}]" arg pos))
+      (push (substring arg pos new-pos) accum)
+      (push "\\" accum)
+      (push (list (aref arg new-pos)) accum)
+      (setq pos (1+ new-pos)))
+    (if (= pos 0)
+        arg
+      (apply 'concat (nconc (nreverse accum) (list (substring arg pos)))))))
+
+;; Inputs an action and a filename and returns a full command, making sure
+;; that the filename will be treated as a single argument when the shell
+;; executes the command.
  (defun gnus-uu-command (action file)
  (defun gnus-uu-command (action file)
-  (let ((ofile ""))
-    (while (string-match "!\\|`\\|\"\\|\\$\\|\\\\\\|&" file)
-      (progn
-       (setq ofile
-             (concat ofile (substring file 0 (match-beginning 0)) "\\"
-                     (substring file (match-beginning 0) (match-end 0))))
-       (setq file (substring file (1+ (match-beginning 0))))))
-    (setq ofile (concat "\"" ofile file "\""))
+  (let ((quoted-file (gnus-quote-arg-for-sh-or-csh file)))
      (if (string-match "%s" action)
      (if (string-match "%s" action)
-       (format action ofile)
-      (concat action " " ofile))))
+       (format action quoted-file)
+      (concat action " " quoted-file))))
  
  (defun gnus-uu-delete-work-dir (&optional dir)
    "Delete recursively all files and directories under `gnus-uu-work-dir'."
  
  (defun gnus-uu-delete-work-dir (&optional dir)
    "Delete recursively all files and directories under `gnus-uu-work-dir'."
@@ -1643,40 +1745,53 @@ The headers will be included in the sequence they are matched.")
  ;;;
  
  ;; Any function that is to be used as and encoding method will take two
  ;;;
  
  ;; Any function that is to be used as and encoding method will take two
-;; parameters: PATH-NAME and FILE-NAME. (E.g. "/home/gaga/spiral.jpg"
+;; parameters: PATH-NAME and FILE-NAME.  (E.g. "/home/gaga/spiral.jpg"
  ;; and "spiral.jpg", respectively.) The function should return nil if
  ;; the encoding wasn't successful.
  ;; and "spiral.jpg", respectively.) The function should return nil if
  ;; the encoding wasn't successful.
-(defvar gnus-uu-post-encode-method 'gnus-uu-post-encode-uuencode
+(defcustom gnus-uu-post-encode-method 'gnus-uu-post-encode-uuencode
    "Function used for encoding binary files.
  There are three functions supplied with gnus-uu for encoding files:
  `gnus-uu-post-encode-uuencode', which does straight uuencoding;
    "Function used for encoding binary files.
  There are three functions supplied with gnus-uu for encoding files:
  `gnus-uu-post-encode-uuencode', which does straight uuencoding;
-`gnus-uu-post-encode-mime', which encodes with base64 and adds MIME 
-headers; and `gnus-uu-post-encode-mime-uuencode', which encodes with 
-uuencode and adds MIME headers.")
-
-(defvar gnus-uu-post-include-before-composing nil
+`gnus-uu-post-encode-mime', which encodes with base64 and adds MIME
+headers; and `gnus-uu-post-encode-mime-uuencode', which encodes with
+uuencode and adds MIME headers."
+  :group 'gnus-extract-post
+  :type '(radio (function-item gnus-uu-post-encode-uuencode)
+               (function-item gnus-uu-post-encode-mime)
+               (function-item gnus-uu-post-encode-mime-uuencode)
+               (function :tag "Other")))
+
+(defcustom gnus-uu-post-include-before-composing nil
    "Non-nil means that gnus-uu will ask for a file to encode before you compose the article.
  If this variable is t, you can either include an encoded file with
    "Non-nil means that gnus-uu will ask for a file to encode before you compose the article.
  If this variable is t, you can either include an encoded file with
-\\[gnus-uu-post-insert-binary-in-article] or have one included for you when you post the article.")
+\\[gnus-uu-post-insert-binary-in-article] or have one included for you when you post the article."
+  :group 'gnus-extract-post
+  :type 'boolean)
  
  
-(defvar gnus-uu-post-length 990
+(defcustom gnus-uu-post-length 990
    "Maximum length of an article.
  The encoded file will be split into how many articles it takes to
    "Maximum length of an article.
  The encoded file will be split into how many articles it takes to
-post the entire file.")
+post the entire file."
+  :group 'gnus-extract-post
+  :type 'integer)
  
  
-(defvar gnus-uu-post-threaded nil
+(defcustom gnus-uu-post-threaded nil
    "Non-nil means that gnus-uu will post the encoded file in a thread.
  This may not be smart, as no other decoder I have seen are able to
    "Non-nil means that gnus-uu will post the encoded file in a thread.
  This may not be smart, as no other decoder I have seen are able to
-follow threads when collecting uuencoded articles. (Well, I have seen
-one package that does that - gnus-uu, but somehow, I don't think that 
-counts...) Default is nil.")
+follow threads when collecting uuencoded articles.  (Well, I have seen
+one package that does that - gnus-uu, but somehow, I don't think that
+counts...)  The default is nil."
+  :group 'gnus-extract-post
+  :type 'boolean)
  
  
-(defvar gnus-uu-post-separate-description t
+(defcustom gnus-uu-post-separate-description t
    "Non-nil means that the description will be posted in a separate article.
    "Non-nil means that the description will be posted in a separate article.
-The first article will typically be numbered (0/x). If this variable
-is nil, the description the user enters will be included at the 
-beginning of the first article, which will be numbered (1/x). Default 
-is t.")
+The first article will typically be numbered (0/x).  If this variable
+is nil, the description the user enters will be included at the
+beginning of the first article, which will be numbered (1/x).  Default
+is t."
+  :group 'gnus-extract-post
+  :type 'boolean)
  
  (defvar gnus-uu-post-binary-separator "--binary follows this line--")
  (defvar gnus-uu-post-message-id nil)
  
  (defvar gnus-uu-post-binary-separator "--binary follows this line--")
  (defvar gnus-uu-post-message-id nil)
@@ -1696,47 +1811,44 @@ is t.")
    (local-set-key "\C-c\C-c" 'gnus-uu-post-news-inews)
    (local-set-key "\C-c\C-s" 'gnus-uu-post-news-inews)
    (local-set-key "\C-c\C-i" 'gnus-uu-post-insert-binary-in-article)
    (local-set-key "\C-c\C-c" 'gnus-uu-post-news-inews)
    (local-set-key "\C-c\C-s" 'gnus-uu-post-news-inews)
    (local-set-key "\C-c\C-i" 'gnus-uu-post-insert-binary-in-article)
-      
-  (if gnus-uu-post-include-before-composing
-      (save-excursion (setq gnus-uu-post-inserted-file-name 
-                           (gnus-uu-post-insert-binary)))))
+
+  (when gnus-uu-post-include-before-composing
+    (save-excursion (setq gnus-uu-post-inserted-file-name
+                         (gnus-uu-post-insert-binary)))))
  
  (defun gnus-uu-post-insert-binary-in-article ()
    "Inserts an encoded file in the buffer.
  The user will be asked for a file name."
    (interactive)
  
  (defun gnus-uu-post-insert-binary-in-article ()
    "Inserts an encoded file in the buffer.
  The user will be asked for a file name."
    (interactive)
-  (save-excursion 
+  (save-excursion
      (setq gnus-uu-post-inserted-file-name (gnus-uu-post-insert-binary))))
  
  ;; Encodes with uuencode and substitutes all spaces with backticks.
  (defun gnus-uu-post-encode-uuencode (path file-name)
      (setq gnus-uu-post-inserted-file-name (gnus-uu-post-insert-binary))))
  
  ;; Encodes with uuencode and substitutes all spaces with backticks.
  (defun gnus-uu-post-encode-uuencode (path file-name)
-  (if (gnus-uu-post-encode-file "uuencode" path file-name)
-      (progn
-       (goto-char (point-min))
-       (forward-line 1)
-       (while (re-search-forward " " nil t)
-         (replace-match "`"))
-       t)))
+  (when (gnus-uu-post-encode-file "uuencode" path file-name)
+    (goto-char (point-min))
+    (forward-line 1)
+    (while (re-search-forward " " nil t)
+      (replace-match "`"))
+    t))
  
  ;; Encodes with uuencode and adds MIME headers.
  (defun gnus-uu-post-encode-mime-uuencode (path file-name)
  
  ;; Encodes with uuencode and adds MIME headers.
  (defun gnus-uu-post-encode-mime-uuencode (path file-name)
-  (if (gnus-uu-post-encode-uuencode path file-name)
-      (progn
-       (gnus-uu-post-make-mime file-name "x-uue")
-       t)))
+  (when (gnus-uu-post-encode-uuencode path file-name)
+    (gnus-uu-post-make-mime file-name "x-uue")
+    t))
  
  ;; Encodes with base64 and adds MIME headers
  (defun gnus-uu-post-encode-mime (path file-name)
  
  ;; Encodes with base64 and adds MIME headers
  (defun gnus-uu-post-encode-mime (path file-name)
-  (if (gnus-uu-post-encode-file "mmencode" path file-name)
-      (progn
-       (gnus-uu-post-make-mime file-name "base64")
-       t)))
+  (when (gnus-uu-post-encode-file "mmencode" path file-name)
+    (gnus-uu-post-make-mime file-name "base64")
+    t))
  
  ;; Adds MIME headers.
  (defun gnus-uu-post-make-mime (file-name encoding)
    (goto-char (point-min))
  
  ;; Adds MIME headers.
  (defun gnus-uu-post-make-mime (file-name encoding)
    (goto-char (point-min))
-  (insert (format "Content-Type: %s; name=\"%s\"\n" 
-                 (gnus-uu-choose-action file-name gnus-uu-ext-to-mime-list) 
+  (insert (format "Content-Type: %s; name=\"%s\"\n"
+                 (gnus-uu-choose-action file-name gnus-uu-ext-to-mime-list)
                   file-name))
    (insert (format "Content-Transfer-Encoding: %s\n\n" encoding))
    (save-restriction
                   file-name))
    (insert (format "Content-Transfer-Encoding: %s\n\n" encoding))
    (save-restriction
@@ -1745,16 +1857,15 @@ The user will be asked for a file name."
      (re-search-forward (concat "^" (regexp-quote mail-header-separator) "$"))
      (forward-line -1)
      (narrow-to-region 1 (point))
      (re-search-forward (concat "^" (regexp-quote mail-header-separator) "$"))
      (forward-line -1)
      (narrow-to-region 1 (point))
-    (or (mail-fetch-field "mime-version")
-       (progn
-         (widen)
-         (insert "MIME-Version: 1.0\n")))
+    (unless (mail-fetch-field "mime-version")
+      (widen)
+      (insert "MIME-Version: 1.0\n"))
      (widen)))
  
  ;; Encodes a file PATH with COMMAND, leaving the result in the
  ;; current buffer.
  (defun gnus-uu-post-encode-file (command path file-name)
      (widen)))
  
  ;; Encodes a file PATH with COMMAND, leaving the result in the
  ;; current buffer.
  (defun gnus-uu-post-encode-file (command path file-name)
-  (= 0 (call-process shell-file-name nil t nil shell-command-switch 
+  (= 0 (call-process shell-file-name nil t nil shell-command-switch
                      (format "%s %s %s" command path file-name))))
  
  (defun gnus-uu-post-news-inews ()
                      (format "%s %s %s" command path file-name))))
  
  (defun gnus-uu-post-news-inews ()
@@ -1767,57 +1878,35 @@ If no file has been included, the user will be asked for a file."
      (if gnus-uu-post-inserted-file-name
         (setq file-name gnus-uu-post-inserted-file-name)
        (setq file-name (gnus-uu-post-insert-binary)))
      (if gnus-uu-post-inserted-file-name
         (setq file-name gnus-uu-post-inserted-file-name)
        (setq file-name (gnus-uu-post-insert-binary)))
-  
-    (if gnus-uu-post-threaded
-       (let ((message-required-news-headers 
-              (if (memq 'Message-ID message-required-news-headers)
-                  message-required-news-headers
-                (cons 'Message-ID message-required-news-headers)))
-             gnus-inews-article-hook)
-
-         (setq gnus-inews-article-hook (if (listp gnus-inews-article-hook)
-                                           gnus-inews-article-hook
-                                         (list gnus-inews-article-hook)))
-         (setq gnus-inews-article-hook 
-               (cons
-                '(lambda ()
-                   (save-excursion
-                     (goto-char (point-min))
-                     (if (re-search-forward "^Message-ID: \\(.*\\)$" nil t)
-                         (setq gnus-uu-post-message-id 
-                               (buffer-substring 
-                                (match-beginning 1) (match-end 1)))
-                       (setq gnus-uu-post-message-id nil))))
-                gnus-inews-article-hook))
-         (gnus-uu-post-encoded file-name t))
-      (gnus-uu-post-encoded file-name nil)))
+
+    (gnus-uu-post-encoded file-name gnus-uu-post-threaded))
    (setq gnus-uu-post-inserted-file-name nil)
    (setq gnus-uu-post-inserted-file-name nil)
-  (and gnus-uu-winconf-post-news
-       (set-window-configuration gnus-uu-winconf-post-news)))
-      
+  (when gnus-uu-winconf-post-news
+    (set-window-configuration gnus-uu-winconf-post-news)))
+
  ;; Asks for a file to encode, encodes it and inserts the result in
  ;; Asks for a file to encode, encodes it and inserts the result in
-;; the current buffer. Returns the file name the user gave.
+;; the current buffer.  Returns the file name the user gave.
  (defun gnus-uu-post-insert-binary ()
    (let ((uuencode-buffer-name "*uuencode buffer*")
         file-path uubuf file-name)
  
  (defun gnus-uu-post-insert-binary ()
    (let ((uuencode-buffer-name "*uuencode buffer*")
         file-path uubuf file-name)
  
-    (setq file-path (read-file-name 
+    (setq file-path (read-file-name
                      "What file do you want to encode? "))
                      "What file do you want to encode? "))
-    (if (not (file-exists-p file-path))
-       (error "%s: No such file" file-path))
+    (when (not (file-exists-p file-path))
+      (error "%s: No such file" file-path))
  
      (goto-char (point-max))
      (insert (format "\n%s\n" gnus-uu-post-binary-separator))
  
      (goto-char (point-max))
      (insert (format "\n%s\n" gnus-uu-post-binary-separator))
-    
-    (if (string-match "^~/" file-path)
-       (setq file-path (concat "$HOME" (substring file-path 1))))
+
+    (when (string-match "^~/" file-path)
+      (setq file-path (concat "$HOME" (substring file-path 1))))
      (if (string-match "/[^/]*$" file-path)
         (setq file-name (substring file-path (1+ (match-beginning 0))))
        (setq file-name file-path))
  
      (unwind-protect
         (if (save-excursion
      (if (string-match "/[^/]*$" file-path)
         (setq file-name (substring file-path (1+ (match-beginning 0))))
        (setq file-name file-path))
  
      (unwind-protect
         (if (save-excursion
-             (set-buffer (setq uubuf 
+             (set-buffer (setq uubuf
                                 (get-buffer-create uuencode-buffer-name)))
               (erase-buffer)
               (funcall gnus-uu-post-encode-method file-path file-name))
                                 (get-buffer-create uuencode-buffer-name)))
               (erase-buffer)
               (funcall gnus-uu-post-encode-method file-path file-name))
@@ -1838,45 +1927,46 @@ If no file has been included, the user will be asked for a file."
      (setq post-buf (current-buffer))
  
      (goto-char (point-min))
      (setq post-buf (current-buffer))
  
      (goto-char (point-min))
-    (if (not (re-search-forward 
-             (if gnus-uu-post-separate-description 
-                 (concat "^" (regexp-quote gnus-uu-post-binary-separator)
-                         "$")
-               (concat "^" (regexp-quote mail-header-separator) "$")) nil t))
-       (error "Internal error: No binary/header separator"))
+    (when (not (re-search-forward
+               (if gnus-uu-post-separate-description
+                   (concat "^" (regexp-quote gnus-uu-post-binary-separator)
+                           "$")
+                 (concat "^" (regexp-quote mail-header-separator) "$"))
+               nil t))
+      (error "Internal error: No binary/header separator"))
      (beginning-of-line)
      (forward-line 1)
      (setq beg-binary (point))
      (setq end-binary (point-max))
  
      (beginning-of-line)
      (forward-line 1)
      (setq beg-binary (point))
      (setq end-binary (point-max))
  
-    (save-excursion 
+    (save-excursion
        (set-buffer (setq uubuf (get-buffer-create encoded-buffer-name)))
        (erase-buffer)
        (insert-buffer-substring post-buf beg-binary end-binary)
        (goto-char (point-min))
        (setq length (count-lines 1 (point-max)))
        (setq parts (/ length gnus-uu-post-length))
        (set-buffer (setq uubuf (get-buffer-create encoded-buffer-name)))
        (erase-buffer)
        (insert-buffer-substring post-buf beg-binary end-binary)
        (goto-char (point-min))
        (setq length (count-lines 1 (point-max)))
        (setq parts (/ length gnus-uu-post-length))
-      (if (not (< (% length gnus-uu-post-length) 4))
-         (setq parts (1+ parts))))
+      (unless (< (% length gnus-uu-post-length) 4)
+       (incf parts)))
  
  
-    (if gnus-uu-post-separate-description
-       (forward-line -1))
-    (kill-region (point) (point-max))
+    (when gnus-uu-post-separate-description
+      (forward-line -1))
+    (delete-region (point) (point-max))
  
      (goto-char (point-min))
  
      (goto-char (point-min))
-    (re-search-forward 
+    (re-search-forward
       (concat "^" (regexp-quote mail-header-separator) "$") nil t)
      (beginning-of-line)
      (setq header (buffer-substring 1 (point)))
  
      (goto-char (point-min))
       (concat "^" (regexp-quote mail-header-separator) "$") nil t)
      (beginning-of-line)
      (setq header (buffer-substring 1 (point)))
  
      (goto-char (point-min))
-    (if (not gnus-uu-post-separate-description)
-       ()
-      (if (and (not threaded) (re-search-forward "^Subject: " nil t))
-         (progn
-           (end-of-line)
-           (insert (format " (0/%d)" parts))))
-      (message-send))
+    (when gnus-uu-post-separate-description
+      (when (re-search-forward "^Subject: " nil t)
+       (end-of-line)
+       (insert (format " (0/%d)" parts)))
+      (save-excursion
+       (message-send))
+      (setq gnus-uu-post-message-id (message-fetch-field "message-id")))
  
      (save-excursion
        (setq i 1)
  
      (save-excursion
        (setq i 1)
@@ -1885,32 +1975,26 @@ If no file has been included, the user will be asked for a file."
         (set-buffer (get-buffer-create send-buffer-name))
         (erase-buffer)
         (insert header)
         (set-buffer (get-buffer-create send-buffer-name))
         (erase-buffer)
         (insert header)
-       (if (and threaded gnus-uu-post-message-id)
-           (insert (format "References: %s\n" gnus-uu-post-message-id)))
+       (when (and threaded gnus-uu-post-message-id)
+         (insert "References: " gnus-uu-post-message-id "\n"))
         (insert separator)
         (setq whole-len
               (- 62 (length (format top-string "" file-name i parts ""))))
         (insert separator)
         (setq whole-len
               (- 62 (length (format top-string "" file-name i parts ""))))
-       (if (> 1 (setq minlen (/ whole-len 2)))
-           (setq minlen 1))
-       (setq 
-        beg-line 
+       (when (> 1 (setq minlen (/ whole-len 2)))
+         (setq minlen 1))
+       (setq
+        beg-line
          (format top-string
          (format top-string
-                (make-string minlen ?-) 
+                (make-string minlen ?-)
                  file-name i parts
                  file-name i parts
-                (make-string 
+                (make-string
                   (if (= 0 (% whole-len 2)) (1- minlen) minlen) ?-)))
  
         (goto-char (point-min))
                   (if (= 0 (% whole-len 2)) (1- minlen) minlen) ?-)))
  
         (goto-char (point-min))
-       (if (not (re-search-forward "^Subject: " nil t))
-           ()
-         (if (not threaded)
-             (progn
-               (end-of-line)
-               (insert (format " (%d/%d)" i parts)))
-           (if (or (and (= i 2) gnus-uu-post-separate-description)
-                   (and (= i 1) (not gnus-uu-post-separate-description)))
-               (replace-match "Subject: Re: "))))
-                 
+       (when (re-search-forward "^Subject: " nil t)
+         (end-of-line)
+         (insert (format " (%d/%d)" i parts)))
+
         (goto-char (point-max))
         (save-excursion
           (set-buffer uubuf)
         (goto-char (point-max))
         (save-excursion
           (set-buffer uubuf)
@@ -1918,39 +2002,39 @@ If no file has been included, the user will be asked for a file."
           (if (= i parts)
               (goto-char (point-max))
             (forward-line gnus-uu-post-length))
           (if (= i parts)
               (goto-char (point-max))
             (forward-line gnus-uu-post-length))
-         (if (and (= (1+ i) parts) (< (count-lines (point) (point-max)) 4))
-             (forward-line -4))
+         (when (and (= (1+ i) parts) (< (count-lines (point) (point-max)) 4))
+           (forward-line -4))
           (setq end (point)))
         (insert-buffer-substring uubuf beg end)
           (setq end (point)))
         (insert-buffer-substring uubuf beg end)
-       (insert beg-line)
-       (insert "\n")
+       (insert beg-line "\n")
         (setq beg end)
         (setq beg end)
-       (setq i (1+ i))
+       (incf i)
         (goto-char (point-min))
         (re-search-forward
          (concat "^" (regexp-quote mail-header-separator) "$") nil t)
         (beginning-of-line)
         (forward-line 2)
         (goto-char (point-min))
         (re-search-forward
          (concat "^" (regexp-quote mail-header-separator) "$") nil t)
         (beginning-of-line)
         (forward-line 2)
-       (if (re-search-forward 
-            (concat "^" (regexp-quote gnus-uu-post-binary-separator) "$")
-            nil t)
-           (progn 
-             (replace-match "")
-             (forward-line 1)))
+       (when (re-search-forward
+              (concat "^" (regexp-quote gnus-uu-post-binary-separator) "$")
+              nil t)
+         (replace-match "")
+         (forward-line 1))
         (insert beg-line)
         (insert "\n")
         (let (message-sent-message-via)
         (insert beg-line)
         (insert "\n")
         (let (message-sent-message-via)
-         (message-send))))
-
-    (and (setq buf (get-buffer send-buffer-name))
-        (kill-buffer buf))
-    (and (setq buf (get-buffer encoded-buffer-name))
-        (kill-buffer buf))
-
-    (if (not gnus-uu-post-separate-description)
-       (progn
-         (set-buffer-modified-p nil)
-         (and (fboundp 'bury-buffer) (bury-buffer))))))
+         (save-excursion
+           (message-send))
+         (setq gnus-uu-post-message-id
+               (concat (message-fetch-field "references") " "
+                       (message-fetch-field "message-id"))))))
+
+    (gnus-kill-buffer send-buffer-name)
+    (gnus-kill-buffer encoded-buffer-name)
+
+    (when (not gnus-uu-post-separate-description)
+      (set-buffer-modified-p nil)
+      (when (fboundp 'bury-buffer)
+       (bury-buffer)))))
  
  (provide 'gnus-uu)
  
  
  (provide 'gnus-uu)