Fix last nnmail-get-new-mail-1 change that caused double encoding in XEmacs.

2010-08-01  Katsumi Yamaoka  <yamaoka@jpl.org>
 * nnmail.el (nnmail-get-new-mail-1): Revert.
 * nnml.el (nnml-active-number): Make sure names of newly created groups
 in nnml-group-alist are encoded.

diff --git a/lisp/gnus/ChangeLog b/lisp/gnus/ChangeLog
index 44c28c52fff0b1f5568691ecdad2a566bede8388..4c364c3270df4aa859d99505c7b67ed65875fe9a 100644 (file)
--- a/lisp/gnus/ChangeLog
+++ b/lisp/gnus/ChangeLog
@@ -1,3 +1,10 @@
+2010-08-01  Katsumi Yamaoka  <yamaoka@jpl.org>
+
+       * nnmail.el (nnmail-get-new-mail-1): Revert.
+
+       * nnml.el (nnml-active-number): Make sure names of newly created groups
+       in nnml-group-alist are encoded.
+
 2010-07-30  Katsumi Yamaoka  <yamaoka@jpl.org>
 
        * nnmail.el (nnmail-get-new-mail-1): Encode group names possibly

diff --git a/lisp/gnus/nnmail.el b/lisp/gnus/nnmail.el
index 537d98cc6aeb052930611cc263ecc2d98038b9b2..8bf0cbf5de10fc0aebd2dca0d9efb593d6671215 100644 (file)
--- a/lisp/gnus/nnmail.el
+++ b/lisp/gnus/nnmail.el
@@ -1844,20 +1844,9 @@ See the Info node `(gnus)Fancy Mail Splitting' for more details."
       (if (zerop total)
          (nnheader-message 4 "%s: Reading incoming mail (no new mail)...done"
                            method (car source))
-       (let ((group-alist (nnmail-get-value "%s-group-alist" method))
-             (active-file (nnmail-get-value "%s-active-file" method))
-             encoded)
-         ;; Encode group names possibly containing non-ASCII characters.
-         (cond ((eq method 'nnml)
-                (nnmail-save-active
-                 (dolist (elem group-alist (nreverse encoded))
-                   (push (cons (nnml-encoded-group-name (car elem)
-                                                        gnus-command-method)
-                               (cdr elem))
-                         encoded))
-                 active-file))
-               (t
-                (nnmail-save-active group-alist active-file))))
+       (nnmail-save-active
+        (nnmail-get-value "%s-group-alist" method)
+        (nnmail-get-value "%s-active-file" method))
        (when exit-func
          (funcall exit-func))
        (run-hooks 'nnmail-read-incoming-hook)

diff --git a/lisp/gnus/nnml.el b/lisp/gnus/nnml.el
index fb5fb44113f213e1cc7051f72f4effc72b3bec40..238e0221b97a39ea37e46338fb044b9e30fdae68 100644 (file)
--- a/lisp/gnus/nnml.el
+++ b/lisp/gnus/nnml.el
@@ -751,10 +751,9 @@ article number.  This function is called narrowed to an article."
 
 (defun nnml-active-number (group &optional server)
   "Compute the next article number in GROUP on SERVER."
-  (let ((active (cadr (assoc (if nnmail-group-names-not-encoded-p
-                                (nnml-encoded-group-name group server)
-                              group)
-                            nnml-group-alist))))
+  (let* ((encoded (if nnmail-group-names-not-encoded-p
+                     (nnml-encoded-group-name group server)))
+        (active (cadr (assoc (or encoded group) nnml-group-alist))))
     ;; The group wasn't known to nnml, so we just create an active
     ;; entry for it.
     (unless active
@@ -772,7 +771,7 @@ article number.  This function is called narrowed to an article."
                (cons (caar nnml-article-file-alist)
                      (caar (last nnml-article-file-alist)))
              (cons 1 0)))
-      (push (list group active) nnml-group-alist))
+      (push (list (or encoded group) active) nnml-group-alist))
     (setcdr active (1+ (cdr active)))
     (while (file-exists-p
            (nnml-group-pathname group (int-to-string (cdr active)) server))