*** empty log message ***
[gnus] / lisp / nnsoup.el
index bf7dfdc..6eb1a02 100644 (file)
@@ -1,5 +1,5 @@
 ;;; nnsoup.el --- SOUP access for Gnus
-;; Copyright (C) 1995 Free Software Foundation, Inc.
+;; Copyright (C) 1995,96,97,98 Free Software Foundation, Inc.
 
 ;; Author: Lars Magne Ingebrigtsen <larsi@ifi.uio.no>
 ;;     Masanobu UMEDA <umerin@flab.flab.fujitsu.junet>
@@ -18,8 +18,9 @@
 ;; GNU General Public License for more details.
 
 ;; You should have received a copy of the GNU General Public License
-;; along with GNU Emacs; see the file COPYING.  If not, write to
-;; the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.
+;; along with GNU Emacs; see the file COPYING.  If not, write to the
+;; Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+;; Boston, MA 02111-1307, USA.
 
 ;;; Commentary:
 
 (require 'nnmail)
 (require 'gnus-soup)
 (require 'gnus-msg)
+(require 'nnoo)
+(eval-when-compile (require 'cl))
 
-(defvar nnsoup-directory "~/SOUP/"
-  "*SOUP packet directory directory.")
+(nnoo-declare nnsoup)
 
-(defvar nnsoup-replies-directory (concat nnsoup-directory "replies/")
+(defvoo nnsoup-directory "~/SOUP/"
+  "*SOUP packet directory.")
+
+(defvoo nnsoup-tmp-directory "/tmp/"
+  "*Where nnsoup will store temporary files.")
+
+(defvoo nnsoup-replies-directory (concat nnsoup-directory "replies/")
   "*Directory where outgoing packets will be composed.")
 
-(defvar nnsoup-replies-format-type ?n
+(defvoo nnsoup-replies-format-type ?n
   "*Format of the replies packages.")
 
-(defvar nnsoup-replies-index-type ?n
+(defvoo nnsoup-replies-index-type ?n
   "*Index type of the replies packages.")
 
-(defvar nnsoup-active-file (concat nnsoup-directory "active")
+(defvoo nnsoup-active-file (concat nnsoup-directory "active")
   "Active file.")
 
-(defvar nnsoup-packer "tar cf - %s | gzip > $HOME/Soupin%d.tgz"
+(defvoo nnsoup-packer "tar cf - %s | gzip > $HOME/Soupin%d.tgz"
   "Format string command for packing a SOUP packet.
 The SOUP files will be inserted where the %s is in the string.
-This string MUST contain both %s and %d. The file number will be
+This string MUST contain both %s and %d.  The file number will be
 inserted where %d appears.")
 
-(defvar nnsoup-unpacker "gunzip -c %s | tar xvf -"
+(defvoo nnsoup-unpacker "gunzip -c %s | tar xvf -"
   "*Format string command for unpacking a SOUP packet.
 The SOUP packet file name will be inserted at the %s.")
 
-(defvar nnsoup-packet-directory "~/"
+(defvoo nnsoup-packet-directory "~/"
   "*Where nnsoup will look for incoming packets.")
 
-(defvar nnsoup-packet-regexp "Soupout"
+(defvoo nnsoup-packet-regexp "Soupout"
   "*Regular expression matching SOUP packets in `nnsoup-packet-directory'.")
 
+(defvoo nnsoup-always-save t
+  "If non nil commit the reply buffer on each message send. 
+This is necessary if using message mode outside Gnus with nnsoup as a 
+backend for the messages.")
+
 \f
 
 (defconst nnsoup-version "nnsoup 0.0"
   "nnsoup version.")
 
-(defvar nnsoup-status-string "")
-(defvar nnsoup-group-alist nil)
-(defvar nnsoup-replies-list nil)
-(defvar nnsoup-buffers nil)
-(defvar nnsoup-current-group nil)
-
-\f
-
-;; Server variables.
-
-(defvar nnsoup-current-server nil)
-(defvar nnsoup-server-alist nil)
-(defvar nnsoup-server-variables 
-  (list 
-   (list 'nnsoup-directory nnsoup-directory)
-   (list 'nnsoup-active-file nnsoup-active-file)
-   '(nnsoup-status-string "")
-   '(nnsoup-group-alist nil)))
-
+(defvoo nnsoup-status-string "")
+(defvoo nnsoup-group-alist nil)
+(defvoo nnsoup-current-prefix 0)
+(defvoo nnsoup-replies-list nil)
+(defvoo nnsoup-buffers nil)
+(defvoo nnsoup-current-group nil)
+(defvoo nnsoup-group-alist-touched nil)
+(defvoo nnsoup-article-alist nil)
 \f
 
 ;;; Interface functions.
 
-(defun nnsoup-retrieve-headers (sequence &optional group server fetch-old)
+(nnoo-define-basics nnsoup)
+
+(deffoo nnsoup-retrieve-headers (sequence &optional group server fetch-old)
   (nnsoup-possibly-change-group group)
   (save-excursion
     (set-buffer nntp-server-buffer)
     (erase-buffer)
-    (let ((areas (cdr (assoc nnsoup-current-group nnsoup-group-alist)))
+    (let ((areas (cddr (assoc nnsoup-current-group nnsoup-group-alist)))
          (articles sequence)
          (use-nov t)
-         useful-areas this-area-seq)
+         useful-areas this-area-seq msg-buf)
       (if (stringp (car sequence))
+         ;; We don't support fetching by Message-ID.
          'headers
        ;; We go through all the areas and find which files the
        ;; articles in SEQUENCE come from.
        (while (and areas sequence)
          ;; Peel off areas that are below sequence.
-         (while (and areas (< (cdr (car (car areas))) (car sequence)))
+         (while (and areas (< (cdaar areas) (car sequence)))
            (setq areas (cdr areas)))
-         (if (not areas)
-             ()
+         (when areas
            ;; This is a useful area.
-           (setq useful-areas (cons (car areas) useful-areas)
-                 this-area-seq nil)
+           (push (car areas) useful-areas)
+           (setq this-area-seq nil)
            ;; We take note whether this MSG has a corresponding IDX
            ;; for later use.
-           (if (or (= (gnus-soup-encoding-index 
-                       (gnus-soup-area-encoding (nth 1 (car areas)))) ?n)
-                   (not (file-exists-p
-                         (nnsoup-file
-                          (gnus-soup-area-prefix (nth 1 (car areas)))))))
-               (setq use-nov nil))
-           ;; We assing the portion of `sequence' that is relevant to
+           (when (or (= (gnus-soup-encoding-index
+                         (gnus-soup-area-encoding (nth 1 (car areas)))) ?n)
+                     (not (file-exists-p
+                           (nnsoup-file
+                            (gnus-soup-area-prefix (nth 1 (car areas)))))))
+             (setq use-nov nil))
+           ;; We assign the portion of `sequence' that is relevant to
            ;; this MSG packet to this packet.
-           (while (and sequence (<= (car sequence) (cdr (car (car areas)))))
-             (setq this-area-seq (cons (car sequence) this-area-seq)
-                   sequence (cdr sequence)))
+           (while (and sequence (<= (car sequence) (cdaar areas)))
+             (push (car sequence) this-area-seq)
+             (setq sequence (cdr sequence)))
            (setcar useful-areas (cons (nreverse this-area-seq)
                                       (car useful-areas)))))
 
        ;; We now have a list of article numbers and corresponding
-       ;; areas. 
+       ;; areas.
        (setq useful-areas (nreverse useful-areas))
 
        ;; Two different approaches depending on whether all the MSG
@@ -137,222 +141,314 @@ The SOUP packet file name will be inserted at the %s.")
        ;; what lines are relevant.  If some of the IDX files are
        ;; missing, we must return HEADs for all the articles.
        (if use-nov
-           (while useful-areas
-             (goto-char (point-max))
-             (let ((b (point))
-                   (number (car (nth 1 (car useful-areas)))))
-               (insert-buffer-substring
-                (nnsoup-index-buffer
-                 (gnus-soup-area-prefix
-                  (nth 2 (car useful-areas)))))
-               (goto-char b)
-               ;; We have to remove the index number entires and
-               ;; insert article numbers instead.
-               (while (looking-at "[0-9]+")
-                 (replace-match (int-to-string number) t t)
-                 (setq number (1+ number))
-                 (forward-line 1)))
-             (setq useful-areas (cdr useful-areas)))
+           ;; We have IDX files for all areas.
+           (progn
+             (while useful-areas
+               (goto-char (point-max))
+               (let ((b (point))
+                     (number (car (nth 1 (car useful-areas))))
+                     (index-buffer (nnsoup-index-buffer
+                                    (gnus-soup-area-prefix
+                                     (nth 2 (car useful-areas))))))
+                 (when index-buffer
+                   (insert-buffer-substring index-buffer)
+                   (goto-char b)
+                   ;; We have to remove the index number entires and
+                   ;; insert article numbers instead.
+                   (while (looking-at "[0-9]+")
+                     (replace-match (int-to-string number) t t)
+                     (incf number)
+                     (forward-line 1))))
+               (setq useful-areas (cdr useful-areas)))
+             'nov)
          ;; We insert HEADs.
          (while useful-areas
-           (setq articles (car (car useful-areas))
+           (setq articles (caar useful-areas)
                  useful-areas (cdr useful-areas))
            (while articles
-             (goto-char (point-max))
-             (insert (format "221 %d Article retrieved.\n" (car articles)))
-             (insert-buffer-substring
-              (nnsoup-narrow-to-article 
-               (car articles) (cdr (car useful-areas)) 'head))
-             (goto-char (point-max))
-             (insert ".\n")
+             (when (setq msg-buf
+                         (nnsoup-narrow-to-article
+                          (car articles) (cdar useful-areas) 'head))
+               (goto-char (point-max))
+               (insert (format "221 %d Article retrieved.\n" (car articles)))
+               (insert-buffer-substring msg-buf)
+               (goto-char (point-max))
+               (insert ".\n"))
              (setq articles (cdr articles))))
 
-         ;; Fold continuation lines.
-         (goto-char (point-min))
-         (while (re-search-forward "\\(\r?\n[ \t]+\\)+" nil t)
-           (replace-match " " t t)))
-       (if use-nov 'nov 'headers)))))
-
-(defun nnsoup-open-server (server &optional defs)
-  (nnsoup-set-variables)
-  (nnheader-init-server-buffer)
-  (if (equal server nnsoup-current-server)
-      t
-    (if nnsoup-current-server
-       (setq nnsoup-server-alist 
-             (cons (list nnsoup-current-server
-                         (nnheader-save-variables nnsoup-server-variables))
-                   nnsoup-server-alist)))
-    (let ((state (assoc server nnsoup-server-alist)))
-      (if state 
-         (progn
-           (nnheader-restore-variables (nth 1 state))
-           (setq nnsoup-server-alist (delq state nnsoup-server-alist)))
-       (nnheader-set-init-variables nnsoup-server-variables defs)))
-    (setq nnsoup-current-server server))
-  (nnsoup-read-active-file))
-
-(defun nnsoup-request-close ()
+         (nnheader-fold-continuation-lines)
+         'headers)))))
+
+(deffoo nnsoup-open-server (server &optional defs)
+  (nnoo-change-server 'nnsoup server defs)
+  (when (not (file-exists-p nnsoup-directory))
+    (condition-case ()
+       (make-directory nnsoup-directory t)
+      (error t)))
+  (cond
+   ((not (file-exists-p nnsoup-directory))
+    (nnsoup-close-server)
+    (nnheader-report 'nnsoup "Couldn't create directory: %s" nnsoup-directory))
+   ((not (file-directory-p (file-truename nnsoup-directory)))
+    (nnsoup-close-server)
+    (nnheader-report 'nnsoup "Not a directory: %s" nnsoup-directory))
+   (t
+    (nnsoup-read-active-file)
+    (nnheader-report 'nnsoup "Opened server %s using directory %s"
+                    server nnsoup-directory)
+    t)))
+
+(deffoo nnsoup-request-close ()
   (nnsoup-write-active-file)
   (nnsoup-write-replies)
-  (while nnsoup-buffers
-    (and (car nnsoup-buffers)
-        (buffer-name (car nnsoup-buffers))
-        (kill-buffer (car nnsoup-buffers)))
-    (setq nnsoup-buffers (cdr nnsoup-buffers)))
+  (gnus-soup-save-areas)
+  ;; Kill all nnsoup buffers.
+  (let (buffer)
+    (while nnsoup-buffers
+      (setq buffer (cdr (pop nnsoup-buffers)))
+      (and buffer
+          (buffer-name buffer)
+          (kill-buffer buffer))))
   (setq nnsoup-group-alist nil
+       nnsoup-group-alist-touched nil
        nnsoup-current-group nil
-       nnsoup-current-server nil
-       nnsoup-server-alist nil
        nnsoup-replies-list nil)
+  (nnoo-close-server 'nnoo)
   t)
 
-(defun nnsoup-close-server (&optional server)
-  t)
-
-(defun nnsoup-server-opened (&optional server)
-  (and (equal server nnsoup-current-server)
-       nntp-server-buffer
-       (buffer-name nntp-server-buffer)))
-
-(defun nnsoup-status-message (&optional server)
-  nnsoup-status-string)
-
-(defun nnsoup-request-article (id &optional newsgroup server buffer)
+(deffoo nnsoup-request-article (id &optional newsgroup server buffer)
   (nnsoup-possibly-change-group newsgroup)
-  (let ((buffer (or buffer nntp-server-buffer)))
+  (let (buf)
     (save-excursion
-      (set-buffer buffer)
+      (set-buffer (or buffer nntp-server-buffer))
       (erase-buffer)
-      (if (stringp id)
-         ()
-       (insert-buffer-substring
-        (nnsoup-narrow-to-article id))
+      (when (and (not (stringp id))
+                (setq buf (nnsoup-narrow-to-article id)))
+       (insert-buffer-substring buf)
        t))))
 
-(defun nnsoup-request-group (group &optional server dont-check)
+(deffoo nnsoup-request-group (group &optional server dont-check)
   (nnsoup-possibly-change-group group)
-  (if dont-check 
-      ()
-    (let ((area (cdr (assoc group nnsoup-group-alist)))
-         min max)
-      (save-excursion
-       (set-buffer nntp-server-buffer)
-       (erase-buffer)
-       (setq min (car (car (car area))))
-       (while (cdr area)
-         (setq area (cdr area)))
-       (setq max (cdr (car (car area))))
-       (insert (format "211 %d %d %d %s\n" 
-                       (max (1+ (- max min)) 0) min max group)))))
-  t)
-
-(defun nnsoup-close-group (group &optional server)
+  (if dont-check
+      t
+    (let ((active (cadr (assoc group nnsoup-group-alist))))
+      (if (not active)
+         (nnheader-report 'nnsoup "No such group: %s" group)
+       (nnheader-insert
+        "211 %d %d %d %s\n"
+        (max (1+ (- (cdr active) (car active))) 0)
+        (car active) (cdr active) group)))))
+
+(deffoo nnsoup-request-type (group &optional article)
+  (nnsoup-possibly-change-group group)
+  ;; Try to guess the type based on the first article in the group.
+  (when (not article)
+    (setq article
+         (cdaar (cddr (assoc group nnsoup-group-alist)))))
+  (if (not article)
+      'unknown
+    (let ((kind (gnus-soup-encoding-kind
+                (gnus-soup-area-encoding
+                 (nth 1 (nnsoup-article-to-area
+                         article nnsoup-current-group))))))
+      (cond ((= kind ?m) 'mail)
+           ((= kind ?n) 'news)
+           (t 'unknown)))))
+
+(deffoo nnsoup-close-group (group &optional server)
+  ;; Kill all nnsoup buffers.
+  (let ((buffers nnsoup-buffers)
+       elem)
+    (while buffers
+      (when (equal (car (setq elem (pop buffers))) group)
+       (setq nnsoup-buffers (delq elem nnsoup-buffers))
+       (and (cdr elem) (buffer-name (cdr elem))
+            (kill-buffer (cdr elem))))))
   t)
 
-(defun nnsoup-request-list (&optional server)
+(deffoo nnsoup-request-list (&optional server)
   (save-excursion
     (set-buffer nntp-server-buffer)
     (erase-buffer)
+    (unless nnsoup-group-alist
+      (nnsoup-read-active-file))
     (let ((alist nnsoup-group-alist)
-         min)
-      (while alist
-       (setq min (car (car (nth 1 (car alist)))))
-       (insert (format "%s %d %d y\n" (car (car alist))
-                       (let ((areas (car alist)))
-                         (while (cdr areas)
-                           (setq areas (cdr areas)))
-                         (cdr (car (car areas)))) min))
-       (setq alist (cdr alist)))
+         (standard-output (current-buffer))
+         entry)
+      (while (setq entry (pop alist))
+       (insert (car entry) " ")
+       (princ (cdadr entry))
+       (insert " ")
+       (princ (caadr entry))
+       (insert " y\n"))
       t)))
 
-(defun nnsoup-request-scan (group &optional server)
-  (or nnsoup-group-alist (nnsoup-read-areas))
+(deffoo nnsoup-request-scan (group &optional server)
   (nnsoup-unpack-packets))
 
-(defun nnsoup-request-newgroups (date &optional server)
+(deffoo nnsoup-request-newgroups (date &optional server)
   (nnsoup-request-list))
 
-(defun nnsoup-request-list-newsgroups (&optional server)
+(deffoo nnsoup-request-list-newsgroups (&optional server)
   nil)
 
-(defun nnsoup-request-post (&optional server)
+(deffoo nnsoup-request-post (&optional server)
   (nnsoup-store-reply "news")
   t)
 
-(defun nnsoup-request-mail ()
+(deffoo nnsoup-request-mail (&optional server)
   (nnsoup-store-reply "mail")
   t)
 
-(defun nnsoup-request-post-buffer (post group &rest args)
+(deffoo nnsoup-request-expire-articles (articles group &optional server force)
   (nnsoup-possibly-change-group group)
-  (apply
-   ;; Find out whether the source for this group is a mail or a news
-   ;; group and call the right function for getting a buffer.
-   (let ((enc (nth 1 (car (cdr (assoc nnsoup-current-group
-                                     nnsoup-group-alist))))))
-     (if (and enc
-             (= (gnus-soup-encoding-kind (gnus-soup-area-encoding enc)) ?m))
-        'nnmail-request-post-buffer 
-       'nntp-request-post-buffer))
-   post group args))
+  (let* ((total-infolist (assoc group nnsoup-group-alist))
+        (active (cadr total-infolist))
+        (infolist (cddr total-infolist))
+        info range-list mod-time prefix)
+    (while infolist
+      (setq info (pop infolist)
+           range-list (gnus-uncompress-range (car info))
+           prefix (gnus-soup-area-prefix (nth 1 info)))
+      (when ;; All the articles in this file are marked for expiry.
+         (and (or (setq mod-time (nth 5 (file-attributes
+                                         (nnsoup-file prefix))))
+                  (setq mod-time (nth 5 (file-attributes
+                                         (nnsoup-file prefix t)))))
+              (gnus-sublist-p articles range-list)
+              ;; This file is old enough.
+              (nnmail-expired-article-p group mod-time force))
+       ;; Ok, we delete this file.
+       (when (ignore-errors
+               (nnheader-message
+                5 "Deleting %s in group %s..." (nnsoup-file prefix)
+                group)
+               (when (file-exists-p (nnsoup-file prefix))
+                 (delete-file (nnsoup-file prefix)))
+               (nnheader-message
+                5 "Deleting %s in group %s..." (nnsoup-file prefix t)
+                group)
+               (when (file-exists-p (nnsoup-file prefix t))
+                 (delete-file (nnsoup-file prefix t)))
+               t)
+         (setcdr (cdr total-infolist) (delq info (cddr total-infolist)))
+         (setq articles (gnus-sorted-complement articles range-list))))
+      (when (not mod-time)
+       (setcdr (cdr total-infolist) (delq info (cddr total-infolist)))))
+    (if (cddr total-infolist)
+       (setcar active (caaadr (cdr total-infolist)))
+      (setcar active (1+ (cdr active))))
+    (nnsoup-write-active-file t)
+    ;; Return the articles that weren't expired.
+    articles))
 
 \f
 ;;; Internal functions
 
 (defun nnsoup-possibly-change-group (group &optional force)
-  (if group
-      (setq nnsoup-current-group group)
-    t))
+  (when (and group
+            (not (equal nnsoup-current-group group)))
+    (setq nnsoup-article-alist nil)
+    (setq nnsoup-current-group group))
+  t)
 
 (defun nnsoup-read-active-file ()
-  (if (file-exists-p nnsoup-active-file)
-      (condition-case ()
-         (load nnsoup-active-file)
-       (error nil))))
-
-(defun nnsoup-write-active-file ()
-  (save-excursion
-    (set-buffer (get-buffer-create " *nnsoup work*"))
-    (buffer-disable-undo (current-buffer))
-    (erase-buffer)
-    (insert (format "(setq nnsoup-group-alist '%S)\n" nnsoup-group-alist))
-    (write-region (point-min) (point-max) nnsoup-active-file
-                 nil 'silent)
-    (kill-buffer (current-buffer))))
+  (setq nnsoup-group-alist nil)
+  (when (file-exists-p nnsoup-active-file)
+    (ignore-errors
+      (load nnsoup-active-file t t t))
+    ;; Be backwards compatible.
+    (when (and nnsoup-group-alist
+              (not (atom (caadar nnsoup-group-alist))))
+      (let ((alist nnsoup-group-alist)
+           entry e min max)
+       (while (setq e (cdr (setq entry (pop alist))))
+         (setq min (caaar e))
+         (while (cdr e)
+           (setq e (cdr e)))
+         (setq max (cdaar e))
+         (setcdr entry (cons (cons min max) (cdr entry)))))
+      (setq nnsoup-group-alist-touched t))
+    nnsoup-group-alist))
+
+(defun nnsoup-write-active-file (&optional force)
+  (when (and nnsoup-group-alist
+            (or force
+                nnsoup-group-alist-touched))
+    (setq nnsoup-group-alist-touched nil)
+    (nnheader-temp-write nnsoup-active-file
+      (gnus-prin1 `(setq nnsoup-group-alist ',nnsoup-group-alist))
+      (insert "\n")
+      (gnus-prin1 `(setq nnsoup-current-prefix ,nnsoup-current-prefix))
+      (insert "\n"))))
+
+(defun nnsoup-next-prefix ()
+  "Return the next free prefix."
+  (let (prefix)
+    (while (or (file-exists-p
+               (nnsoup-file (setq prefix (int-to-string
+                                          nnsoup-current-prefix))))
+              (file-exists-p (nnsoup-file prefix t)))
+      (incf nnsoup-current-prefix))
+    (incf nnsoup-current-prefix)
+    prefix))
+
+(defun nnsoup-file-name (dir file)
+  "Return the full path of FILE (in any case) in DIR."
+  (let* ((case-fold-search t)
+        (files (directory-files dir t))
+        (regexp (concat (regexp-quote file) "$")))
+    (car (delq nil
+              (mapcar
+               (lambda (file)
+                 (if (string-match regexp file)
+                     file
+                   nil))
+               files)))))
 
 (defun nnsoup-read-areas ()
-  (save-excursion
-    (set-buffer nntp-server-buffer)
-    (let ((areas (gnus-soup-parse-areas (concat nnsoup-directory "AREAS")))
-         entry number area lnum)
-      ;; Go through all areas in the new AREAS file.
-      (while areas
-       (setq area (car areas)
-             areas (cdr areas))
-       ;; Find the number of new articles in this area.
-       (setq number (nnsoup-number-of-articles area))
-       (if (not (setq entry (assoc (gnus-soup-area-name area)
-                                   nnsoup-group-alist)))
-           ;; If this is a new area (group), we just add this info to
-           ;; the group alist. 
-           (setq nnsoup-group-alist
-                 (cons (list (gnus-soup-area-name area)
+  (let ((areas-file (nnsoup-file-name nnsoup-tmp-directory "areas")))
+    (when areas-file
+      (save-excursion
+       (set-buffer nntp-server-buffer)
+       (let ((areas (gnus-soup-parse-areas areas-file))
+             entry number area lnum cur-prefix file)
+         ;; Go through all areas in the new AREAS file.
+         (while (setq area (pop areas))
+           ;; Change the name to the permanent name and move the files.
+           (setq cur-prefix (nnsoup-next-prefix))
+           (message "Incorporating file %s..." cur-prefix)
+           (when (file-exists-p
+                  (setq file (concat nnsoup-tmp-directory
+                                     (gnus-soup-area-prefix area) ".IDX")))
+             (rename-file file (nnsoup-file cur-prefix)))
+           (when (file-exists-p
+                  (setq file (concat nnsoup-tmp-directory
+                                     (gnus-soup-area-prefix area) ".MSG")))
+             (rename-file file (nnsoup-file cur-prefix t))
+             (gnus-soup-set-area-prefix area cur-prefix)
+             ;; Find the number of new articles in this area.
+             (setq number (nnsoup-number-of-articles area))
+             (if (not (setq entry (assoc (gnus-soup-area-name area)
+                                         nnsoup-group-alist)))
+                 ;; If this is a new area (group), we just add this info to
+                 ;; the group alist.
+                 (push (list (gnus-soup-area-name area)
+                             (cons 1 number)
                              (list (cons 1 number) area))
-                       nnsoup-group-alist))
-         ;; There are already articles in this group, so we add this
-         ;; info to the end of the entry.
-         (let ((e (cdr entry)))
-           (while (cdr e)
-             (setq e (cdr e)))
-           (setcdr e (list (list (cons (setq lnum (1+ (cdr (car (car e)))))
-                                       (+ lnum number)) 
-                                 area)))))))
-    (nnsoup-write-active-file)))
+                       nnsoup-group-alist)
+               ;; There are already articles in this group, so we add this
+               ;; info to the end of the entry.
+               (nconc entry (list (list (cons (1+ (setq lnum (cdadr entry)))
+                                              (+ lnum number))
+                                        area)))
+               (setcdr (cadr entry) (+ lnum number))))))
+       (nnsoup-write-active-file t)
+       (delete-file areas-file)))))
 
 (defun nnsoup-number-of-articles (area)
   (save-excursion
-    (cond 
+    (cond
      ;; If the number is in the area info, we just return it.
      ((gnus-soup-area-number area)
       (gnus-soup-area-number area))
@@ -361,206 +457,356 @@ The SOUP packet file name will be inserted at the %s.")
       (set-buffer (nnsoup-index-buffer (gnus-soup-area-prefix area)))
       (count-lines (point-min) (point-max)))
      ;; We do it the hard way - re-searching through the message
-     ;; buffer. 
+     ;; buffer.
      (t
       (set-buffer (nnsoup-message-buffer (gnus-soup-area-prefix area)))
-      (goto-char (point-min))
-      (let ((regexp (nnsoup-header (gnus-soup-encoding-format 
-                                   (gnus-soup-area-encoding area))))
-           (num 0))
-       (while (re-search-forward regexp nil t)
-         (setq num (1+ num)))
-       num)))))
+      (unless (assoc (gnus-soup-area-prefix area) nnsoup-article-alist)
+       (nnsoup-dissect-buffer area))
+      (length (cdr (assoc (gnus-soup-area-prefix area)
+                         nnsoup-article-alist)))))))
+
+(defun nnsoup-dissect-buffer (area)
+  (let ((mbox-delim (concat "^" message-unix-mail-delimiter))
+       (format (gnus-soup-encoding-format (gnus-soup-area-encoding area)))
+       (i 0)
+       alist len)
+    (goto-char (point-min))
+    (cond
+     ;; rnews batch format
+     ((= format ?n)
+      (while (looking-at "^#! *rnews \\(+[0-9]+\\) *$")
+       (forward-line 1)
+       (push (list
+              (incf i) (point)
+              (progn
+                (forward-char (string-to-number (match-string 1)))
+                (point)))
+             alist)))
+     ;; Unix mbox format
+     ((= format ?m)
+      (while (looking-at mbox-delim)
+       (forward-line 1)
+       (push (list
+              (incf i) (point)
+              (progn
+                (if (re-search-forward mbox-delim nil t)
+                    (beginning-of-line)
+                  (goto-char (point-max)))
+                (point)))
+             alist)))
+     ;; MMDF format
+     ((= format ?M)
+      (while (looking-at "\^A\^A\^A\^A\n")
+       (forward-line 1)
+       (push (list
+              (incf i) (point)
+              (progn
+                (if (search-forward "\n\^A\^A\^A\^A\n" nil t)
+                    (beginning-of-line)
+                  (goto-char (point-max)))
+                (point)))
+             alist)))
+     ;; Binary format
+     ((or (= format ?B) (= format ?b))
+      (while (not (eobp))
+       (setq len (+ (* (char-after (point)) (expt 2.0 24))
+                    (* (char-after (+ (point) 1)) (expt 2 16))
+                    (* (char-after (+ (point) 2)) (expt 2 8))
+                    (char-after (+ (point) 3))))
+       (push (list
+              (incf i) (+ (point) 4)
+              (progn
+                (forward-char (floor (+ len 4)))
+                (point)))
+             alist)))
+     (t
+      (error "Unknown format: %c" format)))
+    (push (cons (gnus-soup-area-prefix area) alist) nnsoup-article-alist)))
 
 (defun nnsoup-index-buffer (prefix &optional message)
   (let* ((file (concat prefix (if message ".MSG" ".IDX")))
         (buffer-name (concat " *nnsoup " file "*")))
-    (or (get-buffer buffer-name)       ; File aready loaded.
-       (save-excursion                 ; Load the file.
-         (set-buffer (get-buffer-create buffer-name))
-         (setq nnsoup-buffers (cons (current-buffer) nnsoup-buffers))
-         (insert-file-contents (concat nnsoup-directory file))
-         (current-buffer)))))
+    (or (get-buffer buffer-name)       ; File already loaded.
+       (when (file-exists-p (concat nnsoup-directory file))
+         (save-excursion               ; Load the file.
+           (set-buffer (get-buffer-create buffer-name))
+           (buffer-disable-undo (current-buffer))
+           (push (cons nnsoup-current-group (current-buffer)) nnsoup-buffers)
+           (nnheader-insert-file-contents (concat nnsoup-directory file))
+           (current-buffer))))))
 
 (defun nnsoup-file (prefix &optional message)
-  (concat nnsoup-directory prefix (if message ".MSG" ".IDX")))
+  (expand-file-name
+   (concat nnsoup-directory prefix (if message ".MSG" ".IDX"))))
 
 (defun nnsoup-message-buffer (prefix)
   (nnsoup-index-buffer prefix 'msg))
 
 (defun nnsoup-unpack-packets ()
+  "Unpack all packets in `nnsoup-packet-directory'."
   (let ((packets (directory-files
                  nnsoup-packet-directory t nnsoup-packet-regexp))
-       msg)
-    (while packets
-      (message (setq msg (format "nnsoup: unpacking %s..." (car packets))))
-      (gnus-soup-unpack-packet nnsoup-directory nnsoup-unpacker (car packets))
-      (delete-file (car packets))
-      (nnsoup-read-areas)
-      (message "%sdone" msg)
-      (setq packets (cdr packets)))))
+       packet)
+    (while (setq packet (pop packets))
+      (message "nnsoup: unpacking %s..." packet)
+      (if (not (gnus-soup-unpack-packet
+               nnsoup-tmp-directory nnsoup-unpacker packet))
+         (message "Couldn't unpack %s" packet)
+       (delete-file packet)
+       (nnsoup-read-areas)
+       (message "Unpacking...done")))))
 
 (defun nnsoup-narrow-to-article (article &optional area head)
   (let* ((area (or area (nnsoup-article-to-area article nnsoup-current-group)))
-        (prefix (gnus-soup-area-prefix (nth 1 area)))
-        beg end msg-buf)
-    (setq msg-buf (nnsoup-index-buffer prefix 'msg))
-    (save-excursion
-      (cond
-       ;; We use the index file to find out where the article begins and ends. 
-       ((and (= (gnus-soup-encoding-index 
-                (gnus-soup-area-encoding (nth 1 area)))
-               ?c)
-            (file-exists-p (nnsoup-file prefix)))
-       (set-buffer (nnsoup-index-buffer prefix))
-       (widen)
-       (goto-char (point-min))
-       (forward-line (- article (car (car area))))
-       (setq beg (read (current-buffer)))
-       (forward-line 1)
-       (if (looking-at "[0-9]+")
-           (progn
-             (setq end (read (current-buffer)))
-             (set-buffer msg-buf)
-             (widen)
-             (let ((format (gnus-soup-encoding-format
-                            (gnus-soup-area-encoding (nth 1 area)))))
-               (goto-char end)
-               (if (or (= format ?n) (= format ?m))
+        (prefix (and area (gnus-soup-area-prefix (nth 1 area))))
+        (msg-buf (and prefix (nnsoup-index-buffer prefix 'msg)))
+        beg end)
+    (when area
+      (save-excursion
+       (cond
+        ;; There is no MSG file.
+        ((null msg-buf)
+         nil)
+        ;; We use the index file to find out where the article
+        ;; begins and ends.
+        ((and (= (gnus-soup-encoding-index
+                  (gnus-soup-area-encoding (nth 1 area)))
+                 ?c)
+              (file-exists-p (nnsoup-file prefix)))
+         (set-buffer (nnsoup-index-buffer prefix))
+         (widen)
+         (goto-char (point-min))
+         (forward-line (- article (caar area)))
+         (setq beg (read (current-buffer)))
+         (forward-line 1)
+         (if (looking-at "[0-9]+")
+             (progn
+               (setq end (read (current-buffer)))
+               (set-buffer msg-buf)
+               (widen)
+               (let ((format (gnus-soup-encoding-format
+                              (gnus-soup-area-encoding (nth 1 area)))))
+                 (goto-char end)
+                 (when (or (= format ?n) (= format ?m))
                    (setq end (progn (forward-line -1) (point))))))
-         (set-buffer msg-buf))
-       (widen)
-       (narrow-to-region beg (or end (point-max))))
-       (t
-       (set-buffer msg-buf)
-       (widen)
+           (set-buffer msg-buf))
+         (widen)
+         (narrow-to-region beg (or end (point-max))))
+        (t
+         (set-buffer msg-buf)
+         (widen)
+         (unless (assoc (gnus-soup-area-prefix (nth 1 area))
+                        nnsoup-article-alist)
+           (nnsoup-dissect-buffer (nth 1 area)))
+         (let ((entry (assq article (cdr (assoc (gnus-soup-area-prefix
+                                                 (nth 1 area))
+                                                nnsoup-article-alist)))))
+           (when entry
+             (narrow-to-region (cadr entry) (caddr entry))))))
        (goto-char (point-min))
-       (let ((header (nnsoup-header 
-                      (gnus-soup-encoding-format 
-                       (gnus-soup-area-encoding (nth 1 area))))))
-         (re-search-forward header nil t (- article (car (car area))))
+       (if (not head)
+           ()
          (narrow-to-region
-          (match-beginning 0)
-          (if (re-search-forward header nil t)
-              (match-beginning 0)
-            (point-max))))))
-      (goto-char (point-min))
-      (if (not head)
-         ()
-       (narrow-to-region
-        (point-min)
-        (if (search-forward "\n\n" nil t)
-            (1- (point))
-          (point-max))))
-      msg-buf)))
-
-(defun nnsoup-header (format)
-  (cond 
-   ((= format ?n)
-    "^#! *rnews +[0-9]+ *$")
-   ((= format ?m)
-    (concat "^" rmail-unix-mail-delimiter))
-   ((= format ?M)
-    "^\^A\^A\^A\^A\n")
-   (t
-    (error "Unknown format: %c" format))))
+          (point-min)
+          (if (search-forward "\n\n" nil t)
+              (1- (point))
+            (point-max))))
+       msg-buf))))
 
+;;;###autoload
 (defun nnsoup-pack-replies ()
   "Make an outbound package of SOUP replies."
   (interactive)
+  (unless (file-exists-p nnsoup-replies-directory)
+    (message "No such directory: %s" nnsoup-replies-directory))
+  ;; Write all data buffers.
+  (gnus-soup-save-areas)
+  ;; Write the active file.
   (nnsoup-write-active-file)
+  ;; Write the REPLIES file.
   (nnsoup-write-replies)
+  ;; Check whether there is anything here.
+  (when (null (directory-files nnsoup-replies-directory nil "\\.MSG$"))
+    (error "No files to pack"))
+  ;; Pack all these files into a SOUP packet.
   (gnus-soup-pack nnsoup-replies-directory nnsoup-packer))
 
 (defun nnsoup-write-replies ()
-  (gnus-soup-write-replies nnsoup-replies-directory nnsoup-replies-list))
+  "Write the REPLIES file."
+  (when nnsoup-replies-list
+    (gnus-soup-write-replies nnsoup-replies-directory nnsoup-replies-list)
+    (setq nnsoup-replies-list nil)))
 
 (defun nnsoup-article-to-area (article group)
-  (let ((areas (cdr (assoc group nnsoup-group-alist))))
-    (while (and areas (< (cdr (car (car areas))) article))
+  "Return the area that ARTICLE in GROUP is located in."
+  (let ((areas (cddr (assoc group nnsoup-group-alist))))
+    (while (and areas (< (cdaar areas) article))
       (setq areas (cdr areas)))
     (and areas (car areas))))
 
+(defvar nnsoup-old-functions
+  (list message-send-mail-function message-send-news-function))
+
+;;;###autoload
 (defun nnsoup-set-variables ()
-  (setq gnus-inews-article-function 'nnsoup-request-post)
-  (setq gnus-mail-send-method 'nnsoup-request-mail)
-  (setq send-mail-function 'nnsoup-request-mail))
+  "Use the SOUP methods for posting news and mailing mail."
+  (interactive)
+  (setq message-send-news-function 'nnsoup-request-post)
+  (setq message-send-mail-function 'nnsoup-request-mail))
+
+;;;###autoload
+(defun nnsoup-revert-variables ()
+  "Revert posting and mailing methods to the standard Emacs methods."
+  (interactive)
+  (setq message-send-mail-function (car nnsoup-old-functions))
+  (setq message-send-news-function (cadr nnsoup-old-functions)))
 
 (defun nnsoup-store-reply (kind)
-  ;; Mostly stolen from `sendmail.el'.
-  (let ((tembuf (generate-new-buffer " sendmail temp"))
+  ;; Mostly stolen from `message.el'.
+  (require 'mail-utils)
+  (let ((tembuf (generate-new-buffer " message temp"))
        (case-fold-search nil)
-       (mailbuf (current-buffer))
-       delimline)
-    (save-excursion
-      (set-buffer tembuf)
-      (erase-buffer)
-      (insert-buffer-substring mailbuf)
-      (goto-char (point-max))
-      ;; require one newline at the end.
-      (or (= (preceding-char) ?\n)
-         (insert ?\n))
-      ;; Change header-delimiter to be what sendmail expects.
-      (goto-char (point-min))
-      (re-search-forward
-       (concat "^" (regexp-quote mail-header-separator) "\n"))
-      (replace-match "\n")
-      (backward-char 1)
-      (setq delimline (point-marker))
-      (if mail-aliases (expand-mail-aliases (point-min) delimline))
-      (goto-char (point-min))
-      ;; ignore any blank lines in the header
-      (while (and (re-search-forward "\n\n\n*" delimline t)
-                 (< (point) delimline))
-       (replace-match "\n"))
-      (let ((case-fold-search t))
-       (goto-char (point-min))
-       ;; Find and handle any FCC fields.
-       (goto-char (point-min))
-       (if (re-search-forward "^FCC:" delimline t)
-           (mail-do-fcc delimline))
-       (goto-char (point-min))
-       ;; "S:" is an abbreviation for "Subject:".
-       (goto-char (point-min))
-       (if (re-search-forward "^S:" delimline t)
-           (replace-match "Subject:"))
-       ;; Don't send out a blank subject line
-       (goto-char (point-min))
-       (if (re-search-forward "^Subject:[ \t]*\n" delimline t)
-           (replace-match ""))
-       ;; Insert an extra newline if we need it to work around
-       ;; Sun's bug that swallows newlines.
-       (goto-char (1+ delimline))
-       (if (eval mail-mailer-swallows-blank-line)
-           (newline)))
-      (gnus-soup-store 
-       nnsoup-replies-directory 
-       (nnsoup-kind-to-prefix kind) nil nnsoup-replies-format-type
-       nnsoup-replies-index-type)
-      (kill-buffer tembuf))))
+       (real-header-separator mail-header-separator)
+       (mail-header-separator "")
+       delimline
+       (mailbuf (current-buffer)))
+    (unwind-protect
+       (save-excursion
+         (save-restriction
+           (message-narrow-to-headers)
+           (if (equal kind "mail")
+               (message-generate-headers message-required-mail-headers)
+             (message-generate-headers message-required-news-headers)))
+         (set-buffer tembuf)
+         (erase-buffer)
+         (insert-buffer-substring mailbuf)
+         ;; Remove some headers.
+         (save-restriction
+           (message-narrow-to-headers)
+           ;; Remove some headers.
+           (message-remove-header message-ignored-mail-headers t))
+         (goto-char (point-max))
+         ;; require one newline at the end.
+         (or (= (preceding-char) ?\n)
+             (insert ?\n))
+         (let ((case-fold-search t))
+           ;; Change header-delimiter to be what sendmail expects.
+           (goto-char (point-min))
+           (re-search-forward
+            (concat "^" (regexp-quote real-header-separator) "\n"))
+           (replace-match "\n")
+           (backward-char 1)
+           (setq delimline (point-marker))
+           ;; Insert an extra newline if we need it to work around
+           ;; Sun's bug that swallows newlines.
+           (goto-char (1+ delimline))
+           (when (eval message-mailer-swallows-blank-line)
+             (newline))
+           (let ((msg-buf
+                  (gnus-soup-store
+                   nnsoup-replies-directory
+                   (nnsoup-kind-to-prefix kind) nil nnsoup-replies-format-type
+                   nnsoup-replies-index-type))
+                 (num 0))
+             (when (and msg-buf (bufferp msg-buf))
+               (save-excursion
+                 (set-buffer msg-buf)
+                 (goto-char (point-min))
+                 (while (re-search-forward "^#! *rnews" nil t)
+                   (incf num))
+                 (when nnsoup-always-save
+                   (save-buffer)))
+               (message "Stored %d messages" num)))
+           (nnsoup-write-replies)
+           (kill-buffer tembuf))))))
 
 (defun nnsoup-kind-to-prefix (kind)
-  (or nnsoup-replies-list
-      (setq nnsoup-replies-list
-           (gnus-soup-parse-replies 
-            (concat nnsoup-replies-directory "REPLIES"))))
+  (unless nnsoup-replies-list
+    (setq nnsoup-replies-list
+         (gnus-soup-parse-replies
+          (concat nnsoup-replies-directory "REPLIES"))))
   (let ((replies nnsoup-replies-list))
-    (while (and replies 
+    (while (and replies
                (not (string= kind (gnus-soup-reply-kind (car replies)))))
       (setq replies (cdr replies)))
     (if replies
        (gnus-soup-reply-prefix (car replies))
-      (setq nnsoup-replies-list
-           (cons (vector (gnus-soup-unique-prefix nnsoup-replies-directory)
-                         kind 
-                         (format "%c%c%c"
-                                 nnsoup-replies-format-type
-                                 nnsoup-replies-index-type
-                                 (if (string= kind "news")
-                                     ?n ?m)))
-                 nnsoup-replies-list))
+      (push (vector (gnus-soup-unique-prefix nnsoup-replies-directory)
+                   kind
+                   (format "%c%c%c"
+                           nnsoup-replies-format-type
+                           nnsoup-replies-index-type
+                           (if (string= kind "news")
+                               ?n ?m)))
+           nnsoup-replies-list)
       (gnus-soup-reply-prefix (car nnsoup-replies-list)))))
-       
+
+(defun nnsoup-make-active ()
+  "(Re-)create the SOUP active file."
+  (interactive)
+  (let ((files (sort (directory-files nnsoup-directory t "IDX$")
+                    (lambda (f1 f2)
+                      (< (progn (string-match "/\\([0-9]+\\)\\." f1)
+                                (string-to-int (match-string 1 f1)))
+                         (progn (string-match "/\\([0-9]+\\)\\." f2)
+                                (string-to-int (match-string 1 f2)))))))
+       active group lines ident elem min)
+    (set-buffer (get-buffer-create " *nnsoup work*"))
+    (buffer-disable-undo (current-buffer))
+    (while files
+      (message "Doing %s..." (car files))
+      (erase-buffer)
+      (nnheader-insert-file-contents (car files))
+      (goto-char (point-min))
+      (if (not (re-search-forward "^[^\t]*\t[^\t]*\t[^\t]*\t[^\t]*\t[^\t]*\t[^\t]*\t[^\t]*\t[^\t]*\t *\\(Xref: \\)? *[^ ]* \\([^ ]+\\):[0-9]" nil t))
+         (setq group "unknown")
+       (setq group (match-string 2)))
+      (setq lines (count-lines (point-min) (point-max)))
+      (setq ident (progn (string-match
+                         "/\\([0-9]+\\)\\." (car files))
+                        (substring
+                         (car files) (match-beginning 1)
+                         (match-end 1))))
+      (if (not (setq elem (assoc group active)))
+         (push (list group (cons 1 lines)
+                     (list (cons 1 lines)
+                           (vector ident group "ncm" "" lines)))
+               active)
+       (nconc elem
+              (list
+               (list (cons (1+ (setq min (cdadr elem)))
+                           (+ min lines))
+                     (vector ident group "ncm" "" lines))))
+       (setcdr (cadr elem) (+ min lines)))
+      (setq files (cdr files)))
+    (message "")
+    (setq nnsoup-group-alist active)
+    (nnsoup-write-active-file t)))
+
+(defun nnsoup-delete-unreferenced-message-files ()
+  "Delete any *.MSG and *.IDX files that aren't known by nnsoup."
+  (interactive)
+  (let* ((known (apply 'nconc (mapcar
+                              (lambda (ga)
+                                (mapcar
+                                 (lambda (area)
+                                   (gnus-soup-area-prefix (cadr area)))
+                                 (cddr ga)))
+                              nnsoup-group-alist)))
+        (regexp "\\.MSG$\\|\\.IDX$")
+        (files (directory-files nnsoup-directory nil regexp))
+        non-files file)
+    ;; Find all files that aren't known by nnsoup.
+    (while (setq file (pop files))
+      (string-match regexp file)
+      (unless (member (substring file 0 (match-beginning 0)) known)
+       (push file non-files)))
+    ;; Sort and delete the files.
+    (setq non-files (sort non-files 'string<))
+    (map-y-or-n-p "Delete file %s? "
+                 (lambda (file) (delete-file (concat nnsoup-directory file)))
+                 non-files)))
+
 (provide 'nnsoup)
 
 ;;; nnsoup.el ends here