(facemenu-active-faces):
[bpt/emacs.git] / lisp / arc-mode.el
index 1207a57..987f29f 100644 (file)
@@ -1,6 +1,6 @@
 ;;; arc-mode.el --- simple editing of archives
 
-;; Copyright (C) 1995, 1997 Free Software Foundation, Inc.
+;; Copyright (C) 1995, 1997, 1998 Free Software Foundation, Inc.
 
 ;; Author: Morten Welinder <terra@diku.dk>
 ;; Keywords: archives msdog editing major-mode
   :group 'archive)
 
 (defcustom archive-tmpdir
-  (expand-file-name
-   (make-temp-name (if (eq system-type 'ms-dos) "ar" "archive.tmp"))
-   (or (getenv "TMPDIR") (getenv "TMP") "/tmp"))
+  (make-temp-name
+   (expand-file-name (if (eq system-type 'ms-dos) "ar" "archive.tmp")
+                    temporary-file-directory))
   "*Directory for temporary files made by arc-mode.el"
   :type 'directory
   :group 'archive)
@@ -243,7 +243,7 @@ Archive and member names will be added."
   :group 'archive-zip)
 
 (defcustom archive-zip-update
-  (if archive-zip-use-pkzip '("pkzip" "-u") '("zip" "-q"))
+  (if archive-zip-use-pkzip '("pkzip" "-u" "-P") '("zip" "-q"))
   "*Program and its options to run in order to update a zip file member.
 Options should ensure that specified directory will be put into the zip
 file.  Archive and member name will be added."
@@ -265,9 +265,9 @@ Archive and member name will be added."
   :group 'archive-zip)
 
 (defcustom archive-zip-case-fiddle t
-  "*If non-nil then zip file members are case fiddled.
-Case fiddling will only happen for members created by a system that
-uses caseless file names."
+  "*If non-nil then zip file members may be down-cased.
+This case fiddling will only happen for members created by a system
+that uses caseless file names."
   :type 'boolean
   :group 'archive-zip)
 ;; ------------------------------
@@ -311,11 +311,17 @@ Archive and member name will be added."
 (defvar archive-file-list-end nil "*Position just after last contents line.")
 (defvar archive-proper-file-start nil "*Position of real archive's start.")
 (defvar archive-read-only nil "*Non-nil if the archive is read-only on disk.")
-(defvar archive-remote nil "*Non-nil if the archive is outside file system.")
 (defvar archive-local-name nil "*Name of local copy of remote archive.")
 (defvar archive-mode-map nil "*Local keymap for archive mode listings.")
 (defvar archive-file-name-indent nil "*Column where file names start.")
 
+(defvar archive-remote nil "*Non-nil if the archive is outside file system.")
+(make-variable-buffer-local 'archive-remote)
+(put 'archive-remote 'permanent-local t)
+
+(defvar archive-member-coding-system nil "Coding-system of archive member.")
+(make-variable-buffer-local 'archive-member-coding-system)
+
 (defvar archive-alternate-display nil
   "*Non-nil when alternate information is shown.")
 (make-variable-buffer-local 'archive-alternate-display)
@@ -358,24 +364,25 @@ in which case a second argument, length, should be supplied."
     result))
 
 (defun archive-int-to-mode (mode)
-  "Turn an integer like 0700 (i.e., 448) into a mode string like -rwx------"
-  (let ((str (make-string 10 ?-)))
-    (or (zerop (logand 16384 mode)) (aset str 0 ?d))
-    (or (zerop (logand  8192 mode)) (aset str 0 ?c)) ; completeness
-    (or (zerop (logand   256 mode)) (aset str 1 ?r))
-    (or (zerop (logand   128 mode)) (aset str 2 ?w))
-    (or (zerop (logand    64 mode)) (aset str 3 ?x))
-    (or (zerop (logand    32 mode)) (aset str 4 ?r))
-    (or (zerop (logand    16 mode)) (aset str 5 ?w))
-    (or (zerop (logand     8 mode)) (aset str 6 ?x))
-    (or (zerop (logand     4 mode)) (aset str 7 ?r))
-    (or (zerop (logand     2 mode)) (aset str 8 ?w))
-    (or (zerop (logand     1 mode)) (aset str 9 ?x))
-    (or (zerop (logand  1024 mode)) (aset str 3 (if (zerop (logand 64 mode))
-                                                   ?S ?s)))
-    (or (zerop (logand  2048 mode)) (aset str 6 (if (zerop (logand  8 mode))
-                                                   ?S ?s)))
-    str))
+  "Turn an integer like 0700 (i.e., 448) into a mode string like -rwx------."
+  ;; FIXME: merge with tar-grind-file-mode.
+  (string
+    (if (zerop (logand  8192 mode))
+       (if (zerop (logand 16384 mode)) ?- ?d)
+      ?c) ; completeness
+    (if (zerop (logand   256 mode)) ?- ?r)
+    (if (zerop (logand   128 mode)) ?- ?w)
+    (if (zerop (logand    64 mode))
+       (if (zerop (logand  1024 mode)) ?- ?S)
+      (if (zerop (logand  1024 mode)) ?x ?s))
+    (if (zerop (logand    32 mode)) ?- ?r)
+    (if (zerop (logand    16 mode)) ?- ?w)
+    (if (zerop (logand     8 mode))
+       (if (zerop (logand  2048 mode)) ?- ?S)
+      (if (zerop (logand  2048 mode)) ?x ?s))
+    (if (zerop (logand     4 mode)) ?- ?r)
+    (if (zerop (logand     2 mode)) ?- ?w)
+    (if (zerop (logand     1 mode)) ?- ?x)))
 
 (defun archive-calc-mode (oldmode newmode &optional error)
   "From the integer OLDMODE and the string NEWMODE calculate a new file mode.
@@ -440,7 +447,7 @@ the mode is invalid.  If ERROR is nil then nil will be returned."
 (defun archive-dostime (time)
   "Stringify dos packed TIME record."
   (let ((hour (logand (ash time -11) 31))
-        (minute (logand (ash time -5) 53))
+        (minute (logand (ash time -5) 63))
         (second (* 2 (logand time 31)))) ; 2 seconds resolution
     (format "%02d:%02d:%02d" hour minute second)))
 
@@ -499,8 +506,7 @@ archive.
       (funcall default-major-mode)
     (if (and (not force) archive-files) nil
       (let* ((type (archive-find-type))
-            (typename (copy-sequence (symbol-name type))))
-       (aset typename 0 (upcase (aref typename 0)))
+            (typename (capitalize (symbol-name type))))
        (kill-all-local-variables)
        (make-local-variable 'archive-subtype)
        (setq archive-subtype type)
@@ -509,23 +515,40 @@ archive.
        (make-local-variable 'revert-buffer-function)
        (setq revert-buffer-function 'archive-mode-revert)
        (auto-save-mode 0)
-       (make-local-variable 'write-contents-hooks)
-       (add-hook 'write-contents-hooks 'archive-write-file)
 
-       ;; Real file contents is binary
+       ;; Remote archives are not written by a hook.
+       (if archive-remote nil
+         (make-local-variable 'write-contents-hooks)
+         (add-hook 'write-contents-hooks 'archive-write-file))
+
        (make-local-variable 'require-final-newline)
        (setq require-final-newline nil)
-       (make-local-variable 'enable-local-variables)
-       (setq enable-local-variables nil)
+       (make-local-variable 'local-enable-local-variables)
+       (setq local-enable-local-variables nil)
+
+       ;; Prevent loss of data when saving the file.
+       (make-local-variable 'file-precious-flag)
+       (setq file-precious-flag t)
 
        (make-local-variable 'archive-read-only)
-       (setq archive-read-only (not (file-writable-p (buffer-file-name))))
+       ;; Archives which are inside other archives and whose
+       ;; names are invalid for this OS, can't be written.
+       (setq archive-read-only
+             (or (not (file-writable-p (buffer-file-name)))
+                 (and archive-subfile-mode
+                      (string-match file-name-invalid-regexp
+                                    (aref archive-subfile-mode 0)))))
 
        ;; Should we use a local copy when accessing from outside Emacs?
        (make-local-variable 'archive-local-name)
-       (make-local-variable 'archive-remote)
-       (setq archive-remote (string-match archive-remote-regexp
-                                          (buffer-file-name)))
+
+       ;; An archive can contain another archive whose name is invalid
+       ;; on local filesystem.  Treat such archives as remote.
+       (or archive-remote
+           (setq archive-remote
+                 (or (string-match archive-remote-regexp (buffer-file-name))
+                     (string-match file-name-invalid-regexp
+                                   (buffer-file-name)))))
 
        (setq major-mode 'archive-mode)
        (setq mode-name (concat typename "-Archive"))
@@ -537,7 +560,7 @@ archive.
       (make-local-variable 'archive-file-list-start)
       (make-local-variable 'archive-file-list-end)
       (make-local-variable 'archive-file-name-indent)
-      (archive-summarize)
+      (archive-summarize nil)
       (setq buffer-read-only t))))
 
 ;; Archive mode is suitable only for specially formatted data.
@@ -564,6 +587,7 @@ archive.
   (define-key archive-mode-map [down] 'archive-next-line)
   (define-key archive-mode-map "o" 'archive-extract-other-window)
   (define-key archive-mode-map "p" 'archive-previous-line)
+  (define-key archive-mode-map "q" 'quit-window)
   (define-key archive-mode-map "\C-p" 'archive-previous-line)
   (define-key archive-mode-map [up] 'archive-previous-line)
   (define-key archive-mode-map "r" 'archive-rename-entry)
@@ -590,57 +614,64 @@ archive.
 
   (if archive-lemacs
       ()                               ; out of luck
-    ;; Get rid of the Edit menu bar item to save space.
-    (define-key archive-mode-map [menu-bar edit] 'undefined)
 
     (define-key archive-mode-map [menu-bar immediate]
       (cons "Immediate" (make-sparse-keymap "Immediate")))
     (define-key archive-mode-map [menu-bar immediate alternate]
-      '("Alternate Display" . archive-alternate-display))
-    (put 'archive-alternate-display 'menu-enable
-        '(boundp (archive-name "alternate-display")))
+      '(menu-item "Alternate Display" archive-alternate-display
+                 :enable (boundp (archive-name "alternate-display"))
+                 :help "Toggle alternate file info display"))
     (define-key archive-mode-map [menu-bar immediate view]
-      '("View This File" . archive-view))
+      '(menu-item "View This File" archive-view
+                 :help "Display file at cursor in View Mode"))
     (define-key archive-mode-map [menu-bar immediate display]
-      '("Display in Other Window" . archive-display-other-window))
+      '(menu-item "Display in Other Window" archive-display-other-window
+                 :help "Display file at cursor in another window"))
     (define-key archive-mode-map [menu-bar immediate find-file-other-window]
-      '("Find in Other Window" . archive-extract-other-window))
+      '(menu-item "Find in Other Window" archive-extract-other-window
+                 :help "Edit file at cursor in another window"))
     (define-key archive-mode-map [menu-bar immediate find-file]
-      '("Find This File" . archive-extract))
+      '(menu-item "Find This File" archive-extract
+                 :help "Extract file at cursor and edit it"))
 
     (define-key archive-mode-map [menu-bar mark]
       (cons "Mark" (make-sparse-keymap "Mark")))
     (define-key archive-mode-map [menu-bar mark unmark-all]
-      '("Unmark All" . archive-unmark-all-files))
+      '(menu-item "Unmark All" archive-unmark-all-files
+                 :help "Unmark all marked files"))
     (define-key archive-mode-map [menu-bar mark deletion]
-      '("Flag" . archive-flag-deleted))
+      '(menu-item "Flag" archive-flag-deleted
+                 :help "Flag file at cursor for deletion"))
     (define-key archive-mode-map [menu-bar mark unmark]
-      '("Unflag" . archive-unflag))
+      '(menu-item "Unflag" archive-unflag
+                 :help "Unmark file at cursor"))
     (define-key archive-mode-map [menu-bar mark mark]
-      '("Mark" . archive-mark))
+      '(menu-item "Mark" archive-mark
+                 :help "Mark file at cursor"))
 
     (define-key archive-mode-map [menu-bar operate]
       (cons "Operate" (make-sparse-keymap "Operate")))
     (define-key archive-mode-map [menu-bar operate chown]
-      '("Change Owner..." . archive-chown-entry))
-    (put 'archive-chown-entry 'menu-enable
-        '(fboundp (archive-name "chown-entry")))
+      '(menu-item "Change Owner..." archive-chown-entry
+                 :enable (fboundp (archive-name "chown-entry"))
+                 :help "Change owner of marked files"))
     (define-key archive-mode-map [menu-bar operate chgrp]
-      '("Change Group..." . archive-chgrp-entry))
-    (put 'archive-chgrp-entry 'menu-enable
-        '(fboundp (archive-name "chgrp-entry")))
+      '(menu-item "Change Group..." archive-chgrp-entry
+                 :enable (fboundp (archive-name "chgrp-entry"))
+                 :help "Change group ownership of marked files"))
     (define-key archive-mode-map [menu-bar operate chmod]
-      '("Change Mode..." . archive-chmod-entry))
-    (put 'archive-chmod-entry 'menu-enable
-        '(fboundp (archive-name "chmod-entry")))
+      '(menu-item "Change Mode..." archive-chmod-entry
+                 :enable (fboundp (archive-name "chmod-entry"))
+                 :help "Change mode (permissions) of marked files"))
     (define-key archive-mode-map [menu-bar operate rename]
-      '("Rename to..." . archive-rename-entry))
-    (put 'archive-rename-entry 'menu-enable
-        '(fboundp (archive-name "rename-entry")))
+      '(menu-item "Rename to..." archive-rename-entry
+                 :enable (fboundp (archive-name "rename-entry"))
+                 :help "Rename marked files"))
     ;;(define-key archive-mode-map [menu-bar operate copy]
-    ;;  '("Copy to..." . archive-copy))
+    ;;  '(menu-item "Copy to..." archive-copy))
     (define-key archive-mode-map [menu-bar operate expunge]
-      '("Expunge Marked Files" . archive-expunge))
+      '(menu-item "Expunge Marked Files" archive-expunge
+                 :help "Delete all flagged files from archive"))
   ))
 
 (let* ((item1 '(archive-subfile-mode " Archive"))
@@ -661,19 +692,24 @@ archive.
                (string-match "\\.[aA][rR][cC]$"
                              (or buffer-file-name (buffer-name))))
           'arc)
-         (t (error "Buffer format not recognized.")))))
+         (t (error "Buffer format not recognized")))))
 ;; -------------------------------------------------------------------------
-(defun archive-summarize ()
+(defun archive-summarize (&optional shut-up)
   "Parse the contents of the archive file in the current buffer.
 Place a dired-like listing on the front;
 then narrow to it, so that only that listing
-is visible (and the real data of the buffer is hidden)."
+is visible (and the real data of the buffer is hidden).
+Optional argument SHUT-UP, if non-nil, means don't print messages
+when parsing the archive."
   (widen)
+  (set-buffer-multibyte nil)
   (let (buffer-read-only)
-    (message "Parsing archive file...")
+    (or shut-up
+       (message "Parsing archive file..."))
     (buffer-disable-undo (current-buffer))
     (setq archive-files (funcall (archive-name "summarize")))
-    (message "Parsing archive file...done.")
+    (or shut-up
+       (message "Parsing archive file...done."))
     (setq archive-proper-file-start (point-marker))
     (narrow-to-region (point-min) (point))
     (set-buffer-modified-p nil)
@@ -688,7 +724,7 @@ is visible (and the real data of the buffer is hidden)."
        buffer-read-only)
     (widen)
     (delete-region (point-min) archive-proper-file-start)
-    (archive-summarize)
+    (archive-summarize t)
     (set-buffer-modified-p modified)
     (goto-char archive-file-list-start)
     (archive-next-line no)))
@@ -710,9 +746,11 @@ is visible (and the real data of the buffer is hidden)."
        (let ((text (concat (aref fil 0) "\n")))
          (if archive-lemacs
              ()                        ; out of luck
-           (put-text-property (aref fil 1) (aref fil 2)
-                              'mouse-face 'highlight
-                              text))
+           (add-text-properties
+            (aref fil 1) (aref fil 2)
+            '(mouse-face highlight
+              help-echo "mouse-2: extract this file into a buffer")
+            text))
          text)))
      files)))
   (setq archive-file-list-end (point-marker)))
@@ -727,32 +765,66 @@ This function changes the set of information shown for each files."
 ;; -------------------------------------------------------------------------
 ;; Section: Local archive copy handling
 
+(defun archive-unique-fname (fname dir)
+  "Make sure a file FNAME can be created uniquely in directory DIR.
+
+If FNAME can be uniquely created in DIR, it is returned unaltered.
+If FNAME is something our underlying filesystem can't grok, or if another
+file by that name already exists in DIR, a unique new name is generated
+using `make-temp-file', and the generated name is returned."
+  (let ((fullname (expand-file-name fname dir))
+       (alien (string-match file-name-invalid-regexp fname)))
+    (if (or alien (file-exists-p fullname))
+       (make-temp-file
+        (expand-file-name
+         (if (and (fboundp 'msdos-long-file-names)
+                  (not (msdos-long-file-names)))
+             "am"
+           "arc-mode.")
+         dir))
+      fullname)))
+
 (defun archive-maybe-copy (archive)
-  (if archive-remote
-      (let ((start (point-max)))
-       (setq archive-local-name (expand-file-name
-                                 (file-name-nondirectory archive)
-                                 archive-tmpdir))
-       (make-directory archive-tmpdir t)
-       (save-restriction
-         (widen)
-         (write-region start (point-max) archive-local-name nil 'nomessage))
-       archive-local-name)
-    (if (buffer-modified-p) (save-buffer))
-    archive))
+  (let ((coding-system-for-write 'no-conversion))
+    (if archive-remote
+       (let ((start (point-max))
+             ;; Sometimes ARCHIVE is invalid while its actual name, as
+             ;; recorded in its parent archive, is not.  For example, an
+             ;; archive bar.zip inside another archive foo.zip gets a name
+             ;; "foo.zip:bar.zip", which is invalid on DOS/Windows.
+             ;; So use the actual name if available.
+             (archive-name
+              (or (and archive-subfile-mode (aref archive-subfile-mode 0))
+                  archive)))
+         (make-directory archive-tmpdir t)
+         (setq archive-local-name
+               (archive-unique-fname archive-name archive-tmpdir))
+         (save-restriction
+           (widen)
+           (write-region start (point-max) archive-local-name nil 'nomessage))
+         archive-local-name)
+      (if (buffer-modified-p) (save-buffer))
+      archive)))
 
 (defun archive-maybe-update (unchanged)
   (if archive-remote
       (let ((name archive-local-name)
            (modified (buffer-modified-p))
+           (coding-system-for-read 'no-conversion)
+           (lno (archive-get-lineno))
            buffer-read-only)
        (if unchanged nil
+         (setq archive-files nil)
          (erase-buffer)
          (insert-file-contents name)
-         (archive-mode t))
+         (archive-mode t)
+         (goto-char archive-file-list-start)
+         (archive-next-line lno))
        (archive-delete-local name)
        (if (not unchanged)
-           (message "Archive file must be saved for changes to take effect"))
+           (message
+            "Buffer `%s' must be saved for changes to take effect"
+            (buffer-name (current-buffer))))
        (set-buffer-modified-p (or modified (not unchanged))))))
 
 (defun archive-delete-local (name)
@@ -771,6 +843,42 @@ This function changes the set of information shown for each files."
 ;; -------------------------------------------------------------------------
 ;; Section: Member extraction
 
+(defun archive-file-name-handler (op &rest args)
+  (or (eq op 'file-exists-p)
+      (let ((file-name-handler-alist nil))
+       (apply op args))))
+
+(defun archive-set-buffer-as-visiting-file (filename)
+  "Set the current buffer as if it were visiting FILENAME."
+  (save-excursion
+    (goto-char (point-min))
+    (let ((coding
+          (or coding-system-for-read
+              (and set-auto-coding-function
+                   (save-excursion
+                     (funcall set-auto-coding-function
+                              filename (- (point-max) (point-min)))))
+              ;; dos-w32.el defines find-operation-coding-system for
+              ;; DOS/Windows systems which preserves the coding-system
+              ;; of existing files.  We want it to act here as if the
+              ;; extracted file existed.
+              (let ((file-name-handler-alist
+                     '(("" . archive-file-name-handler))))
+                (car (find-operation-coding-system 'insert-file-contents
+                                                   filename t))))))
+      (if (and (not coding-system-for-read)
+              (not enable-multibyte-characters))
+         (setq coding
+               (coding-system-change-text-conversion coding 'raw-text)))
+      (if (and coding
+              (not (eq coding 'no-conversion)))
+         (decode-coding-region (point-min) (point-max) coding)
+       (setq last-coding-system-used coding))
+      (set-buffer-modified-p nil)
+      (kill-local-variable 'buffer-file-coding-system)
+      (after-insert-file-set-buffer-file-coding-system (- (point-max)
+                                                         (point-min))))))
+
 (defun archive-mouse-extract (event)
   "Extract a file whose name you click on."
   (interactive "e")
@@ -793,7 +901,11 @@ This function changes the set of information shown for each files."
          (arcname (file-name-nondirectory archive))
          (bufname (concat (file-name-nondirectory iname) " (" arcname ")"))
          (extractor (archive-name "extract"))
-         (read-only-p (or archive-read-only view-p))
+        ;; Members with file names which aren't valid for the
+        ;; underlying filesystem, are treated as read-only.
+         (read-only-p (or archive-read-only
+                         view-p
+                         (string-match file-name-invalid-regexp ename)))
          (buffer (get-buffer bufname))
          (just-created nil))
       (if buffer
@@ -814,17 +926,30 @@ This function changes the set of information shown for each files."
           (make-local-variable 'local-write-file-hooks)
           (add-hook 'local-write-file-hooks 'archive-write-file-member)
           (setq archive-subfile-mode descr)
-;        (if (boundp 'default-buffer-file-type)
-;            (setq buffer-file-type t))
          (if (and
               (null
-               (if (fboundp extractor)
-                   (funcall extractor archive ename)
-                 (archive-*-extract archive ename (symbol-value extractor))))
+               (let (;; We may have to encode file name arguement for
+                     ;; external programs.
+                     (coding-system-for-write
+                      (and enable-multibyte-characters
+                           file-name-coding-system))
+                     ;; We read an archive member by no-conversion at
+                     ;; first, then decode appropriately by calling
+                     ;; archive-set-buffer-as-visiting-file later.
+                     (coding-system-for-read 'no-conversion))
+                 (condition-case err
+                     (if (fboundp extractor)
+                         (funcall extractor archive ename)
+                       (archive-*-extract archive ename
+                                          (symbol-value extractor)))
+                   (error
+                    (ding (message "%s" (error-message-string err)))
+                    nil))))
               just-created)
              (progn
                (set-buffer-modified-p nil)
                (kill-buffer buffer))
+           (archive-set-buffer-as-visiting-file ename)
            (goto-char (point-min))
            (rename-buffer bufname)
            (setq buffer-read-only read-only-p)
@@ -834,18 +959,25 @@ This function changes the set of information shown for each files."
            (normal-mode)
            ;; Just in case an archive occurs inside another archive.
            (if (eq major-mode 'archive-mode)
-               (setq archive-remote t))
-           (run-hooks 'archive-extract-hooks))
-         (archive-maybe-update t)))
+               (progn
+                 (setq archive-remote t)
+                 (if read-only-p (setq archive-read-only t))
+                 ;; We will write out the archive ourselves if it is
+                 ;; part of another archive.
+                 (remove-hook 'write-contents-hooks 'archive-write-file t)))
+           (run-hooks 'archive-extract-hooks)
+           (if archive-read-only
+               (message "Note: altering this archive is not implemented."))))
+       (archive-maybe-update t))
       (or (not (buffer-name buffer))
          (progn
            (if view-p
-               (view-buffer buffer (and just-created 'kill-buffer)))
-           (if (eq other-window-p 'display)
-               (display-buffer buffer)
-             (if other-window-p
-                 (switch-to-buffer-other-window buffer)
-               (switch-to-buffer buffer)))))))
+               (view-buffer buffer (and just-created 'kill-buffer))
+             (if (eq other-window-p 'display)
+                 (display-buffer buffer)
+               (if other-window-p
+                   (switch-to-buffer-other-window buffer)
+                 (switch-to-buffer buffer))))))))
 
 (defun archive-*-extract (archive name command)
   (let* ((default-directory (file-name-as-directory archive-tmpdir))
@@ -876,17 +1008,12 @@ This function changes the set of information shown for each files."
     success))
 
 (defun archive-extract-by-stdout (archive name command)
-  ;; We need the coding system of the output of the extract program,
-  ;; including the EOL encoding, be decoded dynamically, since what
-  ;; the extract program outputs is the contents of some file.
-  (let ((coding-system-for-read (or coding-system-for-read 'undecided))
-       (inherit-process-coding-system t))
-    (apply 'call-process
-          (car command)
-          nil
-          t
-          nil
-          (append (cdr command) (list archive name)))))
+  (apply 'call-process
+        (car command)
+        nil
+        t
+        nil
+        (append (cdr command) (list archive name))))
 
 (defun archive-extract-other-window ()
   "In archive mode, find this member in another window."
@@ -952,17 +1079,21 @@ This function changes the set of information shown for each files."
       (let ((writer  (save-excursion (set-buffer archive-superior-buffer)
                                     (archive-name "write-file-member")))
            (archive (save-excursion (set-buffer archive-superior-buffer)
-                                    (buffer-file-name))))
+                                    (archive-maybe-copy (buffer-file-name)))))
        (if (fboundp writer)
            (funcall writer archive archive-subfile-mode)
          (archive-*-write-file-member archive
                                       archive-subfile-mode
-                                      (symbol-value writer))))
-      (set-buffer-modified-p nil)
-      (message "Updating archive...done")
+                                      (symbol-value writer)))
+       (set-buffer-modified-p nil)
+       (message "Updating archive...done"))
       (set-buffer archive-superior-buffer)
-      (revert-buffer)
-      t)))
+      (if (not archive-remote) (revert-buffer) (archive-maybe-update nil))))
+  ;; Restore the value of last-coding-system-used, so that basic-save-buffer
+  ;; won't reset the coding-system of this archive member.
+  (if (local-variable-p 'archive-member-coding-system)
+      (setq last-coding-system-used archive-member-coding-system))
+  t)
 
 (defun archive-*-write-file-member (archive descr command)
   (let* ((ename (aref descr 0))
@@ -972,10 +1103,22 @@ This function changes the set of information shown for each files."
     (unwind-protect
         (progn
           (make-directory (file-name-directory tmpfile) t)
-         (write-region (point-min) (point-max) tmpfile nil 'nomessage)
+         ;; If the member is itself an archive, write it without
+         ;; the dired-like listing we created.
+         (if (eq major-mode 'archive-mode)
+             (archive-write-file tmpfile)
+           (write-region (point-min) (point-max) tmpfile nil 'nomessage))
+         ;; basic-save-buffer needs last-coding-system-used to have
+         ;; the value used to write the file, so save it before any
+         ;; further processing clobbers it (we restore it in
+         ;; archive-write-file-member, above).
+         (setq archive-member-coding-system last-coding-system-used)
          (if (aref descr 3)
              ;; Set the file modes, but make sure we can read it.
              (set-file-modes tmpfile (logior ?\400 (aref descr 3))))
+         (if enable-multibyte-characters
+             (setq ename
+                   (encode-coding-string ename file-name-coding-system)))
           (let ((exitcode (apply 'call-process
                                  (car command)
                                  nil
@@ -987,10 +1130,12 @@ This function changes the set of information shown for each files."
               (error "Updating was unsuccessful (%S)" exitcode))))
       (archive-delete-local tmpfile))))
 
-(defun archive-write-file ()
+(defun archive-write-file (&optional file)
   (save-excursion
-    (write-region archive-proper-file-start (point-max) buffer-file-name nil t)
-    (set-buffer-modified-p nil)
+    (let ((coding-system-for-write 'no-conversion))
+      (write-region archive-proper-file-start (point-max)
+                   (or file buffer-file-name) nil t)
+      (set-buffer-modified-p nil))
     t))
 ;; -------------------------------------------------------------------------
 ;; Section: Marking and unmarking.
@@ -1151,15 +1296,21 @@ as a relative change like \"g+rw\" as for chmod(2)"
        (descr (archive-get-descr)))
     (if (fboundp func)
         (progn
-         (funcall func (buffer-file-name) newname descr)
+         (funcall func (buffer-file-name)
+                  (if enable-multibyte-characters
+                      (encode-coding-string newname file-name-coding-system)
+                    newname)
+                  descr)
          (archive-resummarize))
       (error "Renaming is not supported for this archive type"))))
 
 ;; Revert the buffer and recompute the dired-like listing.
-(defun archive-mode-revert (&optional no-autosave no-confirm)
+(defun archive-mode-revert (&optional no-auto-save no-confirm)
   (let ((no (archive-get-lineno)))
     (setq archive-files nil)
-    (let ((revert-buffer-function nil))
+    (let ((revert-buffer-function nil)
+         (coding-system-for-read 'no-conversion))
+      (set-buffer-multibyte nil)
       (revert-buffer t t))
     (archive-mode)
     (goto-char archive-file-list-start)
@@ -1232,6 +1383,7 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
     (save-restriction
       (save-excursion
        (widen)
+       (set-buffer-multibyte nil)
        (goto-char (+ archive-proper-file-start (aref descr 4) 2))
        (delete-char 13)
        (insert name)))))
@@ -1253,9 +1405,13 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
             (moddate (archive-l-e (+ p 17) 2))
             (hdrlvl  (char-after (+ p 20)))
             (fnlen   (char-after (+ p 21)))
-            (efnname (buffer-substring (+ p 22) (+ p 22 fnlen)))
+            (efnname (let ((str (buffer-substring (+ p 22) (+ p 22 fnlen))))
+                       (if file-name-coding-system
+                           (decode-coding-string str file-name-coding-system)
+                         (string-as-multibyte str))))
             (fiddle  (string= efnname (upcase efnname)))
              (ifnname (if fiddle (downcase efnname) efnname))
+            (width (string-width ifnname))
             (p2      (+ p 22 fnlen))
             (creator (if (>= (- hsize fnlen) 24) (char-after (+ p2 2)) 0))
             mode modestr uid gid text path prname
@@ -1300,7 +1456,7 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
                                (archive-dosdate moddate)
                                (archive-dostime modtime)
                                ifnname)))
-        (setq maxlen (max maxlen fnlen)
+        (setq maxlen (max maxlen width)
              totalsize (+ totalsize ucsize)
              visual (cons (vector text
                                   (- (length text) (length ifnname))
@@ -1310,6 +1466,7 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
                           files)
               p (+ p hsize 2 csize))))
     (goto-char (point-min))
+    (set-buffer-multibyte default-enable-multibyte-characters)
     (let ((dash (concat (if archive-alternate-display
                            "- --------  -----  -----  "
                          "- ----------  --------  -----------  --------  ")
@@ -1348,6 +1505,7 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
   (save-restriction
     (save-excursion
       (widen)
+      (set-buffer-multibyte nil)
       (let* ((p        (+ archive-proper-file-start (aref descr 4)))
             (oldhsize (char-after p))
             (oldfnlen (char-after (+ p 21)))
@@ -1367,6 +1525,7 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
   (save-restriction
     (save-excursion
       (widen)
+      (set-buffer-multibyte nil)
       (while files
        (let* ((fil (car files))
               (p (+ archive-proper-file-start (aref fil 4)))
@@ -1421,12 +1580,15 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
              (exlen   (archive-l-e (+ p 30) 2))
              (fclen   (archive-l-e (+ p 32) 2))
              (lheader (archive-l-e (+ p 42) 4))
-             (efnname (buffer-substring (+ p 46) (+ p 46 fnlen)))
+             (efnname (let ((str (buffer-substring (+ p 46) (+ p 46 fnlen))))
+                       (if file-name-coding-system
+                           (decode-coding-string str file-name-coding-system)
+                         (string-as-multibyte str))))
             (isdir   (and (= ucsize 0)
                           (string= (file-name-nondirectory efnname) "")))
             (mode    (cond ((memq creator '(2 3)) ; Unix + VMS
                             (archive-l-e (+ p 40) 2))
-                           ((memq creator '(0 5 6 7 10 11)) ; Dos etc.
+                           ((memq creator '(0 5 6 7 10 11 15)) ; Dos etc.
                             (logior ?\444
                                     (if isdir (logior 16384 ?\111) 0)
                                     (if (zerop
@@ -1435,15 +1597,17 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
                            (t nil)))
             (modestr (if mode (archive-int-to-mode mode) "??????????"))
             (fiddle  (and archive-zip-case-fiddle
-                          (not (not (memq creator '(0 2 4 5 9))))))
+                          (not (not (memq creator '(0 2 4 5 9))))
+                          (string= (upcase efnname) efnname)))
              (ifnname (if fiddle (downcase efnname) efnname))
+            (width (string-width ifnname))
              (text    (format "  %10s  %8d  %-11s  %-8s  %s"
                              modestr
                               ucsize
                               (archive-dosdate moddate)
                               (archive-dostime modtime)
                               ifnname)))
-        (setq maxlen (max maxlen fnlen)
+        (setq maxlen (max maxlen width)
              totalsize (+ totalsize ucsize)
              visual (cons (vector text
                                   (- (length text) (length ifnname))
@@ -1485,6 +1649,7 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
   (save-restriction
     (save-excursion
       (widen)
+      (set-buffer-multibyte nil)
       (while files
        (let* ((fil (car files))
               (p (+ archive-proper-file-start (car (aref fil 4))))
@@ -1496,7 +1661,7 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
                 (goto-char (+ p 40))
                 (delete-char 2)
                 (insert (logand newval 255) (lsh newval -8)))
-               ((memq creator '(0 5 6 7 10 11)) ; Dos etc.
+               ((memq creator '(0 5 6 7 10 11 15)) ; Dos etc.
                 (goto-char (+ p 38))
                 (insert (logior (logand (char-after (point)) 254)
                                 (logand (logxor 1 (lsh newval -7)) 1)))
@@ -1523,23 +1688,30 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
             (lfnlen  (if (= dirtype 2) (char-after (+ p 56)) 0))
             (ldirlen (if (= dirtype 2) (char-after (+ p 57)) 0))
             (fnlen   (or (string-match "\0" namefld) 13))
-            (efnname (concat
-                      (if (> ldirlen 0)
-                          (concat (buffer-substring
-                                   (+ p 58 lfnlen) (+ p 58 lfnlen ldirlen -1))
-                                  "/")
-                        "")
-                      (if (> lfnlen 0)
-                          (buffer-substring (+ p 58) (+ p 58 lfnlen -1))
-                        (substring namefld 0 fnlen))))
+            (efnname (let ((str
+                            (concat
+                             (if (> ldirlen 0)
+                                 (concat (buffer-substring
+                                          (+ p 58 lfnlen)
+                                          (+ p 58 lfnlen ldirlen -1))
+                                         "/")
+                               "")
+                             (if (> lfnlen 0)
+                                 (buffer-substring (+ p 58)
+                                                   (+ p 58 lfnlen -1))
+                               (substring namefld 0 fnlen)))))
+                       (if file-name-coding-system
+                           (decode-coding-string str file-name-coding-system)
+                         (string-as-multibyte str))))
             (fiddle  (and (= lfnlen 0) (string= efnname (upcase efnname))))
              (ifnname (if fiddle (downcase efnname) efnname))
+            (width (string-width ifnname))
              (text    (format "  %8d  %-11s  %-8s  %s"
                               ucsize
                               (archive-dosdate moddate)
                               (archive-dostime modtime)
                               ifnname)))
-        (setq maxlen (max maxlen (length ifnname))
+        (setq maxlen (max maxlen width)
              totalsize (+ totalsize ucsize)
              visual (cons (vector text
                                   (- (length text) (length ifnname))
@@ -1566,6 +1738,10 @@ This doesn't recover lost files, it just undoes changes in the buffer itself."
 (defun archive-zoo-extract (archive name)
   (archive-extract-by-stdout archive name archive-zoo-extract))
 ;; -------------------------------------------------------------------------
+;; This line was a mistake; it is kept now for compatibility.
+;; rms  15 Oct 98
 (provide 'archive-mode)
 
-;; arc-mode.el ends here.
+(provide 'arc-mode)
+
+;;; arc-mode.el ends here