*** empty log message ***
[gnus] / lisp / nnfolder.el
index c415e92..ff92039 100644 (file)
@@ -1,5 +1,5 @@
 ;;; nnfolder.el --- mail folder access for Gnus
-;; Copyright (C) 1995 Free Software Foundation, Inc.
+;; Copyright (C) 1995,96 Free Software Foundation, Inc.
 
 ;; Author: Scott Byer <byer@mv.us.adobe.com>
 ;;     Lars Magne Ingebrigtsen <larsi@ifi.uio.no>
 (require 'nnheader)
 (require 'rmail)
 (require 'nnmail)
+(eval-when-compile (require 'cl))
 
 (defvar nnfolder-directory (expand-file-name "~/Mail/")
   "The name of the mail box file in the users home directory.")
 
-(defvar nnfolder-active-file (concat nnfolder-directory  "active")
+(defvar nnfolder-active-file 
+  (concat (file-name-as-directory nnfolder-directory) "active")
   "The name of the active file.")
 
 ;; I renamed this variable to somehting more in keeping with the general GNU
@@ -55,20 +57,27 @@ it.")
 (defvar nnfolder-always-close nil
   "If non-nil, nnfolder attempts to only ever have one mbox open at a time.  
 This is a straight space/performance trade off, as the mboxes will have to 
-be scaned every time they are read in.  If nil (default), nnfolder will
+be scanned every time they are read in.  If nil (default), nnfolder will
 attempt to keep the buffers around (saving the nnfolder's buffer upon group 
 close, but not killing it), speeding some things up tremendously, especially
 such things as moving mail.  All buffers always get killed upon server close.")
 
-(defvar nnfolder-newsgroups-file (concat nnfolder-directory "newsgroups")
+(defvar nnfolder-newsgroups-file 
+  (concat (file-name-as-directory nnfolder-directory) "newsgroups")
   "Mail newsgroups description file.")
 
 (defvar nnfolder-get-new-mail t
-  "If non-nil, nnml will check the incoming mail file and split the mail.")
+  "If non-nil, nnfolder will check the incoming mail file and split the mail.")
+
+(defvar nnfolder-prepare-save-mail-hook nil
+  "Hook run narrowed to an article before saving.")
+
+(defvar nnfolder-inhibit-expiry nil
+  "If non-nil, inhibit expiry.")
 
 \f
 
-(defconst nnfolder-version "nnfolder 0.2"
+(defconst nnfolder-version "nnfolder 1.0"
   "nnfolder version.")
 
 (defconst nnfolder-article-marker "X-Gnus-Article-Number: "
@@ -79,9 +88,7 @@ such things as moving mail.  All buffers always get killed upon server close.")
 (defvar nnfolder-status-string "")
 (defvar nnfolder-group-alist nil)
 (defvar nnfolder-buffer-alist nil)
-
-(defmacro nnfolder-article-string (article)
-  (` (concat "\n" nnfolder-article-marker (int-to-string (, article)) " ")))
+(defvar nnfolder-active-timestamp nil)
 
 \f
 
@@ -93,56 +100,54 @@ such things as moving mail.  All buffers always get killed upon server close.")
    (list 'nnfolder-active-file nnfolder-active-file)
    (list 'nnfolder-newsgroups-file nnfolder-newsgroups-file)
    (list 'nnfolder-get-new-mail nnfolder-get-new-mail)
+   (list 'nnfolder-inhibit-expiry nnfolder-inhibit-expiry) 
    '(nnfolder-current-group nil)
    '(nnfolder-current-buffer nil)
    '(nnfolder-status-string "")
    '(nnfolder-group-alist nil)
-   '(nnfolder-buffer-alist nil)))
+   '(nnfolder-buffer-alist nil)
+   '(nnfolder-active-timestamp nil)))
 
 \f
 
 ;;; Interface functions
 
-(defun nnfolder-retrieve-headers (sequence &optional newsgroup server)
+(defun nnfolder-retrieve-headers (sequence &optional newsgroup server fetch-old)
   (save-excursion
     (set-buffer nntp-server-buffer)
     (erase-buffer)
-    (let ((file nil)
-         (number (length sequence))
-         (delim-string (concat "^" rmail-unix-mail-delimiter))
-         beg article art-string start stop)
+    (let ((delim-string (concat "^" rmail-unix-mail-delimiter))
+         article art-string start stop)
       (nnfolder-possibly-change-group newsgroup)
       (set-buffer nnfolder-current-buffer)
       (goto-char (point-min))
-      (while sequence
-       (setq article (car sequence))
-       (setq art-string (nnfolder-article-string article))
-       (set-buffer nnfolder-current-buffer)
-       (if (or (search-forward art-string nil t)
-               ;; Don't search the whole file twice!  Also, articles
-               ;; probably have some locality by number, so searching
-               ;; backwards will be faster.  Especially if we're at the
-               ;; beginning of the buffer :-). -SLB
-               (search-backward art-string nil t))
-           (progn
-             (setq start (or (re-search-backward delim-string nil t)
-                             (point)))
-             (search-forward "\n\n" nil t)
-             (setq stop (1- (point)))
-             (set-buffer nntp-server-buffer)
-             (insert (format "221 %d Article retrieved.\n" article))
-             (setq beg (point))
-             (insert-buffer-substring nnfolder-current-buffer start stop)
-             (goto-char (point-max))
-             (insert ".\n")))
-       (setq sequence (cdr sequence)))
-
-      ;; Fold continuation lines.
-      (set-buffer nntp-server-buffer)
-      (goto-char 1)
-      (while (re-search-forward "\\(\r?\n[ \t]+\\)+" nil t)
-       (replace-match " " t t))
-      'headers)))
+      (if (stringp (car sequence))
+         'headers
+       (while sequence
+         (setq article (car sequence))
+         (setq art-string (nnfolder-article-string article))
+         (set-buffer nnfolder-current-buffer)
+         (if (or (search-forward art-string nil t)
+                 ;; Don't search the whole file twice!  Also, articles
+                 ;; probably have some locality by number, so searching
+                 ;; backwards will be faster.  Especially if we're at the
+                 ;; beginning of the buffer :-). -SLB
+                 (search-backward art-string nil t))
+             (progn
+               (setq start (or (re-search-backward delim-string nil t)
+                               (point)))
+               (search-forward "\n\n" nil t)
+               (setq stop (1- (point)))
+               (set-buffer nntp-server-buffer)
+               (insert (format "221 %d Article retrieved.\n" article))
+               (insert-buffer-substring nnfolder-current-buffer start stop)
+               (goto-char (point-max))
+               (insert ".\n")))
+         (setq sequence (cdr sequence)))
+
+       (set-buffer nntp-server-buffer)
+       (nnheader-fold-continuation-lines)
+       'headers))))
 
 (defun nnfolder-open-server (server &optional defs)
   (nnheader-init-server-buffer)
@@ -162,6 +167,7 @@ such things as moving mail.  All buffers always get killed upon server close.")
     (setq nnfolder-current-server server)))
 
 (defun nnfolder-close-server (&optional server)
+  (setq nnfolder-current-server nil)
   t)
 
 (defun nnfolder-server-opened (&optional server)
@@ -182,53 +188,79 @@ such things as moving mail.  All buffers always get killed upon server close.")
 
 (defun nnfolder-request-article (article &optional newsgroup server buffer)
   (nnfolder-possibly-change-group newsgroup)
-  (if (stringp article)
-      nil
-    (save-excursion
-      (set-buffer nnfolder-current-buffer)
-      (goto-char (point-min))
-      (if (search-forward (nnfolder-article-string article) nil t)
-         (let (start stop)
-           (re-search-backward (concat "^" rmail-unix-mail-delimiter) nil t)
-           (setq start (point))
-           (forward-line 1)
-           (or (and (re-search-forward 
-                     (concat "^" rmail-unix-mail-delimiter) nil t)
-                    (forward-line -1))
-               (goto-char (point-max)))
-           (setq stop (point))
-           (let ((nntp-server-buffer (or buffer nntp-server-buffer)))
-             (set-buffer nntp-server-buffer)
-             (erase-buffer)
-             (insert-buffer-substring nnfolder-current-buffer start stop)
+  (save-excursion
+    (set-buffer nnfolder-current-buffer)
+    (goto-char (point-min))
+    (if (search-forward (nnfolder-article-string article) nil t)
+       (let (start stop)
+         (re-search-backward (concat "^" rmail-unix-mail-delimiter) nil t)
+         (setq start (point))
+         (forward-line 1)
+         (or (and (re-search-forward 
+                   (concat "^" rmail-unix-mail-delimiter) nil t)
+                  (forward-line -1))
+             (goto-char (point-max)))
+         (setq stop (point))
+         (let ((nntp-server-buffer (or buffer nntp-server-buffer)))
+           (set-buffer nntp-server-buffer)
+           (erase-buffer)
+           (insert-buffer-substring nnfolder-current-buffer start stop)
+           (goto-char (point-min))
+           (while (looking-at "From ")
+             (delete-char 5)
+             (insert "X-From-Line: ")
+             (forward-line 1))
+           (if (numberp article) 
+               (cons nnfolder-current-group article)
              (goto-char (point-min))
-             (while (looking-at "From ")
-               (delete-char 5)
-               (insert "X-From-Line: ")
-               (forward-line 1))
-             t))))))
+             (search-forward (concat "\n" nnfolder-article-marker))
+             (cons nnfolder-current-group
+                   (string-to-int 
+                    (buffer-substring 
+                     (point) (progn (end-of-line) (point)))))))))))
 
 (defun nnfolder-request-group (group &optional server dont-check)
   (save-excursion
-    (nnfolder-possibly-change-group group)
-    (and (assoc group nnfolder-group-alist)
-        (save-excursion
-          (set-buffer nntp-server-buffer)
-          (erase-buffer)
-          (if dont-check
-              t
-            (nnfolder-get-new-mail)
-            (let ((active (assoc group nnfolder-group-alist)))
-              ;; I've been getting stray 211 lines in my nnfolder active
-              ;; file.  So, let's make sure that doesn't happen. -SLB
-              (set-buffer nntp-server-buffer)
-              (insert (format "211 %d %d %d %s\n" 
-                              (1+ (- (cdr (car (cdr active)))
-                                     (car (car (cdr active)))))
-                              (car (car (cdr active)))
-                              (cdr (car (cdr active)))
-                              (car active))))
-            t)))))
+    (nnmail-activate 'nnfolder)
+    (when (assoc group nnfolder-group-alist)
+      (nnfolder-possibly-change-group group)
+      (cond 
+       (dont-check
+       (nnheader-report 'nnfolder "Selected group %s" group)
+       t)
+       (t
+       (let* ((active (assoc group nnfolder-group-alist))
+              (group (car active))
+              (range (car (cdr active)))
+              (minactive (car range))
+              (maxactive (cdr range)))
+         (set-buffer nntp-server-buffer)
+         (erase-buffer)
+         (cond 
+          ((null active)
+           (nnheader-report 'nnfolder "No such group: %s" group))
+          (t
+           (nnheader-report 'nnfolder "Selected group %s" group)
+           (insert (format "211 %d %d %d %s\n" 
+                           (1+ (- maxactive minactive))
+                           minactive maxactive group))
+           t))))))))
+
+(defun nnfolder-request-scan (&optional group server)
+  (nnmail-get-new-mail
+   'nnfolder 
+   (lambda ()
+     (let ((bufs nnfolder-buffer-alist))
+       (save-excursion
+        (while bufs
+          (if (not (buffer-name (nth 1 (car bufs))))
+              (setq nnfolder-buffer-alist 
+                    (delq (car bufs) nnfolder-buffer-alist))
+            (set-buffer (nth 1 (car bufs)))
+            (and (buffer-modified-p) (save-buffer)))
+          (setq bufs (cdr bufs))))))
+   nnfolder-directory
+   group))
 
 ;; Don't close the buffer if we're not shutting down the server.  This way,
 ;; we can keep the buffer in the group buffer cache, and not have to grovel
@@ -236,68 +268,79 @@ such things as moving mail.  All buffers always get killed upon server close.")
 ;; way.
 
 (defun nnfolder-close-group (group &optional server force)
-  (nnfolder-possibly-change-group group)
-  (save-excursion
-    (set-buffer nnfolder-current-buffer)
-    ;; If the buffer was modified, write the file out now.
-    (save-buffer)
-    (if (or force
-           nnfolder-always-close)
-       ;; If we're shutting the server down, we need to kill the buffer and
-       ;; remove it from the open buffer list.  Or, of course, if we're
-       ;; trying to minimize our space impact.
-       (progn
-         (kill-buffer (current-buffer))
-         (setq nnfolder-buffer-alist (delq (assoc group nnfolder-buffer-alist)
-                                           nnfolder-buffer-alist))))
-    (setq nnfolder-current-group nil
-         nnfolder-current-buffer nil))
+  ;; Make sure we _had_ the group open.
+  (if (or (assoc group nnfolder-buffer-alist)
+         (equal group nnfolder-current-group))
+      (progn
+       (nnfolder-possibly-change-group group)
+       (save-excursion
+         (set-buffer nnfolder-current-buffer)
+         ;; If the buffer was modified, write the file out now.
+         (and (buffer-modified-p) (save-buffer))
+         (if (or force
+                 nnfolder-always-close)
+             ;; If we're shutting the server down, we need to kill the
+             ;; buffer and remove it from the open buffer list.  Or, of
+             ;; course, if we're trying to minimize our space impact.
+             (progn
+               (kill-buffer (current-buffer))
+               (setq nnfolder-buffer-alist (delq (assoc group 
+                                                        nnfolder-buffer-alist)
+                                                 nnfolder-buffer-alist)))))))
+  (setq nnfolder-current-group nil
+       nnfolder-current-buffer nil)
+  t)
+
+(defun nnfolder-request-create-group (group &optional server) 
+  (nnmail-activate 'nnfolder)
+  (or (assoc group nnfolder-group-alist)
+      (let (active)
+       (setq nnfolder-group-alist 
+             (cons (list group (setq active (cons 1 0)))
+                   nnfolder-group-alist))
+       (nnmail-save-active nnfolder-group-alist nnfolder-active-file)))
   t)
 
 (defun nnfolder-request-list (&optional server)
-  (if server (nnfolder-get-new-mail))
-  (or nnfolder-group-alist
-      (nnmail-find-file nnfolder-active-file)
-      (progn
-       (setq nnfolder-group-alist (nnmail-get-active))
-       (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
-       (nnmail-find-file nnfolder-active-file))))
+  (save-excursion
+    (nnmail-find-file nnfolder-active-file)
+    (setq nnfolder-group-alist (nnmail-get-active))))
 
 (defun nnfolder-request-newgroups (date &optional server)
   (nnfolder-request-list server))
 
 (defun nnfolder-request-list-newsgroups (&optional server)
-  (nnmail-find-file nnfolder-newsgroups-file))
+  (save-excursion
+    (nnmail-find-file nnfolder-newsgroups-file)))
 
 (defun nnfolder-request-post (&optional server)
   (mail-send-and-exit nil))
 
-(fset 'nnfolder-request-post-buffer 'nnmail-request-post-buffer)
-
-(defun nnfolder-request-expire-articles (articles newsgroup &optional server force)
+(defun nnfolder-request-expire-articles 
+  (articles newsgroup &optional server force)
   (nnfolder-possibly-change-group newsgroup)
-  (let* ((days (or (and nnmail-expiry-wait-function
-                       (funcall nnmail-expiry-wait-function newsgroup))
-                  nnmail-expiry-wait))
-        article rest)
+  (let* ((is-old t)
+        rest)
+    (nnmail-activate 'nnfolder)
+
     (save-excursion 
       (set-buffer nnfolder-current-buffer)
-      (while articles
-       (goto-char 1)
+      (while (and articles is-old)
+       (goto-char (point-min))
        (if (search-forward (nnfolder-article-string (car articles)) nil t)
-           (if (or force
-                   (> (nnmail-days-between 
-                       (current-time-string)
-                       (buffer-substring 
-                        (point) (progn (end-of-line) (point))))
-                      days))
+           (if (setq is-old
+                     (nnmail-expired-article-p 
+                      newsgroup
+                      (buffer-substring 
+                       (point) (progn (end-of-line) (point))) 
+                      force nnfolder-inhibit-expiry))
                (progn
-                 (and gnus-verbose-backends
-                      (message "Deleting: %s" (car articles)))
+                 (nnheader-message 5 "Deleting article %d..." 
+                                   (car articles) newsgroup)
                  (nnfolder-delete-mail))
              (setq rest (cons (car articles) rest))))
        (setq articles (cdr articles)))
-      (save-buffer)
+      (and (buffer-modified-p) (save-buffer))
       ;; Find the lowest active article in this group.
       (let* ((active (car (cdr (assoc newsgroup nnfolder-group-alist))))
             (marker (concat "\n" nnfolder-article-marker))
@@ -312,7 +355,7 @@ such things as moving mail.  All buffers always get killed upon server close.")
                                                  (match-end 0))))))
        (setcar active activemin))
       (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
-      rest)))
+      (nconc rest articles))))
 
 (defun nnfolder-request-move-article
   (article group server accept-form &optional last)
@@ -328,7 +371,7 @@ such things as moving mail.  All buffers always get killed upon server close.")
        (insert-buffer-substring nntp-server-buffer)
        (goto-char (point-min))
        (while (re-search-forward 
-              "^X-Gnus-Newsgroup:" 
+              (concat "^" nnfolder-article-marker)
               (save-excursion (search-forward "\n\n" nil t) (point)) t)
         (delete-region (progn (beginning-of-line) (point))
                        (progn (forward-line 1) (point))))
@@ -338,53 +381,99 @@ such things as moving mail.  All buffers always get killed upon server close.")
      (save-excursion
        (nnfolder-possibly-change-group group)
        (set-buffer nnfolder-current-buffer)
-       (goto-char 1)
+       (goto-char (point-min))
        (if (search-forward (nnfolder-article-string article) nil t)
           (nnfolder-delete-mail))
-       (and last (save-buffer))))
+       (and last 
+           (buffer-modified-p)
+           (save-buffer))))
     result))
 
 (defun nnfolder-request-accept-article (group &optional last)
-  (nnfolder-possibly-change-group group)
+  (and (stringp group) (nnfolder-possibly-change-group group))
   (let ((buf (current-buffer))
-       result beg)
+       result)
     (goto-char (point-min))
-    (if (looking-at "X-From-Line: ")
-       (replace-match "From ")
-      (insert "From nobody " (current-time-string) "\n"))
+    (when (looking-at "X-From-Line: ")
+      (replace-match "From "))
     (and 
      (nnfolder-request-list)
-     (progn
+     (save-excursion
        (set-buffer buf)
        (goto-char (point-min))
        (search-forward "\n\n" nil t)
        (forward-line -1)
-       (while (re-search-backward "^X-Gnus-Newsgroup: " nil t)
+       (while (re-search-backward (concat "^" nnfolder-article-marker) nil t)
         (delete-region (point) (progn (forward-line 1) (point))))
        (setq result (car (nnfolder-save-mail (and (stringp group) group)))))
      (save-excursion
        (set-buffer nnfolder-current-buffer)
-       (insert-buffer-substring buf)
-       (and last (save-buffer))
-       result)
-     (nnmail-save-active nnfolder-group-alist nnfolder-active-file))
+       (and last (buffer-modified-p) (save-buffer))))
+    (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
     result))
 
 (defun nnfolder-request-replace-article (article group buffer)
   (nnfolder-possibly-change-group group)
   (save-excursion
     (set-buffer nnfolder-current-buffer)
-    (goto-char 1)
+    (goto-char (point-min))
     (if (not (search-forward (nnfolder-article-string article) nil t))
        nil
       (nnfolder-delete-mail t t)
       (insert-buffer-substring buffer)
-      (save-buffer)
+      (and (buffer-modified-p) (save-buffer))
       t)))
 
+(defun nnfolder-request-delete-group (group &optional force server)
+  (nnfolder-close-group group server t)
+  ;; Delete all articles in GROUP.
+  (if (not force)
+      ()                               ; Don't delete the articles.
+    ;; Delete the file that holds the group.
+    (condition-case nil
+       (delete-file (concat (file-name-as-directory nnfolder-directory)
+                            group))
+      (error nil)))
+  ;; Remove the group from all structures.
+  (setq nnfolder-group-alist 
+       (delq (assoc group nnfolder-group-alist) nnfolder-group-alist)
+       nnfolder-current-group nil
+       nnfolder-current-buffer nil)
+  ;; Save the active file.
+  (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
+  t)
+
+(defun nnfolder-request-rename-group (group new-name &optional server)
+  (nnfolder-possibly-change-group group)
+  (save-excursion
+    (set-buffer nnfolder-current-buffer)
+    (and (file-writable-p buffer-file-name)
+        (condition-case ()
+            (progn
+              (rename-file buffer-file-name
+                           (concat (file-name-as-directory nnfolder-directory)
+                                   new-name))
+              t)
+          (error nil))
+        ;; That went ok, so we change the internal structures.
+        (let ((entry (assoc group nnfolder-group-alist)))
+          (and entry (setcar entry new-name))
+          (setq nnfolder-current-buffer nil
+                nnfolder-current-group nil)
+          ;; Save the new group alist.
+          (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
+          ;; We kill the buffer instead of renaming it and stuff.
+          (kill-buffer (current-buffer))
+          t))))
+
 \f
 ;;; Internal functions.
 
+(defun nnfolder-article-string (article)
+  (if (numberp article)
+      (concat "\n" nnfolder-article-marker (int-to-string article) " ")
+    (concat "\nMessage-ID: " article)))
+
 (defun nnfolder-delete-mail (&optional force leave-delim)
   ;; Beginning of the article.
   (save-excursion
@@ -406,19 +495,18 @@ such things as moving mail.  All buffers always get killed upon server close.")
 
 (defun nnfolder-possibly-change-group (group)
   (or (file-exists-p nnfolder-directory)
-      (make-directory (directory-file-name nnfolder-directory)))
-  (if (not nnfolder-group-alist)
-      (progn
-       (nnfolder-request-list)
-       (setq nnfolder-group-alist (nnmail-get-active))))
+      (make-directory (directory-file-name nnfolder-directory) t))
+  (nnfolder-possibly-activate-groups nil)
   (or (assoc group nnfolder-group-alist)
-      (not (file-exists-p (concat nnfolder-directory group)))
+      (not (file-exists-p (concat (file-name-as-directory nnfolder-directory)
+                                 group)))
       (progn
        (setq nnfolder-group-alist 
              (cons (list group (cons 1 0)) nnfolder-group-alist))
        (nnmail-save-active nnfolder-group-alist nnfolder-active-file)))
   (let (inf file)
     (if (and (equal group nnfolder-current-group)
+            nnfolder-current-buffer
             (buffer-name nnfolder-current-buffer))
        ()
       (setq nnfolder-current-group group)
@@ -432,11 +520,14 @@ such things as moving mail.  All buffers always get killed upon server close.")
       ;; If the buffer is not live, make sure it isn't in the alist.  If it
       ;; is live, verify that nobody else has touched the file since last
       ;; time.
-      (if (or (not (buffer-name nnfolder-current-buffer))
+      (if (or (not (and nnfolder-current-buffer
+                       (buffer-name nnfolder-current-buffer)))
              (not (and (bufferp nnfolder-current-buffer)
-                       (verify-visited-file-modtime nnfolder-current-buffer))))
+                       (verify-visited-file-modtime 
+                        nnfolder-current-buffer))))
          (progn
-           (if (and (buffer-name nnfolder-current-buffer)
+           (if (and nnfolder-current-buffer
+                    (buffer-name nnfolder-current-buffer)
                     (bufferp nnfolder-current-buffer))
                (kill-buffer nnfolder-current-buffer))
            (setq nnfolder-buffer-alist (delq inf nnfolder-buffer-alist))
@@ -445,12 +536,16 @@ such things as moving mail.  All buffers always get killed upon server close.")
       (if inf
          ()
        (save-excursion
-         (setq file (concat nnfolder-directory group))
-         (if (not (file-exists-p file))
-             (write-region 1 1 file t 'nomesg))
-         (set-buffer (nnfolder-read-folder file))
-         (setq nnfolder-buffer-alist (cons (list group (current-buffer))
-                                           nnfolder-buffer-alist))))))
+         (setq file (concat (file-name-as-directory nnfolder-directory)
+                            group))
+         (if (file-directory-p (file-truename file))
+             ()
+           (if (not (file-exists-p file))
+               (write-region 1 1 file t 'nomesg))
+           (setq nnfolder-current-buffer 
+                 (set-buffer (nnfolder-read-folder file)))
+           (setq nnfolder-buffer-alist (cons (list group (current-buffer))
+                                             nnfolder-buffer-alist)))))))
   (setq nnfolder-current-group group))
 
 (defun nnfolder-save-mail (&optional group)
@@ -459,31 +554,45 @@ such things as moving mail.  All buffers always get killed upon server close.")
          (if group (list (list group "")) nnmail-split-methods))
         (group-art-list
          (nreverse (nnmail-article-group 'nnfolder-active-number)))
+        (delim (concat "^" rmail-unix-mail-delimiter))
         save-list group-art)
+    (goto-char (point-min))
+    ;; This might come from somewhere else.
+    (unless (looking-at delim)
+      (insert "From nobody " (current-time-string) "\n")
+      (goto-char (point-min)))
+    ;; Quote all "From " lines in the article.
+    (forward-line 1)
+    (while (re-search-forward delim nil t)
+      (beginning-of-line)
+      (insert "> "))
     (setq save-list group-art-list)
     (nnmail-insert-lines)
     (nnmail-insert-xref group-art-list)
-
-    ;; Kill the previous newsgroup markers.
-    (goto-char (point-min))
-    (search-forward "\n\n" nil t)
-    (forward-line -1)
-    (while (re-search-backward (concat "^" nnfolder-article-marker) nil t)
-      (delete-region (point) (progn (forward-line 1) (point))))
+    (run-hooks 'nnfolder-prepare-save-mail-hook)
 
     ;; Insert the mail into each of the destination groups.
     (while group-art-list
       (setq group-art (car group-art-list)
            group-art-list (cdr group-art-list))
+
+      ;; Kill the previous newsgroup markers.
+      (goto-char (point-min))
+      (search-forward "\n\n" nil t)
+      (forward-line -1)
+      (while (search-backward (concat "\n" nnfolder-article-marker) nil t)
+       (delete-region (1+ (point)) (progn (forward-line 2) (point))))
+
+      ;; Insert the new newsgroup marker.
       (nnfolder-possibly-change-group (car group-art))
       (nnfolder-insert-newsgroup-line group-art)
       (let ((beg (point-min))
            (end (point-max))
            (obuf (current-buffer)))
-       (save-excursion
-         (set-buffer nnfolder-current-buffer)
-         (goto-char (point-max))
-         (insert-buffer-substring obuf beg end))))
+       (set-buffer nnfolder-current-buffer)
+       (goto-char (point-max))
+       (insert-buffer-substring obuf beg end)
+       (set-buffer obuf)))
 
     ;; Did we save it anywhere?
     save-list))
@@ -497,19 +606,35 @@ such things as moving mail.  All buffers always get killed upon server close.")
          (insert (format (concat nnfolder-article-marker "%d   %s\n")
                          (cdr group-art) (current-time-string)))))))
 
+(defun nnfolder-possibly-activate-groups (&optional group)
+  (save-excursion
+    ;; If we're looking for the activation of a specific group, find out
+    ;; its real name and switch to it.
+    (if group (nnfolder-possibly-change-group group))
+    ;; If the group alist isn't active, activate it now.
+    (nnmail-activate 'nnfolder)))
+
 (defun nnfolder-active-number (group)
-  (if (not nnfolder-group-alist)
-      (save-excursion
-       (nnfolder-request-list)
-       (setq nnfolder-group-alist (nnmail-get-active))))
-  (let ((active (car (cdr (assoc group nnfolder-group-alist)))))
-    (setcdr active (1+ (cdr active)))
-    (cdr active)))
+  (save-excursion 
+    ;; Find the next article number in GROUP.
+    (prog1
+       (let ((active (car (cdr (assoc group nnfolder-group-alist)))))
+         (if active
+             (setcdr active (1+ (cdr active)))
+           ;; This group is new, so we create a new entry for it.
+           ;; This might be a bit naughty... creating groups on the drop of
+           ;; a hat, but I don't know...
+           (setq nnfolder-group-alist 
+                 (cons (list group (setq active (cons 1 1)))
+                       nnfolder-group-alist)))
+         (cdr active))
+      (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
+      (nnfolder-possibly-activate-groups group))))
 
 
 ;; This method has a problem if you've accidentally let the active list get
 ;; out of sync with the files.  This could happen, say, if you've
-;; accidentally gotten new mail with something other than (ding) (but why
+;; accidentally gotten new mail with something other than Gnus (but why
 ;; would _that_ ever happen? :-).  In that case, we will be in the middle of
 ;; processing the file, ready to add new X-Gnus article number markers, and
 ;; we'll run accross a message with no ID yet - the active list _may_not_ be
@@ -524,14 +649,12 @@ such things as moving mail.  All buffers always get killed upon server close.")
 
 (defun nnfolder-read-folder (file)
   (save-excursion
-    (if (not nnfolder-group-alist)
-       (progn
-         (nnfolder-request-list)
-         (setq nnfolder-group-alist (nnmail-get-active))))
+    (nnfolder-possibly-activate-groups nil)
     ;; We should be paranoid here and make sure the group is in the alist,
     ;; and add it if it isn't.
     ;;(if (not (assoc nnfoler-current-group nnfolder-group-alist)
-    (set-buffer (setq nnfolder-current-buffer (find-file-noselect file)))
+    (set-buffer (setq nnfolder-current-buffer 
+                     (find-file-noselect file nil 'raw)))
     (buffer-disable-undo (current-buffer))
     (let ((delim (concat "^" rmail-unix-mail-delimiter))
          (marker (concat "\n" nnfolder-article-marker))
@@ -555,13 +678,13 @@ such things as moving mail.  All buffers always get killed upon server close.")
                                 (1- (lsh 1 25))))
            (while (and (search-forward marker nil t)
                        (re-search-forward number nil t))
-             (let (newnum (string-to-number (buffer-substring
-                                             (match-beginning 0)
-                                             (match-end 0))))
+             (let ((newnum (string-to-number (buffer-substring
+                                              (match-beginning 0)
+                                              (match-end 0)))))
                (setq activenumber (max activenumber newnum))
                (setq activemin (min activemin newnum))))
-           (setcar active (min activemin activenumber))
-           (setcdr active activenumber)
+           (setcar active (max 1 (min activemin activenumber)))
+           (setcdr active (max activenumber (cdr active)))
            (goto-char (point-min))))
 
       ;; Keep track of the active number on our own, and insert it back into
@@ -574,7 +697,10 @@ such things as moving mail.  All buffers always get killed upon server close.")
       (while (not (= end (point-max)))
        (setq start (marker-position end))
        (goto-char end)
-       (end-of-line)
+       ;; There may be more than one "From " line, so we skip past
+       ;; them.  
+       (while (looking-at delim) 
+         (forward-line 1))
        (set-marker end (or (and (re-search-forward delim nil t)
                                 (match-beginning 0))
                            (point-max)))
@@ -583,44 +709,28 @@ such things as moving mail.  All buffers always get killed upon server close.")
            (progn
              (narrow-to-region start end)
              (nnmail-insert-lines)
-             (setq activenumber (1+ activenumber))
-             (nnfolder-insert-newsgroup-line (cons nil activenumber))
+             (nnfolder-insert-newsgroup-line
+              (cons nil (nnfolder-active-number nnfolder-current-group)))
              (widen))))
 
       ;; Make absolutely sure that the active list reflects reality!
-      (setcdr active activenumber)
       (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
       (current-buffer))))
 
-(defun nnfolder-get-new-mail ()
-  (let (incoming)
-    (if (and nnmail-spool-file
-            nnfolder-get-new-mail
-            (file-exists-p nnmail-spool-file)
-            (> (nth 7 (file-attributes nnmail-spool-file)) 0))
-       (progn
-         (and gnus-verbose-backends
-              (message "nnfolder: Reading incoming mail..."))
-         (setq incoming 
-               (nnmail-move-inbox nnmail-spool-file
-                                  (concat nnfolder-directory "Incoming")))
-         (nnmail-split-incoming incoming 'nnfolder-save-mail)
-         (run-hooks 'nnmail-read-incoming-hook)
-         (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
-         (and gnus-verbose-backends
-              (message "nnfolder: Reading incoming mail...done"))))
-    (let ((bufs nnfolder-buffer-alist))
-      (save-excursion
-       (while bufs
-         (if (not (buffer-name (nth 1 (car bufs))))
-             (setq nnfolder-buffer-alist 
-                   (delq (car bufs) nnfolder-buffer-alist))
-           (set-buffer (nth 1 (car bufs)))
-           (and (buffer-modified-p)
-                (save-buffer)))
-         (setq bufs (cdr bufs)))))
-    ;; (if incoming (delete-file incoming))
-    ))
+;;;###autoload
+(defun nnfolder-generate-active-file ()
+  "Look for mbox folders in the nnfolder directory and make them into groups."
+  (interactive)
+  (nnmail-activate 'nnfolder)
+  (let ((files (directory-files nnfolder-directory))
+       file group)
+    (while (setq file (pop files))
+      (when (nnheader-mail-file-mbox-p file)
+       (nnheader-message 5 "Adding group %s..." file)
+       (push (list file (cons 1 0)) nnfolder-group-alist)
+       (nnfolder-read-folder file)
+       (nnfolder-close-group file))
+      (message ""))))
 
 (provide 'nnfolder)