This commit was generated by cvs2svn to compensate for changes in r6565,
[elisp/gnus.git-] / lisp / nnml.el
index 0767962..e7a1df9 100644 (file)
@@ -1,10 +1,8 @@
 ;;; nnml.el --- mail spool access for Gnus
-;; Copyright (C) 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2003
-;;        Free Software Foundation, Inc.
+;; Copyright (C) 1995,96,97,98 Free Software Foundation, Inc.
 
-;; Author: Simon Josefsson <simon@josefsson.org> (adding MARKS)
-;;      Lars Magne Ingebrigtsen <larsi@gnus.org>
-;;     Masanobu UMEDA <umerin@flab.flab.fujitsu.junet>
+;; Author: Lars Magne Ingebrigtsen <larsi@gnus.org>
+;;     Masanobu UMEDA <umerin@flab.flab.fujitsu.junet>
 ;; Keywords: news, mail
 
 ;; This file is part of GNU Emacs.
 
 ;;; Code:
 
-(require 'gnus)
 (require 'nnheader)
 (require 'nnmail)
 (require 'nnoo)
 (eval-when-compile (require 'cl))
 
-(eval-and-compile
-  (autoload 'gnus-article-unpropagatable-p "gnus-sum"))
-
 (nnoo-declare nnml)
 
 (defvoo nnml-directory message-directory
   "Spool directory for the nnml mail backend.")
 
 (defvoo nnml-active-file
-    (expand-file-name "active" nnml-directory)
+  (concat (file-name-as-directory nnml-directory) "active")
   "Mail active file.")
 
 (defvoo nnml-newsgroups-file
-    (expand-file-name "newsgroups" nnml-directory)
+  (concat (file-name-as-directory nnml-directory) "newsgroups")
   "Mail newsgroups description file.")
 
 (defvoo nnml-get-new-mail t
   "If non-nil, nnml will check the incoming mail file and split the mail.")
 
 (defvoo nnml-nov-is-evil nil
-  "If non-nil, Gnus will never generate and use nov databases for mail spools.
+  "If non-nil, Gnus will never generate and use nov databases for mail groups.
 Using nov databases will speed up header fetching considerably.
 This variable shouldn't be flipped much.  If you have, for some reason,
 set this to t, and want to set it to nil again, you should always run
@@ -66,23 +60,12 @@ the `nnml-generate-nov-databases' command.  The function will go
 through all nnml directories and generate nov databases for them
 all.  This may very well take some time.")
 
-(defvoo nnml-marks-is-evil nil
-  "If non-nil, Gnus will never generate and use marks file for mail spools.
-Using marks files makes it possible to backup and restore mail groups
-separately from `.newsrc.eld'.  If you have, for some reason, set this
-to t, and want to set it to nil again, you should always remove the
-corresponding marks file (usually named `.marks' in the nnml group
-directory, but see `nnml-marks-file-name') for the group.  Then the
-marks file will be regenerated properly by Gnus.")
-
 (defvoo nnml-prepare-save-mail-hook nil
   "Hook run narrowed to an article before saving.")
 
 (defvoo nnml-inhibit-expiry nil
   "If non-nil, inhibit expiry.")
 
-(defvoo nnml-use-compressed-files nil
-  "If non-nil, allow using compressed message files.")
 
 \f
 
@@ -90,7 +73,6 @@ marks file will be regenerated properly by Gnus.")
   "nnml version.")
 
 (defvoo nnml-nov-file-name ".overview")
-(defvoo nnml-marks-file-name ".marks")
 
 (defvoo nnml-current-directory nil)
 (defvoo nnml-current-group nil)
@@ -104,13 +86,8 @@ marks file will be regenerated properly by Gnus.")
 
 (defvar nnml-nov-buffer-file-name nil)
 
-(defvoo nnml-file-coding-system nnmail-file-coding-system)
-
-(defvoo nnml-marks nil)
-
-(defvar nnml-marks-modtime (gnus-make-hashtable))
-
 \f
+
 ;;; Interface functions.
 
 (nnoo-define-basics nnml)
@@ -120,11 +97,11 @@ marks file will be regenerated properly by Gnus.")
     (save-excursion
       (set-buffer nntp-server-buffer)
       (erase-buffer)
-      (let* ((file nil)
-            (number (length sequence))
-            (count 0)
-            (file-name-coding-system nnmail-pathname-coding-system)
-            beg article)
+      (let ((file nil)
+           (number (length sequence))
+           (count 0)
+           (pathname-coding-system 'binary)
+           beg article)
        (if (stringp (car sequence))
            'headers
          (if (nnml-retrieve-headers-with-nov sequence fetch-old)
@@ -139,7 +116,7 @@ marks file will be regenerated properly by Gnus.")
                (setq beg (point))
                (nnheader-insert-head file)
                (goto-char beg)
-               (if (re-search-forward "\n\r?\n" nil t)
+               (if (search-forward "\n\n" nil t)
                    (forward-char -1)
                  (goto-char (point-max))
                  (insert "\n\n"))
@@ -163,7 +140,9 @@ marks file will be regenerated properly by Gnus.")
 (deffoo nnml-open-server (server &optional defs)
   (nnoo-change-server 'nnml server defs)
   (when (not (file-exists-p nnml-directory))
-    (ignore-errors (make-directory nnml-directory t)))
+    (condition-case ()
+       (make-directory nnml-directory t)
+      (error)))
   (cond
    ((not (file-exists-p nnml-directory))
     (nnml-close-server)
@@ -176,15 +155,15 @@ marks file will be regenerated properly by Gnus.")
                     server nnml-directory)
     t)))
 
-(deffoo nnml-request-regenerate (server)
+(defun nnml-request-regenerate (server)
   (nnml-possibly-change-directory nil server)
-  (nnml-generate-nov-databases server)
+  (nnml-generate-nov-databases)
   t)
 
 (deffoo nnml-request-article (id &optional group server buffer)
   (nnml-possibly-change-directory group server)
   (let* ((nntp-server-buffer (or buffer nntp-server-buffer))
-        (file-name-coding-system nnmail-pathname-coding-system)
+        (pathname-coding-system 'binary)
         path gpath group-num)
     (if (stringp id)
        (when (and (setq group-num (nnml-find-group-number id))
@@ -204,9 +183,7 @@ marks file will be regenerated properly by Gnus.")
       (nnheader-report 'nnml "No such file: %s" path))
      ((file-directory-p path)
       (nnheader-report 'nnml "File is a directory: %s" path))
-     ((not (save-excursion (let ((nnmail-file-coding-system
-                                 nnml-file-coding-system))
-                            (nnmail-find-file path))))
+     ((not (save-excursion (nnmail-find-file path)))
       (nnheader-report 'nnml "Couldn't read file: %s" path))
      (t
       (nnheader-report 'nnml "Article %s retrieved" id)
@@ -215,7 +192,7 @@ marks file will be regenerated properly by Gnus.")
            (string-to-int (file-name-nondirectory path)))))))
 
 (deffoo nnml-request-group (group &optional server dont-check)
-  (let ((file-name-coding-system nnmail-pathname-coding-system))
+  (let ((pathname-coding-system 'binary))
     (cond
      ((not (nnml-possibly-change-directory group server))
       (nnheader-report 'nnml "Invalid group (no such directory)"))
@@ -248,7 +225,6 @@ marks file will be regenerated properly by Gnus.")
   t)
 
 (deffoo nnml-request-create-group (group &optional server args)
-  (nnml-possibly-change-directory nil server)
   (nnmail-activate 'nnml)
   (cond
    ((assoc group nnml-group-alist)
@@ -263,7 +239,7 @@ marks file will be regenerated properly by Gnus.")
            nnml-group-alist)
       (nnml-possibly-create-directory group)
       (nnml-possibly-change-directory group server)
-      (let ((articles (nnml-directory-articles nnml-current-directory)))
+      (let ((articles (nnheader-directory-articles nnml-current-directory)))
        (when articles
          (setcar active (apply 'min articles))
          (setcdr active (apply 'max articles))))
@@ -273,8 +249,9 @@ marks file will be regenerated properly by Gnus.")
 (deffoo nnml-request-list (&optional server)
   (save-excursion
     (let ((nnmail-file-coding-system nnmail-active-file-coding-system)
-         (file-name-coding-system nnmail-pathname-coding-system))
-      (nnmail-find-file nnml-active-file))
+         (pathname-coding-system 'binary)) ; for XEmacs/mule
+      (nnmail-find-file nnml-active-file)
+      )
     (setq nnml-group-alist (nnmail-get-active))
     t))
 
@@ -285,10 +262,11 @@ marks file will be regenerated properly by Gnus.")
   (save-excursion
     (nnmail-find-file nnml-newsgroups-file)))
 
-(deffoo nnml-request-expire-articles (articles group &optional server force)
+(deffoo nnml-request-expire-articles (articles group
+                                              &optional server force)
   (nnml-possibly-change-directory group server)
   (let ((active-articles
-        (nnml-directory-articles nnml-current-directory))
+        (nnheader-directory-articles nnml-current-directory))
        (is-old t)
        article rest mod-time number)
     (nnmail-activate 'nnml)
@@ -299,32 +277,22 @@ marks file will be regenerated properly by Gnus.")
     (setq articles (gnus-sorted-intersection articles active-articles))
 
     (while (and articles is-old)
-      (if (and (setq article (nnml-article-to-file
-                             (setq number (pop articles))))
-              (setq mod-time (nth 5 (file-attributes article)))
-              (nnml-deletable-article-p group number)
-              (setq is-old (nnmail-expired-article-p group mod-time force
-                                                     nnml-inhibit-expiry)))
-         (progn
-           ;; Allow a special target group.
-           (unless (eq nnmail-expiry-target 'delete)
-             (with-temp-buffer
-               (nnml-request-article number group server (current-buffer))
-               (let (nnml-current-directory
-                     nnml-current-group
-                     nnml-article-file-alist)
-                 (nnmail-expiry-target-group nnmail-expiry-target group)))
-             ;; Maybe directory is changed during nnmail-expiry-target-group.
-             (nnml-possibly-change-directory group server))
-           (nnheader-message 5 "Deleting article %s in %s"
-                             number group)
-           (condition-case ()
-               (funcall nnmail-delete-file-function article)
-             (file-error
-              (push number rest)))
-           (setq active-articles (delq number active-articles))
-           (nnml-nov-delete-article group number))
-       (push number rest)))
+      (when (setq article (nnml-article-to-file (setq number (pop articles))))
+       (when (setq mod-time (nth 5 (file-attributes article)))
+         (if (and (nnml-deletable-article-p group number)
+                  (setq is-old
+                        (nnmail-expired-article-p group mod-time force
+                                                  nnml-inhibit-expiry)))
+             (progn
+               (nnheader-message 5 "Deleting article %s in %s"
+                                 article group)
+               (condition-case ()
+                   (funcall nnmail-delete-file-function article)
+                 (file-error
+                  (push number rest)))
+               (setq active-articles (delq number active-articles))
+               (nnml-nov-delete-article group number))
+           (push number rest)))))
     (let ((active (nth 1 (assoc group nnml-group-alist))))
       (when active
        (setcar active (or (and active-articles
@@ -335,7 +303,7 @@ marks file will be regenerated properly by Gnus.")
     (nconc rest articles)))
 
 (deffoo nnml-request-move-article
-    (article group server accept-form &optional last)
+  (article group server accept-form &optional last)
   (let ((buf (get-buffer-create " *nnml move*"))
        result)
     (nnml-possibly-change-directory group server)
@@ -343,15 +311,12 @@ marks file will be regenerated properly by Gnus.")
     (and
      (nnml-deletable-article-p group article)
      (nnml-request-article article group server)
-     (let (nnml-current-directory
-          nnml-current-group
-          nnml-article-file-alist)
-       (save-excursion
-        (set-buffer buf)
-        (insert-buffer-substring nntp-server-buffer)
-        (setq result (eval accept-form))
-        (kill-buffer (current-buffer))
-        result))
+     (save-excursion
+       (set-buffer buf)
+       (insert-buffer-substring nntp-server-buffer)
+       (setq result (eval accept-form))
+       (kill-buffer (current-buffer))
+       result)
      (progn
        (nnml-possibly-change-directory group server)
        (condition-case ()
@@ -369,10 +334,7 @@ marks file will be regenerated properly by Gnus.")
   (nnmail-check-syntax)
   (let (result)
     (when nnmail-cache-accepted-message-ids
-      (nnmail-cache-insert (nnmail-fetch-field "message-id") 
-                          group
-                          (nnmail-fetch-field "subject")
-                          (nnmail-fetch-field "from")))
+      (nnmail-cache-insert (nnmail-fetch-field "message-id")))
     (if (stringp group)
        (and
         (nnmail-activate 'nnml)
@@ -394,9 +356,6 @@ marks file will be regenerated properly by Gnus.")
         (nnml-save-nov))))
     result))
 
-(deffoo nnml-request-post (&optional server)
-  (nnmail-do-request-post 'nnml-request-accept-article server))
-
 (deffoo nnml-request-replace-article (article group buffer)
   (nnml-possibly-change-directory group)
   (save-excursion
@@ -405,14 +364,16 @@ marks file will be regenerated properly by Gnus.")
     (let ((chars (nnmail-insert-lines))
          (art (concat (int-to-string article) "\t"))
          headers)
-      (when (ignore-errors
-             (nnmail-write-region
-              (point-min) (point-max)
-              (or (nnml-article-to-file article)
-                  (expand-file-name (int-to-string article)
-                                    nnml-current-directory))
-              nil (if (nnheader-be-verbose 5) nil 'nomesg))
-             t)
+      (when (condition-case ()
+               (progn
+                 (nnmail-write-region
+                  (point-min) (point-max)
+                  (or (nnml-article-to-file article)
+                      (concat nnml-current-directory
+                              (int-to-string article)))
+                  nil (if (nnheader-be-verbose 5) nil 'nomesg))
+                 t)
+             (error nil))
        (setq headers (nnml-parse-head chars article))
        ;; Replace the NOV line in the NOV file.
        (save-excursion
@@ -421,7 +382,8 @@ marks file will be regenerated properly by Gnus.")
          (if (or (looking-at art)
                  (search-forward (concat "\n" art) nil t))
              ;; Delete the old NOV line.
-             (gnus-delete-line)
+             (delete-region (progn (beginning-of-line) (point))
+                            (progn (forward-line 1) (point)))
            ;; The line isn't here, so we have to find out where
            ;; we should insert it.  (This situation should never
            ;; occur, but one likes to make sure...)
@@ -444,8 +406,7 @@ marks file will be regenerated properly by Gnus.")
           (directory-files
            nnml-current-directory t
            (concat nnheader-numerical-short-files
-                   "\\|" (regexp-quote nnml-nov-file-name) "$"
-                   "\\|" (regexp-quote nnml-marks-file-name) "$")))
+                   "\\|" (regexp-quote nnml-nov-file-name) "$")))
          article)
       (while articles
        (setq article (pop articles))
@@ -453,7 +414,9 @@ marks file will be regenerated properly by Gnus.")
          (nnheader-message 5 "Deleting article %s in %s..." article group)
          (funcall nnmail-delete-file-function article))))
     ;; Try to delete the directory itself.
-    (ignore-errors (delete-directory nnml-current-directory)))
+    (condition-case ()
+       (delete-directory nnml-current-directory)
+      (error nil)))
   ;; Remove the group from all structures.
   (setq nnml-group-alist
        (delq (assoc group nnml-group-alist) nnml-group-alist)
@@ -467,9 +430,11 @@ marks file will be regenerated properly by Gnus.")
   (nnml-possibly-change-directory group server)
   (let ((new-dir (nnmail-group-pathname new-name nnml-directory))
        (old-dir (nnmail-group-pathname group nnml-directory)))
-    (when (ignore-errors
-           (make-directory new-dir t)
-           t)
+    (when (condition-case ()
+             (progn
+               (make-directory new-dir t)
+               t)
+           (error nil))
       ;; We move the articles file by file instead of renaming
       ;; the directory -- there may be subgroups in this group.
       ;; One might be more clever, I guess.
@@ -483,12 +448,10 @@ marks file will be regenerated properly by Gnus.")
       (let ((overview (concat old-dir nnml-nov-file-name)))
        (when (file-exists-p overview)
          (rename-file overview (concat new-dir nnml-nov-file-name))))
-      ;; Move .marks file.
-      (let ((marks (concat old-dir nnml-marks-file-name)))
-       (when (file-exists-p marks)
-         (rename-file marks (concat new-dir nnml-marks-file-name))))
       (when (<= (length (directory-files old-dir)) 2)
-       (ignore-errors (delete-directory old-dir)))
+       (condition-case ()
+           (delete-directory old-dir)
+         (error nil)))
       ;; That went ok, so we change the internal structures.
       (let ((entry (assoc group nnml-group-alist)))
        (when entry
@@ -506,7 +469,7 @@ marks file will be regenerated properly by Gnus.")
      ((not (file-exists-p file))
       (nnheader-report 'nnml "File %s does not exist" file))
      (t
-      (with-temp-file file
+      (nnheader-temp-write file
        (nnheader-insert-file-contents file)
        (nnmail-replace-status name value))
       t))))
@@ -517,19 +480,15 @@ marks file will be regenerated properly by Gnus.")
 (defun nnml-article-to-file (article)
   (nnml-update-file-alist)
   (let (file)
-    (if (setq file
-             (if nnml-use-compressed-files
-                 (cdr (assq article nnml-article-file-alist))
-               (number-to-string article)))
-       (expand-file-name file nnml-current-directory)
-      (when (not nnheader-directory-files-is-safe)
-       ;; Just to make sure nothing went wrong when reading over NFS --
-       ;; check once more.
-       (when (file-exists-p
-              (setq file (expand-file-name (number-to-string article)
-                                           nnml-current-directory)))
-         (nnml-update-file-alist t)
-         file)))))
+    (if (setq file (cdr (assq article nnml-article-file-alist)))
+       (concat nnml-current-directory file)
+      ;; Just to make sure nothing went wrong when reading over NFS --
+      ;; check once more.
+      (when (file-exists-p
+            (setq file (expand-file-name (number-to-string article)
+                                         nnml-current-directory)))
+       (nnml-update-file-alist t)
+       file))))
 
 (defun nnml-deletable-article-p (group article)
   "Say whether ARTICLE in GROUP can be deleted."
@@ -544,6 +503,7 @@ marks file will be regenerated properly by Gnus.")
 (defun nnml-find-group-number (id)
   (save-excursion
     (set-buffer (get-buffer-create " *nnml id*"))
+    (buffer-disable-undo (current-buffer))
     (let ((alist nnml-group-alist)
          number)
       ;; We want to look through all .overview files, but we want to
@@ -551,7 +511,7 @@ marks file will be regenerated properly by Gnus.")
       ;; likely that the article we are looking for is in that group.
       (if (setq number (nnml-find-id nnml-current-group id))
          (cons nnml-current-group number)
-      ;; It wasn't there, so we look through the other groups as well.
+       ;; It wasn't there, so we look through the other groups as well.
        (while (and (not number)
                    alist)
          (or (string= (caar alist) nnml-current-group)
@@ -563,8 +523,8 @@ marks file will be regenerated properly by Gnus.")
 
 (defun nnml-find-id (group id)
   (erase-buffer)
-  (let ((nov (expand-file-name nnml-nov-file-name
-                              (nnmail-group-pathname group nnml-directory)))
+  (let ((nov (concat (nnmail-group-pathname group nnml-directory)
+                    nnml-nov-file-name))
        number found)
     (when (file-exists-p nov)
       (nnheader-insert-file-contents nov)
@@ -578,13 +538,15 @@ marks file will be regenerated properly by Gnus.")
          (setq found t)
          ;; We return the article number.
          (setq number
-               (ignore-errors (read (current-buffer))))))
+               (condition-case ()
+                   (read (current-buffer))
+                 (error nil)))))
       number)))
 
 (defun nnml-retrieve-headers-with-nov (articles &optional fetch-old)
   (if (or gnus-nov-is-evil nnml-nov-is-evil)
       nil
-    (let ((nov (expand-file-name nnml-nov-file-name nnml-current-directory)))
+    (let ((nov (concat nnml-current-directory nnml-nov-file-name)))
       (when (file-exists-p nov)
        (save-excursion
          (set-buffer nntp-server-buffer)
@@ -606,7 +568,7 @@ marks file will be regenerated properly by Gnus.")
   (if (not group)
       t
     (let ((pathname (nnmail-group-pathname group nnml-directory))
-         (file-name-coding-system nnmail-pathname-coding-system))
+         (pathname-coding-system 'binary))
       (when (not (equal pathname nnml-current-directory))
        (setq nnml-current-directory pathname
              nnml-current-group group
@@ -614,10 +576,15 @@ marks file will be regenerated properly by Gnus.")
       (file-exists-p nnml-current-directory))))
 
 (defun nnml-possibly-create-directory (group)
-  (let ((dir (nnmail-group-pathname group nnml-directory)))
-    (unless (file-exists-p dir)
-      (make-directory (directory-file-name dir) t)
-      (nnheader-message 5 "Creating mail directory %s" dir))))
+  (let (dir dirs)
+    (setq dir (nnmail-group-pathname group nnml-directory))
+    (while (not (file-directory-p dir))
+      (push dir dirs)
+      (setq dir (file-name-directory (directory-file-name dir))))
+    (while dirs
+      (make-directory (directory-file-name (car dirs)))
+      (nnheader-message 5 "Creating mail directory %s" (car dirs))
+      (setq dirs (cdr dirs)))))
 
 (defun nnml-save-mail (group-art)
   "Called narrowed to an article."
@@ -670,7 +637,7 @@ marks file will be regenerated properly by Gnus.")
       (unless nnml-article-file-alist
        (setq nnml-article-file-alist
              (sort
-              (nnml-current-group-article-to-file-alist)
+              (nnheader-article-to-file-alist nnml-current-directory)
               'car-less-than-car)))
       (setq active
            (if nnml-article-file-alist
@@ -680,8 +647,8 @@ marks file will be regenerated properly by Gnus.")
       (push (list group active) nnml-group-alist))
     (setcdr active (1+ (cdr active)))
     (while (file-exists-p
-           (expand-file-name (int-to-string (cdr active))
-                             (nnmail-group-pathname group nnml-directory)))
+           (concat (nnmail-group-pathname group nnml-directory)
+                   (int-to-string (cdr active))))
       (setcdr active (1+ (cdr active))))
     (cdr active)))
 
@@ -694,7 +661,7 @@ marks file will be regenerated properly by Gnus.")
     (nnheader-insert-nov headers)))
 
 (defsubst nnml-header-value ()
-  (buffer-substring (match-end 0) (gnus-point-at-eol)))
+  (buffer-substring (match-end 0) (progn (end-of-line) (point))))
 
 (defun nnml-parse-head (chars &optional number)
   "Parse the head of the current buffer."
@@ -703,30 +670,29 @@ marks file will be regenerated properly by Gnus.")
       (unless (zerop (buffer-size))
        (narrow-to-region
         (goto-char (point-min))
-        (if (re-search-forward "\n\r?\n" nil t)
-            (1- (point))
-          (point-max))))
-      (let ((headers (nnheader-parse-naked-head)))
+        (if (search-forward "\n\n" nil t) (1- (point)) (point-max))))
+      ;; Fold continuation lines.
+      (goto-char (point-min))
+      (while (re-search-forward "\\(\r?\n[ \t]+\\)+" nil t)
+       (replace-match " " t t))
+      ;; Remove any tabs; they are too confusing.
+      (subst-char-in-region (point-min) (point-max) ?\t ? )
+      (let ((headers (nnheader-parse-head t)))
        (mail-header-set-chars headers chars)
        (mail-header-set-number headers number)
        headers))))
 
-(defun nnml-get-nov-buffer (group)
-  (let ((buffer (get-buffer-create (format " *nnml overview %s*" group))))
-    (save-excursion
-      (set-buffer buffer)
-      (set (make-local-variable 'nnml-nov-buffer-file-name)
-          (expand-file-name
-           nnml-nov-file-name
-           (nnmail-group-pathname group nnml-directory)))
-      (erase-buffer)
-      (when (file-exists-p nnml-nov-buffer-file-name)
-       (nnheader-insert-file-contents nnml-nov-buffer-file-name)))
-    buffer))
-
 (defun nnml-open-nov (group)
   (or (cdr (assoc group nnml-nov-buffer-alist))
-      (let ((buffer (nnml-get-nov-buffer group)))
+      (let ((buffer (get-buffer-create (format " *nnml overview %s*" group))))
+       (save-excursion
+         (set-buffer buffer)
+         (set (make-local-variable 'nnml-nov-buffer-file-name)
+              (concat (nnmail-group-pathname group nnml-directory)
+                      nnml-nov-file-name))
+         (erase-buffer)
+         (when (file-exists-p nnml-nov-buffer-file-name)
+           (nnheader-insert-file-contents nnml-nov-buffer-file-name)))
        (push (cons group buffer) nnml-nov-buffer-alist)
        buffer)))
 
@@ -736,21 +702,20 @@ marks file will be regenerated properly by Gnus.")
       (when (buffer-name (cdar nnml-nov-buffer-alist))
        (set-buffer (cdar nnml-nov-buffer-alist))
        (when (buffer-modified-p)
-         (nnmail-write-region (point-min) (point-max)
-                              nnml-nov-buffer-file-name nil 'nomesg))
+         (nnmail-write-region 1 (point-max) nnml-nov-buffer-file-name
+                              nil 'nomesg))
        (set-buffer-modified-p nil)
        (kill-buffer (current-buffer)))
       (setq nnml-nov-buffer-alist (cdr nnml-nov-buffer-alist)))))
 
 ;;;###autoload
-(defun nnml-generate-nov-databases (&optional server)
+(defun nnml-generate-nov-databases ()
   "Generate NOV databases in all nnml directories."
-  (interactive (list (or (nnoo-current-server 'nnml) "")))
+  (interactive)
   ;; Read the active file to make sure we don't re-use articles
   ;; numbers in empty groups.
   (nnmail-activate 'nnml)
-  (unless (nnml-server-opened server)
-    (nnml-open-server server))
+  (nnml-open-server (or (nnoo-current-server 'nnml) ""))
   (setq nnml-directory (expand-file-name nnml-directory))
   ;; Recurse down the directories.
   (nnml-generate-nov-databases-1 nnml-directory nil t)
@@ -768,7 +733,7 @@ marks file will be regenerated properly by Gnus.")
     (let ((dirs (directory-files dir t nil t))
          dir)
       (while (setq dir (pop dirs))
-       (when (and (not (string-match "^\\." (file-name-nondirectory dir)))
+       (when (and (not (member (file-name-nondirectory dir) '("." "..")))
                   (file-directory-p dir))
          (nnml-generate-nov-databases-1 dir seen))))
     ;; Do this directory.
@@ -786,21 +751,18 @@ marks file will be regenerated properly by Gnus.")
        (unless no-active
          (nnmail-save-active nnml-group-alist nnml-active-file))))))
 
-(eval-when-compile (defvar files))
+(defvar files)
 (defun nnml-generate-active-info (dir)
   ;; Update the active info for this group.
-  (let* ((group (nnheader-file-to-group
-                (directory-file-name dir) nnml-directory))
-        (entry (assoc group nnml-group-alist))
-        (last (or (caadr entry) 0)))
-    (setq nnml-group-alist (delq entry nnml-group-alist))
+  (let ((group (nnheader-file-to-group
+               (directory-file-name dir) nnml-directory)))
+    (setq nnml-group-alist
+         (delq (assoc group nnml-group-alist) nnml-group-alist))
     (push (list group
-               (cons (or (caar files) (1+ last))
-                     (max last
-                          (or (let ((f files))
-                                (while (cdr f) (setq f (cdr f)))
-                                (caar f))
-                              0))))
+               (cons (caar files)
+                     (let ((f files))
+                       (while (cdr f) (setq f (cdr f)))
+                       (caar f))))
          nnml-group-alist)))
 
 (defun nnml-generate-nov-file (dir files)
@@ -811,7 +773,7 @@ marks file will be regenerated properly by Gnus.")
     (save-excursion
       ;; Init the nov buffer.
       (set-buffer nov-buffer)
-      (buffer-disable-undo)
+      (buffer-disable-undo (current-buffer))
       (erase-buffer)
       (set-buffer nntp-server-buffer)
       ;; Delete the old NOV file.
@@ -824,9 +786,9 @@ marks file will be regenerated properly by Gnus.")
          (narrow-to-region
           (goto-char (point-min))
           (progn
-            (re-search-forward "\n\r?\n" nil t)
+            (search-forward "\n\n" nil t)
             (setq chars (- (point-max) (point)))
-            (max (point-min) (1- (point)))))
+            (max 1 (1- (point)))))
          (unless (zerop (buffer-size))
            (goto-char (point-min))
            (setq headers (nnml-parse-head chars (caar files)))
@@ -838,7 +800,7 @@ marks file will be regenerated properly by Gnus.")
        (setq files (cdr files)))
       (save-excursion
        (set-buffer nov-buffer)
-       (nnmail-write-region (point-min) (point-max) nov nil 'nomesg)
+       (nnmail-write-region 1 (point-max) nov nil 'nomesg)
        (kill-buffer (current-buffer))))))
 
 (defun nnml-nov-delete-article (group article)
@@ -858,158 +820,10 @@ marks file will be regenerated properly by Gnus.")
     t))
 
 (defun nnml-update-file-alist (&optional force)
-  (when nnml-use-compressed-files
-    (when (or (not nnml-article-file-alist)
-             force)
-      (setq nnml-article-file-alist
-           (nnml-current-group-article-to-file-alist)))))
-
-(defun nnml-directory-articles (dir)
-  "Return a list of all article files in a directory.
-Use the nov database for that directory if available."
-  (if (or gnus-nov-is-evil nnml-nov-is-evil
-         (not (file-exists-p
-               (expand-file-name nnml-nov-file-name dir))))
-      (nnheader-directory-articles dir)
-    ;; build list from .overview if available
-    ;; We would use nnml-open-nov, except that nnml-nov-buffer-alist is
-    ;; defvoo'd, and we might get called when it hasn't been swapped in.
-    (save-excursion
-      (let ((list nil)
-           art
-           (buffer (nnml-get-nov-buffer nnml-current-group)))
-       (set-buffer buffer)
-       (goto-char (point-min))
-       (while (not (eobp))
-         (setq art (read (current-buffer)))
-         (push art list)
-         (forward-line 1))
-       list))))
-
-(defun nnml-current-group-article-to-file-alist ()
-  "Return an alist of article/file pairs in the current group.
-Use the nov database for the current group if available."
-  (if (or nnml-use-compressed-files
-         gnus-nov-is-evil
-         nnml-nov-is-evil
-         (not (file-exists-p
-               (expand-file-name nnml-nov-file-name
-                                 nnml-current-directory))))
-      (nnheader-article-to-file-alist nnml-current-directory)
-    ;; build list from .overview if available
-    (save-excursion
-      (let ((alist nil)
-           (buffer (nnml-get-nov-buffer nnml-current-group))
-           art)
-       (set-buffer buffer)
-       (goto-char (point-min))
-       (while (not (eobp))
-         (setq art (read (current-buffer)))
-         ;; assume file name is unadorned (ie. not compressed etc)
-         (push (cons art (int-to-string art)) alist)
-         (forward-line 1))
-       alist))))
-
-(deffoo nnml-request-set-mark (group actions &optional server)
-  (nnml-possibly-change-directory group server)
-  (unless nnml-marks-is-evil
-    (nnml-open-marks group server)
-    (dolist (action actions)
-      (let ((range (nth 0 action))
-           (what  (nth 1 action))
-           (marks (nth 2 action)))
-       (assert (or (eq what 'add) (eq what 'del)) t
-               "Unknown request-set-mark action: %s" what)
-       (dolist (mark marks)
-         (setq nnml-marks (gnus-update-alist-soft
-                           mark
-                           (funcall (if (eq what 'add) 'gnus-range-add
-                                      'gnus-remove-from-range)
-                                    (cdr (assoc mark nnml-marks)) range)
-                           nnml-marks)))))
-    (nnml-save-marks group server))
-  nil)
-
-(deffoo nnml-request-update-info (group info &optional server)
-  (nnml-possibly-change-directory group server)
-  (when (and (not nnml-marks-is-evil) (nnml-marks-changed-p group))
-    (nnheader-message 8 "Updating marks for %s..." group)
-    (nnml-open-marks group server)
-    ;; Update info using `nnml-marks'.
-    (mapcar (lambda (pred)
-             (unless (memq (cdr pred) gnus-article-unpropagated-mark-lists)
-               (gnus-info-set-marks
-                info
-                (gnus-update-alist-soft
-                 (cdr pred)
-                 (cdr (assq (cdr pred) nnml-marks))
-                 (gnus-info-marks info))
-                t)))
-           gnus-article-mark-lists)
-    (let ((seen (cdr (assq 'read nnml-marks))))
-      (gnus-info-set-read info
-                         (if (and (integerp (car seen))
-                                  (null (cdr seen)))
-                             (list (cons (car seen) (car seen)))
-                           seen)))
-    (nnheader-message 8 "Updating marks for %s...done" group))
-  info)
-
-(defun nnml-marks-changed-p (group)
-  (let ((file (expand-file-name nnml-marks-file-name
-                               (nnmail-group-pathname group nnml-directory))))
-    (if (null (gnus-gethash file nnml-marks-modtime))
-       t ;; never looked at marks file, assume it has changed
-      (not (equal (gnus-gethash file nnml-marks-modtime)
-                 (nth 5 (file-attributes file)))))))
-
-(defun nnml-save-marks (group server)
-  (let ((file-name-coding-system nnmail-pathname-coding-system)
-       (file (expand-file-name nnml-marks-file-name
-                               (nnmail-group-pathname group nnml-directory))))
-    (condition-case err
-       (progn
-         (nnml-possibly-create-directory group)
-         (with-temp-file file
-           (erase-buffer)
-           (gnus-prin1 nnml-marks)
-           (insert "\n"))
-         (gnus-sethash file
-                       (nth 5 (file-attributes file))
-                       nnml-marks-modtime))
-      (error (or (gnus-yes-or-no-p
-                 (format "Could not write to %s (%s).  Continue? " file err))
-                (error "Cannot write to %s (%s)" err))))))
-
-(defun nnml-open-marks (group server)
-  (let ((file (expand-file-name
-              nnml-marks-file-name
-              (nnmail-group-pathname group nnml-directory))))
-    (if (file-exists-p file)
-       (condition-case err
-           (with-temp-buffer
-             (gnus-sethash file (nth 5 (file-attributes file))
-                           nnml-marks-modtime)
-             (nnheader-insert-file-contents file)
-             (setq nnml-marks (read (current-buffer)))
-             (dolist (el gnus-article-unpropagated-mark-lists)
-               (setq nnml-marks (gnus-remassoc el nnml-marks))))
-         (error (or (gnus-yes-or-no-p
-                     (format "Error reading nnml marks file %s (%s).  Continuing will use marks from .newsrc.eld.  Continue? " file err))
-                    (error "Cannot read nnml marks file %s (%s)" file err))))
-      ;; User didn't have a .marks file.  Probably first time
-      ;; user of the .marks stuff.  Bootstrap it from .newsrc.eld.
-      (let ((info (gnus-get-info
-                  (gnus-group-prefixed-name
-                   group
-                   (gnus-server-to-method (format "nnml:%s" server))))))
-       (nnheader-message 7 "Bootstrapping marks for %s..." group)
-       (setq nnml-marks (gnus-info-marks info))
-       (push (cons 'read (gnus-info-read info)) nnml-marks)
-       (dolist (el gnus-article-unpropagated-mark-lists)
-         (setq nnml-marks (gnus-remassoc el nnml-marks)))
-       (nnml-save-marks group server)
-       (nnheader-message 7 "Bootstrapping marks for %s...done" group)))))
+  (when (or (not nnml-article-file-alist)
+           force)
+    (setq nnml-article-file-alist
+         (nnheader-article-to-file-alist nnml-current-directory))))
 
 (provide 'nnml)