(revert-buffer): Doc fix.
[bpt/emacs.git] / lisp / files.el
index 0eadeb9..be31158 100644 (file)
@@ -1,6 +1,6 @@
 ;;; files.el --- file input and output commands for Emacs
 
-;; Copyright (C) 1985, 86, 87, 92, 93, 94 Free Software Foundation, Inc.
+;; Copyright (C) 1985, 86, 87, 92, 93, 94, 1995 Free Software Foundation, Inc.
 
 ;; Maintainer: FSF
 
@@ -115,7 +115,7 @@ both at the file level and at the levels of the containing directories.")
 
 (defvar buffer-file-truename nil
   "The abbreviated truename of the file visited in the current buffer.
-That is, (abbreviated-file-name (file-truename buffer-file-name)).
+That is, (abbreviate-file-name (file-truename buffer-file-name)).
 This variable is automatically local in all buffers, when non-nil.")
 (make-variable-buffer-local 'buffer-file-truename)
 (put 'buffer-file-truename 'permanent-local t)
@@ -351,8 +351,15 @@ Do not specify them in other calls."
        (if (string= filename "")
            (setq filename "/"))))
   (or counter (setq counter (list 100)))
-  (or prev-dirs (setq prev-dirs (list nil)))
-  (let (done)
+  (let (done
+       ;; For speed, remove the ange-ftp completion handler from the list.
+       ;; We know it's not needed here.
+       ;; For even more speed, do this only on the outermost call.
+       (file-name-handler-alist
+        (if prev-dirs file-name-handler-alist
+          (let ((tem (copy-sequence file-name-handler-alist)))
+            (delq (rassq 'ange-ftp-completion-hook-function tem) tem)))))
+    (or prev-dirs (setq prev-dirs (list nil)))
     ;; If this file directly leads to a link, process that iteratively
     ;; so that we don't use lots of stack.
     (while (not done)
@@ -365,7 +372,7 @@ Do not specify them in other calls."
        (if handler
            (setq filename (funcall handler 'file-truename filename)
                  done t)
-         (let ((dir (file-name-directory filename))
+         (let ((dir (or (file-name-directory filename) default-directory))
                target dirfile)
            ;; Get the truename of the directory.
            (setq dirfile (directory-file-name dir))
@@ -509,7 +516,7 @@ If the current buffer now contains an empty file that you just visited
                file-dir (file-name-directory file)))
      (list (read-file-name
            "Find alternate file: " file-dir nil nil file-name))))
-  (and (buffer-modified-p)
+  (and (buffer-modified-p) (buffer-file-name)
        ;; (not buffer-read-only)
        (not (yes-or-no-p (format "Buffer %s is modified; kill anyway? "
                                 (buffer-name))))
@@ -590,7 +597,8 @@ Type \\[describe-variable] directory-abbrev-alist RET for more information."
             ;; If the home dir is just /, don't change it.
             (not (and (= (match-end 0) 1)
                       (= (aref filename 0) ?/)))
-            (not (and (eq system-type 'ms-dos)
+            (not (and (or (eq system-type 'ms-dos) 
+                          (eq system-type 'windows-nt))
                       (save-match-data
                         (string-match "^[a-zA-Z]:/$" filename)))))
        (setq filename
@@ -677,7 +685,7 @@ The buffer is not selected, just returned to the caller."
                      (format
                       (if (buffer-modified-p buf)
     "File %s changed on disk.  Discard your edits? "
-    "File %s changed on disk.  Read the new version? ")
+    "File %s changed on disk.  Reread from disk? ")
                       (file-name-nondirectory filename)))
                     (save-excursion
                       (set-buffer buf)
@@ -691,21 +699,16 @@ The buffer is not selected, just returned to the caller."
 ;;;            (message "Symbolic link to file in buffer %s"
 ;;;                     (buffer-name linked-buf))))
          (setq buf (create-file-buffer filename))
+         (set-buffer-major-mode buf)
          (set-buffer buf)
          (erase-buffer)
          (condition-case ()
              (insert-file-contents filename t)
            (file-error
-            (setq error t)
             ;; Run find-file-not-found-hooks until one returns non-nil.
-            (let ((hooks find-file-not-found-hooks))
-              (while (and hooks
-                          (not (and (funcall (car hooks))
-                                    ;; If a hook succeeded, clear error.
-                                    (progn (setq error nil)
-                                           ;; Also exit the loop.
-                                           t))))
-                (setq hooks (cdr hooks))))))
+            (or (run-hook-with-args-until-success 'find-file-not-found-hooks)
+                ;; If they fail too, set error.
+                (setq error t))))
          ;; Find the file's truename, and maybe use that as visited name.
          (setq buffer-file-truename truename)
          (setq buffer-file-number number)
@@ -733,13 +736,18 @@ The buffer is not selected, just returned to the caller."
          (after-find-file error (not nowarn))))
       buf)))
 \f
-(defun after-find-file (&optional error warn noauto)
+(defvar after-find-file-from-revert-buffer nil)
+
+(defun after-find-file (&optional error warn noauto
+                                 after-find-file-from-revert-buffer)
   "Called after finding a file and by the default revert function.
 Sets buffer mode, parses local variables.
 Optional args ERROR, WARN, and NOAUTO: ERROR non-nil means there was an
 error in reading the file.  WARN non-nil means warn if there
 exists an auto-save file more recent than the visited file.
 NOAUTO means don't mess with auto-save mode.
+Fourth arg AFTER-FIND-FILE-FROM-REVERT-BUFFER non-nil
+ means this call was from `revert-buffer'.
 Finishes by calling the functions in `find-file-hooks'."
   (setq buffer-read-only (not (file-writable-p buffer-file-name)))
   (if noninteractive
@@ -775,7 +783,7 @@ Finishes by calling the functions in `find-file-hooks'."
     (if (and auto-save-default (not noauto))
        (auto-save-mode t)))
   (normal-mode t)
-  (mapcar 'funcall find-file-hooks))
+  (run-hooks 'find-file-hooks))
 
 (defun normal-mode (&optional find-file)
   "Choose the major mode for this buffer automatically.
@@ -804,8 +812,8 @@ run `normal-mode' explicitly."
                                '(("\\.text\\'" . text-mode)
                                  ("\\.c\\'" . c-mode)
                                  ("\\.h\\'" . c-mode)
-                                 ("\\.tex\\'" . TeX-mode)
-                                 ("\\.ltx\\'" . LaTeX-mode)
+                                 ("\\.tex\\'" . tex-mode)
+                                 ("\\.ltx\\'" . latex-mode)
                                  ("\\.el\\'" . emacs-lisp-mode)
                                  ("\\.mm\\'" . nroff-mode)
                                  ("\\.me\\'" . nroff-mode)
@@ -819,11 +827,18 @@ run `normal-mode' explicitly."
                                  ("\\.p\\'" . pascal-mode)
                                  ("\\.pas\\'" . pascal-mode)
                                  ("\\.mss\\'" . scribe-mode)
+                                 ("\\.ada\\'" . ada-mode)
+                                 ("\\.icn\\'" . icon-mode)
                                  ("\\.pl\\'" . prolog-mode)
                                  ("\\.cc\\'" . c++-mode)
                                  ("\\.hh\\'" . c++-mode)
                                  ("\\.C\\'" . c++-mode)
                                  ("\\.H\\'" . c++-mode)
+                                 ("\\.cpp\\'" . c++-mode)
+                                 ("\\.cxx\\'" . c++-mode)
+                                 ("\\.hxx\\'" . c++-mode)
+                                 ("\\.c\\+\\+\\'" . c++-mode)
+                                 ("\\.h\\+\\+\\'" . c++-mode)
 ;;;                              ("\\.mk\\'" . makefile-mode)
 ;;;                              ("[Mm]akefile" . makefile-mode)
 ;;; Less common extensions come here
@@ -839,9 +854,10 @@ run `normal-mode' explicitly."
 ;; The following should come after the ChangeLog pattern
 ;; for the sake of ChangeLog.1, etc.
                                  ("\\.[12345678]\\'" . nroff-mode)
-                                 ("\\.TeX\\'" . TeX-mode)
-                                 ("\\.sty\\'" . LaTeX-mode)
-                                 ("\\.bbl\\'" . LaTeX-mode)
+                                 ("\\.TeX\\'" . tex-mode)
+                                 ("\\.sty\\'" . latex-mode)
+                                 ("\\.cls\\'" . latex-mode) ;LaTeX 2e class
+                                 ("\\.bbl\\'" . latex-mode)
                                  ("\\.bib\\'" . bibtex-mode)
                                  ("\\.article\\'" . text-mode)
                                  ("\\.letter\\'" . text-mode)
@@ -854,6 +870,7 @@ run `normal-mode' explicitly."
                                  ;; /tmp/Re.... or Message
                                  ("^/tmp/Re" . text-mode)
                                  ("/Message[0-9]*\\'" . text-mode)
+                                 ("/drafts/[0-9]+\\'" . mh-letter-mode)
                                  ;; some news reader is reported to use this
                                  ("^/tmp/fol/" . text-mode)
                                  ("\\.y\\'" . c-mode)
@@ -872,11 +889,14 @@ run `normal-mode' explicitly."
                                  ("\\.ml\\'" . lisp-mode)))
   "\
 Alist of filename patterns vs corresponding major mode functions.
-Each element looks like (REGEXP . FUNCTION) or (REGEXP FUNCTION).
-Visiting a file whose name matches REGEXP causes FUNCTION to be called.
-If the element has the form (REGEXP FUNCTION), then after calling
-FUNCTION, we delete the suffix that matched REGEXP and search the list
-again for another match.")
+Each element looks like (REGEXP . FUNCTION) or (REGEXP FUNCTION NON-NIL).
+\(NON-NIL stands for anything that is not nil; the value does not matter.)
+Visiting a file whose name matches REGEXP specifies FUNCTION as the
+mode function to use.  FUNCTION will be called, unless it is nil.
+
+If the element has the form (REGEXP FUNCTION NON-NIL), then after
+calling FUNCTION (if it's not nil), we delete the suffix that matched
+REGEXP and search the list again for another match.")
 
 (defconst interpreter-mode-alist
   '(("perl" . perl-mode)
@@ -921,10 +941,11 @@ If `enable-local-variables' is nil, this function does not check for a
       (and enable-local-variables
           ;; Don't look for -*- if this file name matches any
           ;; of the regexps in inhibit-first-line-modes-regexps.
-          (let ((temp inhibit-first-line-modes-regexps))
+          (let ((temp inhibit-first-line-modes-regexps)
+                (name (file-name-sans-versions buffer-file-name)))
             (while (and temp
                         (not (string-match (car temp)
-                                           buffer-file-name)))
+                                           name)))
               (setq temp (cdr temp)))
             (not temp))
           (search-forward "-*-" (save-excursion
@@ -952,7 +973,9 @@ If `enable-local-variables' is nil, this function does not check for a
                 ;; Find all specifications for the `mode:' variable
                 ;; and execute them left to right.
                 (while (let ((case-fold-search t))
-                         (search-forward "mode:" end t))
+                         (or (and (looking-at "mode:")
+                                  (goto-char (match-end 0)))
+                             (re-search-forward "[ \t;]mode:" end t)))
                   (skip-chars-forward " \t")
                   (setq beg (point))
                   (if (search-forward ";" end t)
@@ -1061,6 +1084,11 @@ If `enable-local-variables' is nil, this function does not check for a
            (hack-one-local-variable (car (car result)) (cdr (car result)))
            (setq result (cdr result)))))))
 
+(defvar hack-local-variables-hook nil
+  "Normal hook run after processing a file's local variables specs.
+Major modes can use this to examine user-specified local variables
+in order to initialize other data structure based on them.")
+
 (defun hack-local-variables ()
   "Parse and put into effect this buffer's local variables spec."
   (hack-local-variables-prop-line)
@@ -1131,7 +1159,8 @@ If `enable-local-variables' is nil, this function does not check for a
                (or (if suffix (looking-at suffix) (eolp))
                    (error "Local variables entry is terminated incorrectly"))
                ;; Set the variable.  "Variables" mode and eval are funny.
-               (hack-one-local-variable var val))))))))
+               (hack-one-local-variable var val)))))))
+  (run-hooks 'hack-local-variables-hook))
 
 (defconst ignored-local-variables
   '(enable-local-eval)
@@ -1140,6 +1169,7 @@ If `enable-local-variables' is nil, this function does not check for a
 ;; Get confirmation before setting these variables as locals in a file.
 (put 'debugger 'risky-local-variable t)
 (put 'enable-local-eval 'risky-local-variable t)
+(put 'ignored-local-variables 'risky-local-variable t)
 (put 'eval 'risky-local-variable t)
 (put 'file-name-handler-alist 'risky-local-variable t)
 (put 'minor-mode-map-alist 'risky-local-variable t)
@@ -1155,6 +1185,9 @@ If `enable-local-variables' is nil, this function does not check for a
 (put 'outline-level 'risky-local-variable t)
 (put 'rmail-output-file-alist 'risky-local-variable t)
 
+;; This one is safe because the user gets to check it before it is used.
+(put 'compile-command 'safe-local-variable t)
+
 (defun hack-one-local-variable-quotep (exp)
   (and (consp exp) (eq (car exp) 'quote) (consp (cdr exp))))
 
@@ -1169,8 +1202,10 @@ If `enable-local-variables' is nil, this function does not check for a
        ;; "Setting" eval means either eval it or do nothing.
        ;; Likewise for setting hook variables.
        ((or (get var 'risky-local-variable)
-            (string-match "-hooks?$\\|-functions?$\\|-forms?$\\|-program$\\|-command$"
-                          (symbol-name var)))
+            (and
+             (string-match "-hooks?$\\|-functions?$\\|-forms?$\\|-program$\\|-command$"
+                           (symbol-name var))
+             (not (get var 'safe-local-variable))))
         ;; Permit evaling a put of a harmless property
         ;; if the args do nothing tricky.
         (if (or (and (eq var 'eval)
@@ -1213,6 +1248,8 @@ nil or empty string as argument means make buffer not be visiting any file.
 Remember to delete the initial contents of the minibuffer
 if you wish to pass an empty string as the argument."
   (interactive "FSet visited file name: ")
+  (if (buffer-base-buffer)
+      (error "An indirect buffer cannot visit a file"))
   (let (truename)
     (if filename
        (setq filename
@@ -1332,63 +1369,64 @@ the modes of the new file to agree with the old modes."
              targets (cdr backup-info))
 ;;;     (if (file-directory-p buffer-file-name)
 ;;;         (error "Cannot save buffer in directory %s" buffer-file-name))
-        (condition-case ()
-           (let ((delete-old-versions
-                  ;; If have old versions to maybe delete,
-                  ;; ask the user to confirm now, before doing anything.
-                  ;; But don't actually delete til later.
-                  (and targets
-                       (or (eq delete-old-versions t) (eq delete-old-versions nil))
-                       (or delete-old-versions
-                           (y-or-n-p (format "Delete excess backup versions of %s? "
-                                             real-file-name))))))
-             ;; Actually write the back up file.
-             (condition-case ()
-                 (if (or file-precious-flag
-;                        (file-symlink-p buffer-file-name)
-                         backup-by-copying
-                         (and backup-by-copying-when-linked
-                              (> (file-nlinks real-file-name) 1))
-                         (and backup-by-copying-when-mismatch
-                              (let ((attr (file-attributes real-file-name)))
-                                (or (nth 9 attr)
-                                    (/= (nth 2 attr) (user-uid))))))
-                     (condition-case ()
-                         (copy-file real-file-name backupname t t)
-                       (file-error
-                        ;; If copying fails because file BACKUPNAME
-                        ;; is not writable, delete that file and try again.
-                        (if (and (file-exists-p backupname)
-                                 (not (file-writable-p backupname)))
-                            (delete-file backupname))
-                        (copy-file real-file-name backupname t t)))
-                   ;; rename-file should delete old backup.
-                   (rename-file real-file-name backupname t)
-                   (setq setmodes (file-modes backupname)))
-               (file-error
-                ;; If trouble writing the backup, write it in ~.
-                (setq backupname (expand-file-name "~/%backup%~"))
-                (message "Cannot write backup file; backing up in ~/%%backup%%~")
-                (sleep-for 1)
-                (condition-case ()
-                    (copy-file real-file-name backupname t t)
-                  (file-error
-                   ;; If copying fails because file BACKUPNAME
-                   ;; is not writable, delete that file and try again.
-                   (if (and (file-exists-p backupname)
-                            (not (file-writable-p backupname)))
-                       (delete-file backupname))
-                   (copy-file real-file-name backupname t t)))))
-             (setq buffer-backed-up t)
-             ;; Now delete the old versions, if desired.
-             (if delete-old-versions
-                 (while targets
-                   (condition-case ()
-                       (delete-file (car targets))
-                     (file-error nil))
-                   (setq targets (cdr targets))))
-             setmodes)
-       (file-error nil)))))
+       (if backup-info
+           (condition-case ()
+               (let ((delete-old-versions
+                      ;; If have old versions to maybe delete,
+                      ;; ask the user to confirm now, before doing anything.
+                      ;; But don't actually delete til later.
+                      (and targets
+                           (or (eq delete-old-versions t) (eq delete-old-versions nil))
+                           (or delete-old-versions
+                               (y-or-n-p (format "Delete excess backup versions of %s? "
+                                                 real-file-name))))))
+                 ;; Actually write the back up file.
+                 (condition-case ()
+                     (if (or file-precious-flag
+    ;                    (file-symlink-p buffer-file-name)
+                             backup-by-copying
+                             (and backup-by-copying-when-linked
+                                  (> (file-nlinks real-file-name) 1))
+                             (and backup-by-copying-when-mismatch
+                                  (let ((attr (file-attributes real-file-name)))
+                                    (or (nth 9 attr)
+                                        (not (file-ownership-preserved-p real-file-name))))))
+                         (condition-case ()
+                             (copy-file real-file-name backupname t t)
+                           (file-error
+                            ;; If copying fails because file BACKUPNAME
+                            ;; is not writable, delete that file and try again.
+                            (if (and (file-exists-p backupname)
+                                     (not (file-writable-p backupname)))
+                                (delete-file backupname))
+                            (copy-file real-file-name backupname t t)))
+                       ;; rename-file should delete old backup.
+                       (rename-file real-file-name backupname t)
+                       (setq setmodes (file-modes backupname)))
+                   (file-error
+                    ;; If trouble writing the backup, write it in ~.
+                    (setq backupname (expand-file-name "~/%backup%~"))
+                    (message "Cannot write backup file; backing up in ~/%%backup%%~")
+                    (sleep-for 1)
+                    (condition-case ()
+                        (copy-file real-file-name backupname t t)
+                      (file-error
+                       ;; If copying fails because file BACKUPNAME
+                       ;; is not writable, delete that file and try again.
+                       (if (and (file-exists-p backupname)
+                                (not (file-writable-p backupname)))
+                           (delete-file backupname))
+                       (copy-file real-file-name backupname t t)))))
+                 (setq buffer-backed-up t)
+                 ;; Now delete the old versions, if desired.
+                 (if delete-old-versions
+                     (while targets
+                       (condition-case ()
+                           (delete-file (car targets))
+                         (file-error nil))
+                       (setq targets (cdr targets))))
+                 setmodes)
+           (file-error nil))))))
 
 (defun file-name-sans-versions (name &optional keep-backup-version)
   "Return FILENAME sans backup versions or strings.
@@ -1417,6 +1455,26 @@ we do not remove backup version numbers, only true file version numbers."
                         (string-match "~\\'" name)
                         (length name))))))))
 
+(defun file-ownership-preserved-p (file)
+  "Returns t if deleting FILE and rewriting it would preserve the owner."
+  (let ((handler (find-file-name-handler file 'file-ownership-preserved-p)))
+    (if handler
+       (funcall handler 'file-ownership-preserved-p file)
+      (= (nth 2 (file-attributes file)) (user-uid)))))
+
+(defun file-name-sans-extension (filename)
+  "Return FILENAME sans final \"extension\".
+The extension, in a file name, is the part that follows the last `.'."
+  (save-match-data
+    (let ((file (file-name-sans-versions (file-name-nondirectory filename)))
+         directory)
+      (if (string-match "\\.[^.]*\\'" file)
+         (if (setq directory (file-name-directory filename))
+             (expand-file-name (substring file 0 (match-beginning 0))
+                               directory)
+           (substring file 0 (match-beginning 0)))
+       filename))))
+
 (defun make-backup-file-name (file)
   "Create the non-numeric backup file name for FILE.
 This is a separate function so you can redefine it for customization."
@@ -1454,43 +1512,48 @@ the index in the name where the version number begins."
 (defun find-backup-file-name (fn)
   "Find a file name for a backup file, and suggestions for deletions.
 Value is a list whose car is the name for the backup file
- and whose cdr is a list of old versions to consider deleting now."
-  (if (eq version-control 'never)
-      (list (make-backup-file-name fn))
-    (let* ((base-versions (concat (file-name-nondirectory fn) ".~"))
-          (bv-length (length base-versions))
-          possibilities
-          (versions nil)
-          (high-water-mark 0)
-          (deserve-versions-p nil)
-          (number-to-delete 0))
-      (condition-case ()
-         (setq possibilities (file-name-all-completions
-                              base-versions
-                              (file-name-directory fn))
-               versions (sort (mapcar
-                               (function backup-extract-version)
-                               possibilities)
-                              '<)
-               high-water-mark (apply 'max 0 versions)
-               deserve-versions-p (or version-control
-                                      (> high-water-mark 0))
-               number-to-delete (- (length versions)
-                                   kept-old-versions kept-new-versions -1))
-       (file-error
-        (setq possibilities nil)))
-      (if (not deserve-versions-p)
+ and whose cdr is a list of old versions to consider deleting now.
+If the value is nil, don't make a backup."
+  (let ((handler (find-file-name-handler fn 'find-backup-file-name)))
+    ;; Run a handler for this function so that ange-ftp can refuse to do it.
+    (if handler
+       (funcall handler 'find-backup-file-name fn)
+      (if (eq version-control 'never)
          (list (make-backup-file-name fn))
-       (cons (concat fn ".~" (int-to-string (1+ high-water-mark)) "~")
-             (if (and (> number-to-delete 0)
-                       ;; Delete nothing if there is overflow
-                      ;; in the number of versions to keep.
-                      (>= (+ kept-new-versions kept-old-versions -1) 0))
-                 (mapcar (function (lambda (n)
-                                     (concat fn ".~" (int-to-string n) "~")))
-                         (let ((v (nthcdr kept-old-versions versions)))
-                           (rplacd (nthcdr (1- number-to-delete) v) ())
-                           v))))))))
+       (let* ((base-versions (concat (file-name-nondirectory fn) ".~"))
+              (bv-length (length base-versions))
+              possibilities
+              (versions nil)
+              (high-water-mark 0)
+              (deserve-versions-p nil)
+              (number-to-delete 0))
+         (condition-case ()
+             (setq possibilities (file-name-all-completions
+                                  base-versions
+                                  (file-name-directory fn))
+                   versions (sort (mapcar
+                                   (function backup-extract-version)
+                                   possibilities)
+                                  '<)
+                   high-water-mark (apply 'max 0 versions)
+                   deserve-versions-p (or version-control
+                                          (> high-water-mark 0))
+                   number-to-delete (- (length versions)
+                                       kept-old-versions kept-new-versions -1))
+           (file-error
+            (setq possibilities nil)))
+         (if (not deserve-versions-p)
+             (list (make-backup-file-name fn))
+           (cons (concat fn ".~" (int-to-string (1+ high-water-mark)) "~")
+                 (if (and (> number-to-delete 0)
+                          ;; Delete nothing if there is overflow
+                          ;; in the number of versions to keep.
+                          (>= (+ kept-new-versions kept-old-versions -1) 0))
+                     (mapcar (function (lambda (n)
+                                         (concat fn ".~" (int-to-string n) "~")))
+                             (let ((v (nthcdr kept-old-versions versions)))
+                               (rplacd (nthcdr (1- number-to-delete) v) ())
+                               v))))))))))
 
 (defun file-nlinks (filename)
   "Return number of names file FILENAME has."
@@ -1533,7 +1596,8 @@ If `delete-old-versions' is nil, system will query user
   (interactive "p")
   (let ((modp (buffer-modified-p))
        (large (> (buffer-size) 50000))
-       (make-backup-files (and make-backup-files (not (eq args 0)))))
+       (make-backup-files (or (and make-backup-files (not (eq args 0)))
+                              (memq args '(16 64)))))
     (and modp (memq args '(16 64)) (setq buffer-backed-up nil))
     (if (and modp large) (message "Saving file %s..." (buffer-file-name)))
     (basic-save-buffer)
@@ -1555,65 +1619,66 @@ the last real save, but optional arg FORCE non-nil means delete anyway."
 (defun basic-save-buffer ()
   "Save the current buffer in its visited file, if it has been modified."
   (interactive)
-  (if (buffer-modified-p)
-      (let ((recent-save (recent-auto-save-p))
-           setmodes tempsetmodes)
-       ;; On VMS, rename file and buffer to get rid of version number.
-       (if (and (eq system-type 'vax-vms)
-                (not (string= buffer-file-name
-                              (file-name-sans-versions buffer-file-name))))
-           (let (buffer-new-name)
-             ;; Strip VMS version number before save.
-             (setq buffer-file-name
-                   (file-name-sans-versions buffer-file-name))
-             ;; Construct a (unique) buffer name to correspond.
-             (let ((buf (create-file-buffer (downcase buffer-file-name))))
-               (setq buffer-new-name (buffer-name buf))
-               (kill-buffer buf))
-             (rename-buffer buffer-new-name)))
-       ;; If buffer has no file name, ask user for one.
-       (or buffer-file-name
-           (set-visited-file-name
-            (expand-file-name (read-file-name "File to save in: ") nil)))
-       (or (verify-visited-file-modtime (current-buffer))
-           (not (file-exists-p buffer-file-name))
-           (yes-or-no-p
-            (format "%s has changed since visited or saved.  Save anyway? "
-                    (file-name-nondirectory buffer-file-name)))
-           (error "Save not confirmed"))
-       (save-restriction
-         (widen)
-         (and (> (point-max) 1)
-              (/= (char-after (1- (point-max))) ?\n)
-              (not (and (eq selective-display t)
-                        (= (char-after (1- (point-max))) ?\r)))
-              (or (eq require-final-newline t)
-                  (and require-final-newline
-                       (y-or-n-p
-                        (format "Buffer %s does not end in newline.  Add one? "
-                                (buffer-name)))))
-              (save-excursion
-                (goto-char (point-max))
-                (insert ?\n)))
-         (let ((hooks (append write-contents-hooks local-write-file-hooks
-                              write-file-hooks))
-               (done nil))
-           (while (and hooks
-                       (not (setq done (funcall (car hooks)))))
-             (setq hooks (cdr hooks)))
-           ;; If a hook returned t, file is already "written".
-           (cond ((not done)
-                  (setq setmodes (basic-save-buffer-1)))))
-         (setq buffer-file-number (nth 10 (file-attributes buffer-file-name)))
-         (if setmodes
-             (condition-case ()
-                 (set-file-modes buffer-file-name setmodes)
-               (error nil))))
-       ;; If the auto-save file was recent before this command,
-       ;; delete it now.
-       (delete-auto-save-file-if-necessary recent-save)
-       (run-hooks 'after-save-hook))
-    (message "(No changes need to be saved)")))
+  (save-excursion
+    ;; In an indirect buffer, save its base buffer instead.
+    (if (buffer-base-buffer)
+       (set-buffer (buffer-base-buffer)))
+    (if (buffer-modified-p)
+       (let ((recent-save (recent-auto-save-p))
+             setmodes tempsetmodes)
+         ;; On VMS, rename file and buffer to get rid of version number.
+         (if (and (eq system-type 'vax-vms)
+                  (not (string= buffer-file-name
+                                (file-name-sans-versions buffer-file-name))))
+             (let (buffer-new-name)
+               ;; Strip VMS version number before save.
+               (setq buffer-file-name
+                     (file-name-sans-versions buffer-file-name))
+               ;; Construct a (unique) buffer name to correspond.
+               (let ((buf (create-file-buffer (downcase buffer-file-name))))
+                 (setq buffer-new-name (buffer-name buf))
+                 (kill-buffer buf))
+               (rename-buffer buffer-new-name)))
+         ;; If buffer has no file name, ask user for one.
+         (or buffer-file-name
+             (set-visited-file-name
+              (expand-file-name (read-file-name "File to save in: ") nil)))
+         (or (verify-visited-file-modtime (current-buffer))
+             (not (file-exists-p buffer-file-name))
+             (yes-or-no-p
+              (format "%s has changed since visited or saved.  Save anyway? "
+                      (file-name-nondirectory buffer-file-name)))
+             (error "Save not confirmed"))
+         (save-restriction
+           (widen)
+           (and (> (point-max) 1)
+                (/= (char-after (1- (point-max))) ?\n)
+                (not (and (eq selective-display t)
+                          (= (char-after (1- (point-max))) ?\r)))
+                (or (eq require-final-newline t)
+                    (and require-final-newline
+                         (y-or-n-p
+                          (format "Buffer %s does not end in newline.  Add one? "
+                                  (buffer-name)))))
+                (save-excursion
+                  (goto-char (point-max))
+                  (insert ?\n)))
+           (or (run-hook-with-args-until-success 'write-contents-hooks)
+               (run-hook-with-args-until-success 'local-write-file-hooks)
+               (run-hook-with-args-until-success 'write-file-hooks)
+               ;; If a hook returned t, file is already "written".
+               ;; Otherwise, write it the usual way now.
+               (setq setmodes (basic-save-buffer-1)))
+           (setq buffer-file-number (nth 10 (file-attributes buffer-file-name)))
+           (if setmodes
+               (condition-case ()
+                   (set-file-modes buffer-file-name setmodes)
+                 (error nil))))
+         ;; If the auto-save file was recent before this command,
+         ;; delete it now.
+         (delete-auto-save-file-if-necessary recent-save)
+         (run-hooks 'after-save-hook))
+      (message "(No changes need to be saved)"))))
 
 ;; This does the "real job" of writing a buffer into its visited file
 ;; and making a backup file.  This is what is normally done
@@ -1642,7 +1707,8 @@ the last real save, but optional arg FORCE non-nil means delete anyway."
          ;; This requires write access to the containing dir,
          ;; which is why we don't try it if we don't have that access.
          (let ((realname buffer-file-name)
-               tempname temp nogood i succeed)
+               tempname temp nogood i succeed
+               (old-modtime (visited-file-modtime)))
            (setq i 0)
            (setq nogood t)
            ;; Find the temporary name to write under.
@@ -1657,7 +1723,10 @@ the last real save, but optional arg FORCE non-nil means delete anyway."
                       (setq succeed t))
              ;; If writing the temp file fails,
              ;; delete the temp file.
-             (or succeed (delete-file tempname)))
+             (or succeed 
+                 (progn
+                   (delete-file tempname)
+                   (set-visited-file-modtime old-modtime))))
            ;; Since we have created an entirely new file
            ;; and renamed it, make sure it gets the
            ;; right permission bits set.
@@ -1689,6 +1758,7 @@ Optional second argument EXITING means ask about certain non-file buffers
            (function
             (lambda (buffer)
               (and (buffer-modified-p buffer)
+                   (not (buffer-base-buffer buffer))
                    (or
                     (buffer-file-name buffer)
                     (and exiting
@@ -1731,7 +1801,10 @@ Optional second argument EXITING means ask about certain non-file buffers
 \f
 (defun not-modified (&optional arg)
   "Mark current buffer as unmodified, not needing to be saved.
-With prefix arg, mark buffer as modified, so \\[save-buffer] will save."
+With prefix arg, mark buffer as modified, so \\[save-buffer] will save.
+
+It is not a good idea to use this function in Lisp programs, because it
+prints a message in the minibuffer.  Instead, use `set-buffer-modified-p'."
   (interactive "P")
   (message (if arg "Modification-flag set"
               "Modification-flag cleared"))
@@ -1840,7 +1913,7 @@ This undoes all changes since the file was visited or saved.
 With a prefix argument, offer to revert from latest auto-save file, if
 that is more recent than the visited file.
 
-When called from lisp, The first argument is IGNORE-AUTO; only offer
+When called from Lisp, the first argument is IGNORE-AUTO; only offer
 to revert from the auto-save file when this is nil.  Note that the
 sense of this argument is the reverse of the prefix argument, for the
 sake of backward compatibility.  IGNORE-AUTO is optional, defaulting
@@ -1860,7 +1933,7 @@ beginning and `after-revert-hook' at the end."
   ;; there's no straightforward way to encourage authors to notice a
   ;; reversal of the argument sense.  So I'm just changing the user
   ;; interface, but leaving the programmatic interface the same.
-  (interactive (list (not prefix-arg)))
+  (interactive (list (not current-prefix-arg)))
   (if revert-buffer-function
       (funcall revert-buffer-function ignore-auto noconfirm)
     (let* ((opoint (point))
@@ -1904,21 +1977,20 @@ beginning and `after-revert-hook' at the end."
                 (insert-file-contents file-name (not auto-save-p)
                                       nil nil t)))
             (goto-char (min opoint (point-max)))
-            (after-find-file nil nil t)
+            ;; Recompute the truename in case changes in symlinks
+            ;; have changed the truename.
+            (setq buffer-file-truename
+                  (abbreviate-file-name (file-truename buffer-file-name)))
+            (after-find-file nil nil t t)
             (run-hooks 'after-revert-hook)
             t)))))
 
 (defun recover-file (file)
   "Visit file FILE, but get contents from its last auto-save file."
-  (interactive
-   (let ((prompt-file buffer-file-name)
-        (file-name nil)
-        (file-dir nil))
-     (and prompt-file
-         (setq file-name (file-name-nondirectory prompt-file)
-               file-dir (file-name-directory prompt-file)))
-     (list (read-file-name "Recover file: "
-                              file-dir nil nil file-name))))
+  ;; Actually putting the file name in the minibuffer should be used
+  ;; only rarely.
+  ;; Not just because users often use the default.
+  (interactive "fRecover file: ")
   (setq file (expand-file-name file))
   (if (auto-save-file-name-p (file-name-nondirectory file))
       (error "%s is an auto-save file" file))
@@ -2159,7 +2231,10 @@ If WILDCARD, it also runs the shell specified by `shell-file-name'."
 
 (defvar kill-emacs-query-functions nil
   "Functions to call with no arguments to query about killing Emacs.
-If any of these functions returns nil, killing Emacs is cancelled.")
+If any of these functions returns nil, killing Emacs is cancelled.
+`save-buffers-kill-emacs' (\\[save-buffers-kill-emacs]) calls these functions,
+but `kill-emacs', the low level primitive, does not.
+See also `kill-emacs-hook'.")
 
 (defun save-buffers-kill-emacs (&optional arg)
   "Offer to save each buffer, then kill this Emacs process.
@@ -2185,12 +2260,7 @@ With prefix arg, silently save all file-visiting buffers, then kill."
             (or (not active)
                 (yes-or-no-p "Active processes exist; kill them and exit anyway? "))))
        ;; Query the user for other things, perhaps.
-       (let ((functions kill-emacs-query-functions)
-            (yes t))
-        (while (and functions yes)
-          (setq yes (and yes (funcall (car functions))))
-          (setq functions (cdr functions)))
-        yes)
+       (run-hook-with-args-until-failure 'kill-emacs-query-functions)
        (kill-emacs)))
 \f
 (define-key ctl-x-map "\C-f" 'find-file)