]> git.eshelyaron.com Git - emacs.git/commitdiff
Clean up gnus-uu saving code slightly
authorLars Magne Ingebrigtsen <larsi@gnus.org>
Wed, 15 Apr 2015 13:28:20 +0000 (15:28 +0200)
committerLars Magne Ingebrigtsen <larsi@gnus.org>
Wed, 15 Apr 2015 13:28:20 +0000 (15:28 +0200)
* gnus-uu.el (gnus-uu-save-article): Make the
save-restriction/widen calls make more sense.

lisp/gnus/gnus-uu.el

index f5d44959391c6db228c64babc107d9d807d12f50..94f01c6158d8830f5d4b457acf43b0daff53c0f7 100644 (file)
@@ -873,14 +873,8 @@ When called interactively, prompt for REGEXP."
          (setq state (list 'middle))))
       (with-current-buffer "*gnus-uu-body*"
        (goto-char (setq beg (point-max)))
-       (save-excursion
+       (with-current-buffer buffer
          (save-restriction
-            ;; FIXME: We save excursion and restriction in "*gnus-uu-body*",
-            ;; only to immediately move to another buffer?  And we narrow in
-            ;; that buffer without save-restriction?  And we finish the
-            ;; save-restriction with a call to `widen'?  How can that
-            ;; make sense?
-           (set-buffer buffer)
            (let ((inhibit-read-only t))
              (set-text-properties (point-min) (point-max) nil)
              ;; These two are necessary for XEmacs 19.12 fascism.
@@ -915,8 +909,7 @@ When called interactively, prompt for REGEXP."
                                 (match-beginning 0)
                                 (or (and (re-search-forward "^[^ \t]" nil t)
                                          (1- (point)))
-                                    (progn (forward-line 1) (point)))))))))
-           (widen)))
+                                    (progn (forward-line 1) (point)))))))))))
        (if (and message-forward-as-mime gnus-uu-digest-buffer)
          (if message-forward-show-mml
              (progn