(pmail-edit-mode): Bind buffer-swwapped-with to nil.

(pmail-edit-current-message): Save all of buffer as old text.
Clear undo list.
(pmail-cease-edit): Widen first.
Operate in the view buffer; copy edited text back to mbox buffer
and re-encode what was decoded to display the message.
(pmail-abort-edit): Widen first.

diff --git a/lisp/mail/pmailedit.el b/lisp/mail/pmailedit.el
index d26a7b681c9dc4fc641926474e24c9ec62126df2..3c48590a893a9e88f611085696593be701211da9 100644 (file)
--- a/lisp/mail/pmailedit.el
+++ b/lisp/mail/pmailedit.el
@@ -61,20 +61,20 @@ to return to regular PMAIL:
   *  \\[pmail-cease-edit] makes them permanent.
 This functions runs the normal hook `pmail-edit-mode-hook'.
 \\{pmail-edit-map}"
-  ;; Changing the major mode unswaps the Pmail buffer
-  ;; via change-major-mode-hook.
-  (delay-mode-hooks (text-mode))
-  (use-local-map pmail-edit-map)
-  (setq major-mode 'pmail-edit-mode)
-  (setq mode-name "PMAIL Edit")
-  (if (boundp 'mode-line-modified)
-      (setq mode-line-modified (default-value 'mode-line-modified))
-    (setq mode-line-format (default-value 'mode-line-format)))
   (if (pmail-summary-exists)
       (save-excursion
        (set-buffer pmail-summary-buffer)
        (pmail-summary-disable)))
-  (run-mode-hooks 'pmail-edit-mode-hook))
+  (let (buffer-swapped-with)
+    ;; Prevent change-major-mode-hook from unswapping the buffers.
+    (delay-mode-hooks (text-mode))
+    (use-local-map pmail-edit-map)
+    (setq major-mode 'pmail-edit-mode)
+    (setq mode-name "PMAIL Edit")
+    (if (boundp 'mode-line-modified)
+       (setq mode-line-modified (default-value 'mode-line-modified))
+      (setq mode-line-format (default-value 'mode-line-format)))
+    (run-mode-hooks 'pmail-edit-mode-hook)))
 
 (defvar pmail-old-pruned nil)
 (put 'pmail-old-pruned 'permanent-local t)
@@ -87,12 +87,13 @@ This functions runs the normal hook `pmail-edit-mode-hook'.
       (error "No messages in this buffer"))
   (make-local-variable 'pmail-old-pruned)
   (setq pmail-old-pruned (eq pmail-header-style 'normal))
-  ;; Changing the major mode unswaps the Pmail buffer
-  ;; via change-major-mode-hook.
   (pmail-edit-mode)
   (make-local-variable 'pmail-old-text)
-  (setq pmail-old-text (buffer-substring (point-min) (point-max)))
+  (save-restriction
+    (widen)
+    (setq pmail-old-text (buffer-substring (point-min) (point-max))))
   (setq buffer-read-only nil)
+  (setq buffer-undo-list nil)
   (force-mode-line-update)
   (if (and (eq (key-binding "\C-c\C-c") 'pmail-cease-edit)
           (eq (key-binding "\C-c\C-]") 'pmail-abort-edit))
@@ -107,6 +108,7 @@ This functions runs the normal hook `pmail-edit-mode-hook'.
       (save-excursion
        (set-buffer pmail-summary-buffer)
        (pmail-summary-enable)))
+  (widen)
   ;; Disguise any "From " lines so they don't start a new message.
   (save-excursion
     (goto-char (point-min))
@@ -120,45 +122,86 @@ This functions runs the normal hook `pmail-edit-mode-hook'.
     (if (/= (preceding-char) ?\n)
        (insert "\n"))
     (unless (looking-back "\n\n")
-      (insert "\n"))
-    ;; Adjust the marker that points to the end of this message.
-    (set-marker (aref pmail-message-vector (1+ pmail-current-message))
-               (point)))
-  (let ((old pmail-old-text))
+      (insert "\n")))
+  (let ((old pmail-old-text)
+       character-coding is-text-message coding-system
+       headers-end)
     ;; Go back to Pmail mode, but carefully.
     (force-mode-line-update)
-    (kill-all-local-variables)
-    (pmail-mode-1)
-    (if (boundp 'tool-bar-map)
-       (set (make-local-variable 'tool-bar-map) pmail-tool-bar-map))
-    (pmail-variables)
+    (let (buffer-swapped-with)
+      (kill-all-local-variables)
+      (pmail-mode-1)
+      (if (boundp 'tool-bar-map)
+         (set (make-local-variable 'tool-bar-map) pmail-tool-bar-map))
+      (setq buffer-undo-list t)
+      (pmail-variables))
     ;; If text has really changed, mark message as edited.
     (unless (and (= (length old) (- (point-max) (point-min)))
                 (string= old (buffer-substring (point-min) (point-max))))
       (setq old nil)
-      (pmail-set-attribute pmail-edited-attr-index t)
-      ;;;??? BROKEN perhaps.
-      ;; I think that the Summary-Line header may not be kept there any more.
-      (if (boundp 'pmail-summary-vector)
-         (progn
-           (aset pmail-summary-vector (1- pmail-current-message) nil)
-           (save-excursion
-             (pmail-widen-to-current-msgbeg
-               (function (lambda ()
-                           (forward-line 2)
-                           (if (looking-at "Summary-line: ")
-                               (let ((buffer-read-only nil))
-                                 (delete-region (point)
-                                                (progn (forward-line 1)
-                                                       (point))))))))))))
-    (save-excursion
-      (pmail-show-message)
-      (pmail-toggle-header (if pmail-old-pruned 1 0))))
+      (goto-char (point-min))
+      (search-forward "\n\n")
+      (setq headers-end (point))
+
+      (pmail-swap-buffers-maybe)
+
+      (setq character-coding (mail-fetch-field "content-transfer-encoding")
+           is-text-message (pmail-is-text-p)
+           coding-system (pmail-get-coding-system))
+      (if character-coding
+         (setq character-coding (downcase character-coding)))
+
+      (narrow-to-region (pmail-msgbeg pmail-current-message)
+                       (pmail-msgend pmail-current-message))
+      (goto-char (point-min))
+      (search-forward "\n\n")
+      (let ((inhibit-read-only t)
+           (headers-end-1 (point)))
+       (insert-buffer-substring pmail-view-buffer headers-end)
+       (delete-region (point) (point-max))
+
+       ;; Re-encode the message body in whatever
+       ;; way it was decoded.
+       (cond
+        ((string= character-coding "quoted-printable")
+         (mail-quote-printable-region headers-end-1 (point-max)))
+        ((and (string= character-coding "base64") is-text-message)
+         (base64-encode-region headers-end-1 (point-max)))
+        ((eq character-coding 'uuencode)
+         (error "Not supported yet."))
+        (t
+         (if (or (not coding-system) (not (coding-system-p coding-system)))
+             (setq coding-system 'undecided))
+         (encode-coding-region headers-end-1 (point-max) coding-system)))
+       ))
+
+    (pmail-set-attribute pmail-edited-attr-index t)
+       
+    ;;??? BROKEN perhaps.
+    ;; I think that the Summary-Line header may not be kept there any more.
+;;;       (if (boundp 'pmail-summary-vector)
+;;;      (progn
+;;;        (aset pmail-summary-vector (1- pmail-current-message) nil)
+;;;        (save-excursion
+;;;          (pmail-widen-to-current-msgbeg
+;;;            (function (lambda ()
+;;;                        (forward-line 2)
+;;;                        (if (looking-at "Summary-line: ")
+;;;                            (let ((buffer-read-only nil))
+;;;                              (delete-region (point)
+;;;                                             (progn (forward-line 1)
+;;;                                                    (point)))))))))))
+    )
+
+  (save-excursion
+    (pmail-show-message)
+    (pmail-toggle-header (if pmail-old-pruned 1 0)))
   (run-hooks 'pmail-mode-hook))
 
 (defun pmail-abort-edit ()
   "Abort edit of current message; restore original contents."
   (interactive)
+  (widen)
   (delete-region (point-min) (point-max))
   (insert pmail-old-text)
   (pmail-cease-edit)