*** empty log message ***
[gnus] / lisp / nnfolder.el
index eab9621..9c8470e 100644 (file)
@@ -116,22 +116,21 @@ time saver for large mailboxes.")
            (setq article (car articles))
            (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")))
+           (when (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))
+             (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 articles (cdr articles)))
 
          (set-buffer nntp-server-buffer)
@@ -142,7 +141,7 @@ time saver for large mailboxes.")
   (nnoo-change-server 'nnfolder server defs)
   (when (not (file-exists-p nnfolder-directory))
     (condition-case ()
-       (make-directory nnfolder-directory t)
+       (gnus-make-directory nnfolder-directory)
       (error t)))
   (cond 
    ((not (file-exists-p nnfolder-directory))
@@ -171,33 +170,33 @@ time saver for large mailboxes.")
   (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 "^" message-unix-mail-delimiter) nil t)
-         (setq start (point))
-         (forward-line 1)
-         (or (and (re-search-forward 
-                   (concat "^" message-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)
+    (when (search-forward (nnfolder-article-string article) nil t)
+      (let (start stop)
+       (re-search-backward (concat "^" message-unix-mail-delimiter) nil t)
+       (setq start (point))
+       (forward-line 1)
+       (unless (and (re-search-forward 
+                     (concat "^" message-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))
-           (if (numberp article) 
-               (cons nnfolder-current-group article)
-             (goto-char (point-min))
-             (search-forward (concat "\n" nnfolder-article-marker))
-             (cons nnfolder-current-group
-                   (string-to-int 
-                    (buffer-substring 
-                     (point) (progn (end-of-line) (point)))))))))))
+           (search-forward (concat "\n" nnfolder-article-marker))
+           (cons nnfolder-current-group
+                 (string-to-int 
+                  (buffer-substring 
+                   (point) (progn (end-of-line) (point)))))))))))
 
 (deffoo nnfolder-request-group (group &optional server dont-check)
   (save-excursion
@@ -275,7 +274,7 @@ time saver for large mailboxes.")
        nnfolder-current-buffer nil)
   t)
 
-(deffoo nnfolder-request-create-group (group &optional server
+(deffoo nnfolder-request-create-group (group &optional server args)
   (nnfolder-possibly-change-group nil server)
   (nnmail-activate 'nnfolder)
   (when group 
@@ -310,18 +309,18 @@ time saver for large mailboxes.")
       (set-buffer nnfolder-current-buffer)
       (while (and articles is-old)
        (goto-char (point-min))
-       (if (search-forward (nnfolder-article-string (car articles)) nil t)
-           (if (setq is-old
-                     (nnmail-expired-article-p 
-                      newsgroup
-                      (buffer-substring 
-                       (point) (progn (end-of-line) (point))) 
-                      force nnfolder-inhibit-expiry))
-               (progn
-                 (nnheader-message 5 "Deleting article %d..." 
-                                   (car articles) newsgroup)
-                 (nnfolder-delete-mail))
-             (setq rest (cons (car articles) rest))))
+       (when (search-forward (nnfolder-article-string (car articles)) nil t)
+         (if (setq is-old
+                   (nnmail-expired-article-p 
+                    newsgroup
+                    (buffer-substring 
+                     (point) (progn (end-of-line) (point)))
+                    force nnfolder-inhibit-expiry))
+             (progn
+               (nnheader-message 5 "Deleting article %d..." 
+                                 (car articles) newsgroup)
+               (nnfolder-delete-mail))
+           (push (car articles) rest)))
        (setq articles (cdr articles)))
       (nnfolder-save-buffer)
       ;; Find the lowest active article in this group.
@@ -342,7 +341,6 @@ time saver for large mailboxes.")
 
 (deffoo nnfolder-request-move-article
   (article group server accept-form &optional last)
-  (nnfolder-possibly-change-group group server)
   (let ((buf (get-buffer-create " *nnfolder move*"))
        result)
     (and 
@@ -365,15 +363,16 @@ time saver for large mailboxes.")
        (nnfolder-possibly-change-group group server)
        (set-buffer nnfolder-current-buffer)
        (goto-char (point-min))
-       (if (search-forward (nnfolder-article-string article) nil t)
-          (nnfolder-delete-mail))
+       (when (search-forward (nnfolder-article-string article) nil t)
+        (nnfolder-delete-mail))
        (and last (nnfolder-save-buffer))))
     result))
 
 (deffoo nnfolder-request-accept-article (group &optional server last)
   (nnfolder-possibly-change-group group server)
   (nnmail-check-syntax)
-  (and (stringp group) (nnfolder-possibly-change-group group))
+  (when (stringp group)
+    (nnfolder-possibly-change-group group))
   (let ((buf (current-buffer))
        result)
     (goto-char (point-min))
@@ -388,7 +387,11 @@ time saver for large mailboxes.")
        (forward-line -1)
        (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)))))
+       (setq result
+            (car (nnfolder-save-mail
+                  (if (stringp group)
+                      (list (cons group (nnfolder-active-number group)))
+                    (nnmail-article-group 'nnfolder-active-number))))))
      (save-excursion
        (set-buffer nnfolder-current-buffer)
        (and last (nnfolder-save-buffer))))
@@ -482,15 +485,13 @@ time saver for large mailboxes.")
     (nnfolder-open-server server))
   (when (and group (or nnfolder-current-buffer
                       (not (equal group nnfolder-current-group))))
-    (unless (file-exists-p nnfolder-directory)
-      (make-directory (directory-file-name nnfolder-directory) t))
+    (gnus-make-directory (directory-file-name nnfolder-directory))
     (nnfolder-possibly-activate-groups nil)
     (or (assoc group nnfolder-group-alist)
        (not (file-exists-p
              (nnfolder-group-pathname group)))
        (progn
-         (setq nnfolder-group-alist 
-               (cons (list group (cons 1 0)) nnfolder-group-alist))
+         (push (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)
@@ -502,52 +503,42 @@ time saver for large mailboxes.")
        ;; If we have to change groups, see if we don't already have the mbox
        ;; in memory.  If we do, verify the modtime and destroy the mbox if
        ;; needed so we can rescan it.
-       (if (setq inf (assoc group nnfolder-buffer-alist))
-           (setq nnfolder-current-buffer (nth 1 inf)))
+       (when (setq inf (assoc group nnfolder-buffer-alist))
+         (setq nnfolder-current-buffer (nth 1 inf)))
 
        ;; 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 (and nnfolder-current-buffer
-                         (buffer-name nnfolder-current-buffer)))
-               (not (and (bufferp nnfolder-current-buffer)
-                         (verify-visited-file-modtime 
-                          nnfolder-current-buffer))))
-           (progn
-             (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))
-             (setq inf nil)))
+       (when (or (not (and nnfolder-current-buffer
+                           (buffer-name nnfolder-current-buffer)))
+                 (not (and (bufferp nnfolder-current-buffer)
+                           (verify-visited-file-modtime 
+                            nnfolder-current-buffer))))
+         (when (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))
+         (setq inf nil))
       
-       (if inf
-           ()
+       (unless inf
          (save-excursion
            (setq file (nnfolder-group-pathname group))
-           (if (file-directory-p (file-truename file))
-               ()
+           (unless (file-directory-p (file-truename file))
              (unless (file-exists-p file)
-               (unless (file-exists-p (file-name-directory file))
-                 (make-directory (file-name-directory file) t))
-               (write-region 1 1 file t 'nomesg))
+               (gnus-make-directory (file-name-directory file))
+               (nnmail-write-region 1 1 file t 'nomesg))
              (setq nnfolder-current-buffer
                    (nnfolder-read-folder file scanning))
-             (if nnfolder-current-buffer 
-                 (progn
-                   (set-buffer nnfolder-current-buffer)
-                   (setq nnfolder-buffer-alist 
-                         (cons (list group nnfolder-current-buffer)
-                               nnfolder-buffer-alist)))))))))
+             (when nnfolder-current-buffer 
+               (set-buffer nnfolder-current-buffer)
+               (push (list group nnfolder-current-buffer)
+                     nnfolder-buffer-alist)))))))
     (setq nnfolder-current-group group)))
 
-(defun nnfolder-save-mail (&optional group)
+(defun nnfolder-save-mail (group-art-list)
   "Called narrowed to an article."
-  (let* ((nnmail-split-methods 
-         (if group (list (list group "")) nnmail-split-methods))
-        (group-art-list
-         (nreverse (nnmail-article-group 'nnfolder-active-number)))
-        (delim (concat "^" message-unix-mail-delimiter))
+  (let* ((delim (concat "^" message-unix-mail-delimiter))
         save-list group-art)
     (goto-char (point-min))
     ;; The From line may have been quoted by movemail.
@@ -604,17 +595,17 @@ time saver for large mailboxes.")
 (defun nnfolder-insert-newsgroup-line (group-art)
   (save-excursion
     (goto-char (point-min))
-    (if (search-forward "\n\n" nil t)
-       (progn
-         (forward-char -1)
-         (insert (format (concat nnfolder-article-marker "%d   %s\n")
-                         (cdr group-art) (current-time-string)))))))
+    (when (search-forward "\n\n" nil t)
+      (forward-char -1)
+      (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))
+    (when group
+      (nnfolder-possibly-change-group group))
     ;; If the group alist isn't active, activate it now.
     (nnmail-activate 'nnfolder)))
 
@@ -629,9 +620,8 @@ time saver for large mailboxes.")
              ;; 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)))
+             (push (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)))))
@@ -691,30 +681,29 @@ time saver for large mailboxes.")
          ;; file entirely for mboxes.)
          (when (or nnfolder-ignore-active-file
                    (< maxid 2))
-               (while (and (search-forward marker nil t)
-                           (re-search-forward number nil t))
-                 (let ((newnum (string-to-number (match-string 0))))
-                   (setq maxid (max maxid newnum))
-                   (setq minid (min minid newnum))))
-               (setcar active (max 1 (min minid maxid)))
-               (setcdr active (max maxid (cdr active)))
-               (goto-char (point-min)))
+           (while (and (search-forward marker nil t)
+                       (re-search-forward number nil t))
+             (let ((newnum (string-to-number (match-string 0))))
+               (setq maxid (max maxid newnum))
+               (setq minid (min minid newnum))))
+           (setcar active (max 1 (min minid maxid)))
+           (setcdr active (max maxid (cdr active)))
+           (goto-char (point-min)))
 
          ;; As long as we trust that the user will only insert unmarked mail
          ;; at the end, go to the end and search backwards for the last
          ;; marker.  Find the start of that message, and begin to search for
          ;; unmarked messages from there.
-         (if (not (or nnfolder-distrust-mbox
-                      (< maxid 2)))
-             (progn
-               (goto-char (point-max))
-               (if (not (re-search-backward marker nil t))
-                   (goto-char (point-min))
-                 (if (not (re-search-backward delim nil t))
-                     (goto-char (point-min))))))
+         (when (not (or nnfolder-distrust-mbox
+                        (< maxid 2)))
+           (goto-char (point-max))
+           (if (not (re-search-backward marker nil t))
+               (goto-char (point-min))
+             (when (not (re-search-backward delim nil t))
+               (goto-char (point-min)))))
 
          ;; Keep track of the active number on our own, and insert it back
-         ;; into the active list when we're done. Also, prime the pump to
+         ;; into the active list when we're done.  Also, prime the pump to
          ;; cut down on the number of searches we do.
          (setq end (point-marker))
          (set-marker end (or (and (re-search-forward delim nil t)
@@ -725,19 +714,18 @@ time saver for large mailboxes.")
            (goto-char end)
            ;; There may be more than one "From " line, so we skip past
            ;; them.  
-           (while (looking-at delim) 
+           (while (looking-at delim)
              (forward-line 1))
            (set-marker end (or (and (re-search-forward delim nil t)
                                     (match-beginning 0))
                                (point-max)))
            (goto-char start)
-           (if (not (search-forward marker end t))
-               (progn
-                 (narrow-to-region start end)
-                 (nnmail-insert-lines)
-                 (nnfolder-insert-newsgroup-line
-                  (cons nil (nnfolder-active-number nnfolder-current-group)))
-                 (widen))))
+           (when (not (search-forward marker end t))
+             (narrow-to-region start end)
+             (nnmail-insert-lines)
+             (nnfolder-insert-newsgroup-line
+              (cons nil (nnfolder-active-number nnfolder-current-group)))
+             (widen)))
 
          ;; Make absolutely sure that the active list reflects reality!
          (nnmail-save-active nnfolder-group-alist nnfolder-active-file)
@@ -745,7 +733,7 @@ time saver for large mailboxes.")
          (setq newscantime (visited-file-modtime))
          (if scantime
              (setcdr scantime (list newscantime))
-           (push (list nnfolder-current-group newscantime) 
+           (push (list nnfolder-current-group newscantime)
                  nnfolder-scantime-alist))
          (current-buffer))))))
 
@@ -755,15 +743,15 @@ time saver for large mailboxes.")
   (interactive)
   (nnmail-activate 'nnfolder)
   (let ((files (directory-files nnfolder-directory))
-       file)
+        file)
     (while (setq file (pop files))
       (when (and (not (backup-file-name-p file))
-                (nnheader-mail-file-mbox-p file))
-       (nnheader-message 5 "Adding group %s..." file)
-       (push (list file (cons 1 0)) nnfolder-group-alist)
-       (nnfolder-possibly-change-group file)
-;;     (nnfolder-read-folder file)
-       (nnfolder-close-group file))
+                 (nnheader-mail-file-mbox-p
+                 (concat nnfolder-directory file)))
+        (nnheader-message 5 "Adding group %s..." file)
+        (push (list file (cons 1 0)) nnfolder-group-alist)
+        (nnfolder-possibly-change-group file)
+        (nnfolder-close-group file))
       (message ""))))
 
 (defun nnfolder-group-pathname (group)