* lisp/gnus/nnml.el (nnml-files): Add prefix to dynamic var `files'.

(nnml-generate-nov-databases-directory, nnml-generate-active-info):
Update var name.
(nnml-generate-nov-file): Use dolist.
(nnml-directory-articles, nnml-current-group-article-to-file-alist):
Use with-current-buffer.

diff --git a/lisp/gnus/ChangeLog b/lisp/gnus/ChangeLog
index a7d29366cb7818156be3c6312db86a13eab93021..e82b213747b229f91d23c1b32cf4391f4aa21224 100644 (file)
--- a/lisp/gnus/ChangeLog
+++ b/lisp/gnus/ChangeLog
@@ -1,3 +1,12 @@
+2010-09-19  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * nnml.el (nnml-files): Add prefix to dynamic var `files'.
+       (nnml-generate-nov-databases-directory, nnml-generate-active-info):
+       Update var name.
+       (nnml-generate-nov-file): Use dolist.
+       (nnml-directory-articles, nnml-current-group-article-to-file-alist):
+       Use with-current-buffer.
+
 2010-09-18  Julien Danjou  <julien@danjou.info>
 
        * gnus-html.el (gnus-html-schedule-image-fetching): Fetch all images in
@@ -9,7 +18,7 @@
        the range update right.
        (nnimap-request-group): Don't make `M-g' bug out on group with no
        marks.
-       (nnoo): Required, so that other packages can require nnimap.
+       (nnoo): Require, so that other packages can require nnimap.
        (nnimap-wait-for-response): Be a bit more lax in finding the end of the
        command we're looking for.  This helps when the server sends more
        responses after we've gotten everything we expected.

diff --git a/lisp/gnus/nnml.el b/lisp/gnus/nnml.el
index 8fca41eb4d294d07f3e2e5950a8b184eaba0f119..11cdfd768c3b5893247f967cd96acad98bbdcc92 100644 (file)
--- a/lisp/gnus/nnml.el
+++ b/lisp/gnus/nnml.el
@@ -878,6 +878,7 @@ article number.  This function is called narrowed to an article."
   ;; Save the active file.
   (nnmail-save-active nnml-group-alist nnml-active-file))
 
+(defvar nnml-files)
 (defun nnml-generate-nov-databases-directory (dir &optional seen no-active)
   "Regenerate the NOV database in DIR.
 
@@ -897,9 +898,9 @@ Unless no-active is non-nil, update the active file too."
                   (file-directory-p dir))
          (nnml-generate-nov-databases-directory dir seen)))
       ;; Do this directory.
-      (let ((files (sort (nnheader-article-to-file-alist dir)
+      (let ((nnml-files (sort (nnheader-article-to-file-alist dir)
                         'car-less-than-car)))
-       (if (not files)
+       (if (not nnml-files)
            (let* ((group (nnheader-file-to-group
                           (directory-file-name dir) nnml-directory))
                   (info (cadr (assoc group nnml-group-alist))))
@@ -907,11 +908,10 @@ Unless no-active is non-nil, update the active file too."
                (setcar info (1+ (cdr info)))))
          (funcall nnml-generate-active-function dir)
          ;; Generate the nov file.
-         (nnml-generate-nov-file dir files)
+         (nnml-generate-nov-file dir nnml-files)
          (unless no-active
            (nnmail-save-active nnml-group-alist nnml-active-file)))))))
 
-(defvar files)
 (defun nnml-generate-active-info (dir)
   ;; Update the active info for this group.
   (let ((group (directory-file-name dir))
@@ -922,9 +922,9 @@ Unless no-active is non-nil, update the active file too."
          last (or (caadr entry) 0)
          nnml-group-alist (delq entry nnml-group-alist))
     (push (list group
-               (cons (or (caar files) (1+ last))
+               (cons (or (caar nnml-files) (1+ last))
                      (max last
-                          (or (caar (last files))
+                          (or (caar (last nnml-files))
                               0))))
          nnml-group-alist)))
 
@@ -941,8 +941,8 @@ Unless no-active is non-nil, update the active file too."
       ;; Delete the old NOV file.
       (when (file-exists-p nov)
        (funcall nnmail-delete-file-function nov))
-      (while files
-       (unless (file-directory-p (setq file (concat dir (cdar files))))
+      (dolist (file files)
+       (unless (file-directory-p (setq file (concat dir (cdr file))))
          (erase-buffer)
          (nnheader-insert-file-contents file)
          (narrow-to-region
@@ -953,12 +953,11 @@ Unless no-active is non-nil, update the active file too."
             (max (point-min) (1- (point)))))
          (unless (zerop (buffer-size))
            (goto-char (point-min))
-           (setq headers (nnml-parse-head chars (caar files)))
+           (setq headers (nnml-parse-head chars (car file)))
            (with-current-buffer nov-buffer
              (goto-char (point-max))
              (nnheader-insert-nov headers)))
-         (widen))
-       (setq files (cdr files)))
+         (widen)))
       (with-current-buffer nov-buffer
        (nnmail-write-region (point-min) (point-max) nov nil 'nomesg)
        (kill-buffer (current-buffer))))))
@@ -995,11 +994,9 @@ Use the nov database for that directory if available."
     ;; 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
+    (with-current-buffer (nnml-get-nov-buffer nnml-current-group)
       (let ((list nil)
-           art
-           (buffer (nnml-get-nov-buffer nnml-current-group)))
-       (set-buffer buffer)
+           art)
        (goto-char (point-min))
        (while (not (eobp))
          (setq art (read (current-buffer)))
@@ -1018,11 +1015,9 @@ Use the nov database for the current group if available."
                                  nnml-current-directory))))
       (nnheader-article-to-file-alist nnml-current-directory)
     ;; build list from .overview if available
-    (save-excursion
+    (with-current-buffer (nnml-get-nov-buffer nnml-current-group)
       (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)))