(revert-buffer): Doc fix.
[bpt/emacs.git] / lisp / files.el
index 389139b..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
 
@@ -29,7 +29,7 @@
 ;;; Code:
 
 (defconst delete-auto-save-files t
-  "*Non-nil means delete a buffer's auto-save file when the buffer is saved.")
+  "*Non-nil means delete auto-save file when a buffer is saved or killed.")
 
 (defconst directory-abbrev-alist
   nil
@@ -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)
@@ -141,9 +141,9 @@ nil means make them for files that have some already.
 (defvar dired-kept-versions 2
   "*When cleaning directory, number of versions to keep.")
 
-(defvar trim-versions-without-asking nil
-  "*If t, deletes excess backup versions silently.
-If nil, asks confirmation.  Any other value prevents any trimming.")
+(defvar delete-old-versions nil
+  "*If t, delete excess backup versions silently.
+If nil, ask confirmation.  Any other value prevents any trimming.")
 
 (defvar kept-old-versions 2
   "*Number of oldest versions to keep when a new numbered backup is made.")
@@ -244,7 +244,11 @@ and ignores this variable.")
 (defun ange-ftp-completion-hook-function (op &rest args)
   (if (memq op '(file-name-completion file-name-all-completions))
       (apply 'ange-ftp-hook-function op args)
-    (let (file-name-handler-alist)
+    (let ((inhibit-file-name-handlers
+          (cons 'ange-ftp-completion-hook-function
+                (and (eq inhibit-file-name-operation op)
+                     inhibit-file-name-handlers)))
+         (inhibit-file-name-operation op))
       (apply op args))))
 \f
 (defun pwd ()
@@ -274,9 +278,11 @@ Not actually set up until the first time you you use it.")
 
 (defun cd-absolute (dir)
   "Change current directory to given absolute file name DIR."
-  (setq dir (abbreviate-file-name (expand-file-name dir)))
+  ;; Put the name into directory syntax now,
+  ;; because otherwise expand-file-name may give some bad results.
   (if (not (eq system-type 'vax-vms))
       (setq dir (file-name-as-directory dir)))
+  (setq dir (abbreviate-file-name (expand-file-name dir)))
   (if (not (file-directory-p dir))
       (error "%s is not a directory" dir)
     (if (file-executable-p dir)
@@ -316,21 +322,27 @@ This is an interface to the function `load'."
   (interactive "sLoad library: ")
   (load library))
 
-;; OTHER is the other file to be compared.
-(defun file-local-copy (file)
+(defun file-local-copy (file &optional buffer)
   "Copy the file FILE into a temporary file on this machine.
 Returns the name of the local copy, or nil, if FILE is directly
 accessible."
-  (let ((handler (find-file-name-handler file)))
+  (let ((handler (find-file-name-handler file 'file-local-copy)))
     (if handler
        (funcall handler 'file-local-copy file)
       nil)))
 
-(defun file-truename (filename)
+(defun file-truename (filename &optional counter prev-dirs)
   "Return the truename of FILENAME, which should be absolute.
 The truename of a file name is found by chasing symbolic links
 both at the level of the file and at the level of the directories
-containing it, until no links are left at any level."
+containing it, until no links are left at any level.
+
+The arguments COUNTER and PREV-DIRS are used only in recursive calls.
+Do not specify them in other calls."
+  ;; COUNTER can be a cons cell whose car is the count of how many more links
+  ;; to chase before getting an error.
+  ;; PREV-DIRS can be a cons cell whose car is an alist
+  ;; of truenames we've just recently computed.
   (if (or (string= filename "~")
          (and (string= (substring filename 0 1) "~")
               (string-match "~[^/]*" filename)))
@@ -338,37 +350,67 @@ containing it, until no links are left at any level."
        (setq filename (expand-file-name filename))
        (if (string= filename "")
            (setq filename "/"))))
-  (let ((handler (find-file-name-handler filename)))
-    ;; For file name that has a special handler, call handler.
-    ;; This is so that ange-ftp can save time by doing a no-op.
-    (if handler
-       (funcall handler 'file-truename filename)
-      (let ((dir (file-name-directory filename))
-           target dirfile)
-       ;; Get the truename of the directory.
-       (setq dirfile (directory-file-name dir))
-       ;; If these are equal, we have the (or a) root directory.
-       (or (string= dir dirfile)
-           (setq dir (file-name-as-directory (file-truename dirfile))))
-       (if (equal ".." (file-name-nondirectory filename))
-           (directory-file-name (file-name-directory (directory-file-name dir)))
-         (if (equal "." (file-name-nondirectory filename))
-             (directory-file-name dir)
-           ;; Put it back on the file name.
-           (setq filename (concat dir (file-name-nondirectory filename)))
-           ;; Is the file name the name of a link?
-           (setq target (file-symlink-p filename))
-           (if target
-               ;; Yes => chase that link, then start all over
-               ;; since the link may point to a directory name that uses links.
-               ;; We can't safely use expand-file-name here
-               ;; since target might look like foo/../bar where foo
-               ;; is itself a link.  Instead, we handle . and .. above.
-               (if (file-name-absolute-p target)
-                   (file-truename target)
-                 (file-truename (concat dir target)))
-             ;; No, we are done!
-             filename)))))))
+  (or counter (setq counter (list 100)))
+  (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)
+      (setcar counter (1- (car counter)))
+      (if (< (car counter) 0)
+         (error "Apparent cycle of symbolic links for %s" filename))
+      (let ((handler (find-file-name-handler filename 'file-truename)))
+       ;; For file name that has a special handler, call handler.
+       ;; This is so that ange-ftp can save time by doing a no-op.
+       (if handler
+           (setq filename (funcall handler 'file-truename filename)
+                 done t)
+         (let ((dir (or (file-name-directory filename) default-directory))
+               target dirfile)
+           ;; Get the truename of the directory.
+           (setq dirfile (directory-file-name dir))
+           ;; If these are equal, we have the (or a) root directory.
+           (or (string= dir dirfile)
+               ;; If this is the same dir we last got the truename for,
+               ;; save time--don't recalculate.
+               (if (assoc dir (car prev-dirs))
+                   (setq dir (cdr (assoc dir (car prev-dirs))))
+                 (let ((old dir)
+                       (new (file-name-as-directory (file-truename dirfile counter prev-dirs))))
+                   (setcar prev-dirs (cons (cons old new) (car prev-dirs)))
+                   (setq dir new))))
+           (if (equal ".." (file-name-nondirectory filename))
+               (setq filename
+                     (directory-file-name (file-name-directory (directory-file-name dir)))
+                     done t)
+             (if (equal "." (file-name-nondirectory filename))
+                 (setq filename (directory-file-name dir)
+                       done t)
+               ;; Put it back on the file name.
+               (setq filename (concat dir (file-name-nondirectory filename)))
+               ;; Is the file name the name of a link?
+               (setq target (file-symlink-p filename))
+               (if target
+                   ;; Yes => chase that link, then start all over
+                   ;; since the link may point to a directory name that uses links.
+                   ;; We can't safely use expand-file-name here
+                   ;; since target might look like foo/../bar where foo
+                   ;; is itself a link.  Instead, we handle . and .. above.
+                   (setq filename
+                         (if (file-name-absolute-p target)
+                             target
+                           (concat dir target))
+                         done nil)
+                 ;; No, we are done!
+                 (setq done t))))))))
+    filename))
 
 (defun file-chase-links (filename)
   "Chase links in FILENAME until a name that is not a link.
@@ -378,11 +420,15 @@ unlike `file-truename'."
     (while (setq tem (file-symlink-p newname))
       (if (= count 0)
          (error "Apparent cycle of symbolic links for %s" filename))
+      ;; In the context of a link, `//' doesn't mean what Emacs thinks.
+      (while (string-match "//+" tem)
+       (setq tem (concat (substring tem 0 (1+ (match-beginning 0)))
+                         (substring tem (match-end 0)))))
       ;; Handle `..' by hand, since it needs to work in the
       ;; target of any directory symlink.
       ;; This code is not quite complete; it does not handle
       ;; embedded .. in some cases such as ./../foo and foo/bar/../../../lose.
-      (while (string-match "\\.\\./" tem)
+      (while (string-match "\\`\\.\\./" tem)
        (setq tem (substring tem 3))
        (setq newname (file-name-as-directory
                       ;; Do the .. by hand.
@@ -406,7 +452,8 @@ unlike `file-truename'."
   "Switch to buffer BUFFER in another frame."
   (interactive "BSwitch to buffer in other frame: ")
   (let ((pop-up-frames t))
-    (pop-to-buffer buffer t)))
+    (pop-to-buffer buffer t)
+    (raise-frame (window-frame (selected-window)))))
 
 (defun find-file (filename)
   "Edit file FILENAME.
@@ -469,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))))
@@ -549,19 +596,13 @@ Type \\[describe-variable] directory-abbrev-alist RET for more information."
     (if (and (string-match abbreviated-home-dir filename)
             ;; If the home dir is just /, don't change it.
             (not (and (= (match-end 0) 1)
-                      (= (aref filename 0) ?/))))
+                      (= (aref filename 0) ?/)))
+            (not (and (or (eq system-type 'ms-dos) 
+                          (eq system-type 'windows-nt))
+                      (save-match-data
+                        (string-match "^[a-zA-Z]:/$" filename)))))
        (setq filename
              (concat "~"
-                     ;; If abbreviated-home-dir ends with a slash,
-                     ;; don't remove the corresponding slash from
-                     ;; filename.  On MS-DOS and OS/2, you can have
-                     ;; home directories like "g:/", in which it is
-                     ;; important not to remove the slash.  And what
-                     ;; about poor root on Unix systems?
-                     (if (eq ?/ (aref abbreviated-home-dir
-                                      (1- (length abbreviated-home-dir))))
-                         "/"
-                       "")
                      (substring filename (match-beginning 1) (match-end 1))
                      (substring filename (match-end 0)))))
     filename))
@@ -628,8 +669,10 @@ The buffer is not selected, just returned to the caller."
       ;; Let user know if there is a buffer with the same truename.
       (if other
          (progn
-           (or nowarn (message "%s and %s are the same file"
-                               filename (buffer-file-name other)))
+           (or nowarn
+               (string-equal filename (buffer-file-name other))
+               (message "%s and %s are the same file"
+                        filename (buffer-file-name other)))
            ;; Optionally also find that buffer.
            (if (or find-file-existing-other-name find-file-visit-truename)
                (setq buf other))))
@@ -642,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)
@@ -656,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)
@@ -698,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
@@ -740,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.
@@ -769,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)
@@ -784,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
@@ -804,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)
@@ -819,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)
@@ -837,19 +889,21 @@ 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)
-    ("scope" . tcl-mode)
     ("wish" . tcl-mode)
-    ("shell" . tcl-mode)
-    ("form" . tcl-mode)
+    ("wishx" . tcl-mode)
     ("tcl" . tcl-mode)
+    ("tclsh" . tcl-mode)
     ("awk" . awk-mode)
     ("gawk" . awk-mode)
     ("scm" . scheme-mode))
@@ -869,10 +923,13 @@ If it matches, mode MODE is selected.")
 
 (defun set-auto-mode ()
   "Select major mode appropriate for current buffer.
-This checks for a -*- mode tag in the buffer's text, or
-compares the filename against the entries in `auto-mode-alist'.  It does
-not check for the \"mode:\" local variable in the Local Variables
-section of the file; for that, use `hack-local-variables'.
+This checks for a -*- mode tag in the buffer's text,
+compares the filename against the entries in `auto-mode-alist',
+or checks the interpreter that runs this file against
+`interpreter-mode-alist'.
+
+It does not check for the `mode:' local variable in the
+Local Variables section of the file; for that, use `hack-local-variables'.
 
 If `enable-local-variables' is nil, this function does not check for a
 -*- mode tag."
@@ -884,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
@@ -915,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)
@@ -929,7 +989,6 @@ If `enable-local-variables' is nil, this function does not check for a
       ;; If we didn't find a mode from a -*- line, try using the file name.
       (if (and (not done) buffer-file-name)
          (let ((name buffer-file-name)
-               (case-fold-search (eq system-type 'vax-vms))
                (keep-going t))
            ;; Remove backup-suffixes from file name.
            (setq name (file-name-sans-versions name))
@@ -938,17 +997,18 @@ If `enable-local-variables' is nil, this function does not check for a
              (let ((alist auto-mode-alist)
                    (mode nil))
                ;; Find first matching alist entry.
-               (while (and (not mode) alist)
-                 (if (string-match (car (car alist)) name)
-                     (if (and (consp (cdr (car alist)))
-                              (nth 2 (car alist)))
-                         (progn
-                           (setq mode (car (cdr (car alist)))
-                                 name (substring name 0 (match-beginning 0))
-                                 keep-going t))
-                       (setq mode (cdr (car alist))
-                             keep-going nil)))
-                 (setq alist (cdr alist)))
+               (let ((case-fold-search (eq system-type 'vax-vms)))
+                 (while (and (not mode) alist)
+                   (if (string-match (car (car alist)) name)
+                       (if (and (consp (cdr (car alist)))
+                                (nth 2 (car alist)))
+                           (progn
+                             (setq mode (car (cdr (car alist)))
+                                   name (substring name 0 (match-beginning 0))
+                                   keep-going t))
+                         (setq mode (cdr (car alist))
+                               keep-going nil)))
+                   (setq alist (cdr alist))))
                (if mode
                    (funcall mode)
                  ;; If we can't deduce a mode from the file name,
@@ -956,11 +1016,9 @@ If `enable-local-variables' is nil, this function does not check for a
                  (let ((interpreter
                         (save-excursion
                           (goto-char (point-min))
-                          (if (looking-at "#! *")
-                              (progn
-                                (goto-char (match-end 0))
-                                (buffer-substring (point)
-                                                  (progn (end-of-line) (point))))
+                          (if (looking-at "#! *\\([^ \t\n]+\\)")
+                              (buffer-substring (match-beginning 1)
+                                                (match-end 1))
                             "")))
                        elt)
                    ;; Map interpreter name to a mode.
@@ -1026,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)
@@ -1096,18 +1159,38 @@ 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)
   "Variables to be ignored in a file's local variable spec.")
 
 ;; 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)
 (put 'after-load-alist 'risky-local-variable t)
-           
+(put 'buffer-file-name 'risky-local-variable t)
+(put 'buffer-auto-save-file-name 'risky-local-variable t)
+(put 'buffer-file-truename 'risky-local-variable t)
+(put 'exec-path 'risky-local-variable t)
+(put 'load-path 'risky-local-variable t)
+(put 'exec-directory 'risky-local-variable t)
+(put 'process-environment 'risky-local-variable t)
+;; Don't wait for outline.el to be loaded, for the sake of outline-minor-mode.
+(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))))
+
 ;; "Set" one variable in a local variables spec.
 ;; A few variable names are treated specially.
 (defun hack-one-local-variable (var val)
@@ -1119,19 +1202,35 @@ 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?$"
-                          (symbol-name var)))
-        (if (and (not (string= (user-login-name) "root"))
-                 (or (eq enable-local-eval t)
-                     (and enable-local-eval
-                          (save-window-excursion
-                            (switch-to-buffer (current-buffer))
-                            (save-excursion
-                              (beginning-of-line)
-                              (set-window-start (selected-window) (point)))
-                            (setq enable-local-eval
-                                  (y-or-n-p (format "Process `eval' or hook local variables in file %s? "
-                                                    (file-name-nondirectory buffer-file-name))))))))
+            (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)
+                     (consp val)
+                     (eq (car val) 'put)
+                     (hack-one-local-variable-quotep (nth 1 val))
+                     (hack-one-local-variable-quotep (nth 2 val))
+                     ;; Only allow safe values of lisp-indent-hook;
+                     ;; not functions.
+                     (or (numberp (nth 3 val))
+                         (equal (nth 3 val) ''defun))
+                     (memq (nth 1 (nth 2 val))
+                           '(lisp-indent-hook)))
+                ;; Permit eval if not root and user says ok.
+                (and (not (zerop (user-uid)))
+                     (or (eq enable-local-eval t)
+                         (and enable-local-eval
+                              (save-window-excursion
+                                (switch-to-buffer (current-buffer))
+                                (save-excursion
+                                  (beginning-of-line)
+                                  (set-window-start (selected-window) (point)))
+                                (setq enable-local-eval
+                                      (y-or-n-p (format "Process `eval' or hook local variables in file %s? "
+                                                        (file-name-nondirectory buffer-file-name)))))))))
             (if (eq var 'eval)
                 (save-excursion (eval val))
               (make-local-variable var)
@@ -1149,35 +1248,43 @@ 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 filename
-      (setq filename
-           (if (string-equal filename "")
-               nil
-             (expand-file-name filename))))
-  (or (equal filename buffer-file-name)
-      (progn
-       (and filename (lock-buffer filename))
-       (unlock-buffer)))
-  (setq buffer-file-name filename)
-  (if filename                         ; make buffer name reflect filename.
-      (let ((new-name (file-name-nondirectory buffer-file-name)))
-       (if (string= new-name "")
-           (error "Empty file name"))
-       (if (eq system-type 'vax-vms)
-           (setq new-name (downcase new-name)))
-       (setq default-directory (file-name-directory buffer-file-name))
-       (or (string= new-name (buffer-name))
-           (rename-buffer new-name t))))
-  (setq buffer-backed-up nil)
-  (clear-visited-file-modtime)
-  (if filename
-      (progn
-       (setq buffer-file-truename
-             (abbreviate-file-name (file-truename buffer-file-name)))
-       (if find-file-visit-truename
-           (setq buffer-file-name buffer-file-truename))
-       (setq buffer-file-number (nth 10 (file-attributes buffer-file-name))))
-    (setq buffer-file-truename nil buffer-file-number nil))
+  (if (buffer-base-buffer)
+      (error "An indirect buffer cannot visit a file"))
+  (let (truename)
+    (if filename
+       (setq filename
+             (if (string-equal filename "")
+                 nil
+               (expand-file-name filename))))
+    (if filename
+       (progn
+         (setq truename (file-truename filename))
+         (if find-file-visit-truename
+             ;; Do not use the abbreviated filename, because
+             ;; write-region will reset it to the expanded filename
+             (setq filename truename))))
+    (or (equal filename buffer-file-name)
+       (progn
+         (and filename (lock-buffer filename))
+         (unlock-buffer)))
+    (setq buffer-file-name filename)
+    (if filename                       ; make buffer name reflect filename.
+       (let ((new-name (file-name-nondirectory buffer-file-name)))
+         (if (string= new-name "")
+             (error "Empty file name"))
+         (if (eq system-type 'vax-vms)
+             (setq new-name (downcase new-name)))
+         (setq default-directory (file-name-directory buffer-file-name))
+         (or (string= new-name (buffer-name))
+             (rename-buffer new-name t))))
+    (setq buffer-backed-up nil)
+    (clear-visited-file-modtime)
+    (if truename
+       (setq buffer-file-truename (abbreviate-file-name truename)))
+    (setq buffer-file-number
+         (if filename
+             (nth 10 (file-attributes buffer-file-name))
+             nil)))
   ;; write-file-hooks is normally used for things like ftp-find-file
   ;; that visit things that are not local files as if they were files.
   ;; Changing to visit an ordinary local file instead should flush the hook.
@@ -1262,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 trim-versions-without-asking t) (eq trim-versions-without-asking nil))
-                       (or trim-versions-without-asking
-                           (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.
@@ -1326,7 +1434,7 @@ This is a separate procedure so your site-init or startup file can
 redefine it.
 If the optional argument KEEP-BACKUP-VERSION is non-nil,
 we do not remove backup version numbers, only true file version numbers."
-  (let ((handler (find-file-name-handler name)))
+  (let ((handler (find-file-name-handler name 'file-name-sans-versions)))
     (if handler
        (funcall handler 'file-name-sans-versions name keep-backup-version)
       (substring name 0
@@ -1347,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."
@@ -1384,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."
@@ -1458,12 +1591,13 @@ We don't want excessive versions piling up, so there are variables
  and `kept-new-versions', which tells how many newest versions to keep.
  Defaults are 2 old versions and 2 new.
 `dired-kept-versions' controls dired's clean-directory (.) command.
-If `trim-versions-without-asking' is nil, system will query user
+If `delete-old-versions' is nil, system will query user
  before trimming versions.  Otherwise it does it silently."
   (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)
@@ -1485,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
@@ -1565,43 +1700,50 @@ the last real save, but optional arg FORCE non-nil means delete anyway."
                (error "Attempt to save to a file which you aren't allowed to write"))))))
     (or buffer-backed-up
        (setq setmodes (backup-buffer)))
-    (if file-precious-flag
-       ;; If file is precious, write temp name, then rename it.
-       (let ((dir (file-name-directory buffer-file-name))
-             (realname buffer-file-name)
-             tempname temp nogood i succeed)
-         (setq i 0)
-         (setq nogood t)
-         ;; Find the temporary name to write under.
-         (while nogood
-           (setq tempname (format "%s#tmp#%d" dir i))
-           (setq nogood (file-exists-p tempname))
-           (setq i (1+ i)))
-         (unwind-protect
-             (progn (clear-visited-file-modtime)
-                    (write-region (point-min) (point-max)
-                                  tempname nil realname)
-                    (setq succeed t))
-           ;; If writing the temp file fails,
-           ;; delete the temp file.
-           (or succeed (delete-file tempname)))
-         ;; Since we have created an entirely new file
-         ;; and renamed it, make sure it gets the
-         ;; right permission bits set.
-         (setq setmodes (file-modes buffer-file-name))
-         ;; We succeeded in writing the temp file,
-         ;; so rename it.
-         (rename-file tempname buffer-file-name t))
-      ;; If file not writable, see if we can make it writable
-      ;; temporarily while we write it.
-      ;; But no need to do so if we have just backed it up
-      ;; (setmodes is set) because that says we're superseding.
-      (cond ((and tempsetmodes (not setmodes))
-            ;; Change the mode back, after writing.
-            (setq setmodes (file-modes buffer-file-name))
-            (set-file-modes buffer-file-name 511)))
-      (write-region (point-min) (point-max)
-                   buffer-file-name nil t))
+    (let ((dir (file-name-directory buffer-file-name))) 
+      (if (and file-precious-flag
+              (file-writable-p dir))
+         ;; If file is precious, write temp name, then rename it.
+         ;; 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
+               (old-modtime (visited-file-modtime)))
+           (setq i 0)
+           (setq nogood t)
+           ;; Find the temporary name to write under.
+           (while nogood
+             (setq tempname (format "%s#tmp#%d" dir i))
+             (setq nogood (file-exists-p tempname))
+             (setq i (1+ i)))
+           (unwind-protect
+               (progn (clear-visited-file-modtime)
+                      (write-region (point-min) (point-max)
+                                    tempname nil realname)
+                      (setq succeed t))
+             ;; If writing the temp file fails,
+             ;; delete the temp file.
+             (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.
+           (setq setmodes (file-modes buffer-file-name))
+           ;; We succeeded in writing the temp file,
+           ;; so rename it.
+           (rename-file tempname buffer-file-name t))
+       ;; If file not writable, see if we can make it writable
+       ;; temporarily while we write it.
+       ;; But no need to do so if we have just backed it up
+       ;; (setmodes is set) because that says we're superseding.
+       (cond ((and tempsetmodes (not setmodes))
+              ;; Change the mode back, after writing.
+              (setq setmodes (file-modes buffer-file-name))
+              (set-file-modes buffer-file-name 511)))
+       (write-region (point-min) (point-max)
+                     buffer-file-name nil t)))
     setmodes))
 
 (defun save-some-buffers (&optional arg exiting)
@@ -1616,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
@@ -1658,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"))
@@ -1735,7 +1881,7 @@ to create parent directories if they don't exist."
    (list (read-file-name "Make directory: " default-directory default-directory
                         nil nil)
         t))
-  (let ((handler (find-file-name-handler dir)))
+  (let ((handler (find-file-name-handler dir 'make-directory)))
     (if handler
        (funcall handler 'make-directory dir parents)
       (if (not parents)
@@ -1751,7 +1897,9 @@ to create parent directories if they don't exist."
 \f
 (put 'revert-buffer-function 'permanent-local t)
 (defvar revert-buffer-function nil
-  "Function to use to revert this buffer, or nil to do the default.")
+  "Function to use to revert this buffer, or nil to do the default.
+The function receives two arguments IGNORE-AUTO and NOCONFIRM,
+which are the arguments that `revert-buffer' received.")
 
 (put 'revert-buffer-insert-file-contents-function 'permanent-local t)
 (defvar revert-buffer-insert-file-contents-function nil
@@ -1765,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
@@ -1785,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))
@@ -1825,24 +1973,24 @@ beginning and `after-revert-hook' at the end."
                 (let ((buffer-file-name nil))
                   (or auto-save-p
                       (unlock-buffer)))
+                (widen)
                 (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))
@@ -1888,7 +2036,10 @@ With prefix argument ARG, turn auto-saving on if positive, else off."
   (interactive "P")
   (setq buffer-auto-save-file-name
         (and (if (null arg)
-                (not buffer-auto-save-file-name)
+                (or (not buffer-auto-save-file-name)
+                    ;; If autosave is off because buffer has shrunk,
+                    ;; then toggling should turn it on.
+                    (< buffer-saved-size 0))
               (or (eq arg t) (listp arg) (and (integerp arg) (> arg 0))))
             (if (and buffer-file-name auto-save-visited-file-name
                      (not buffer-read-only))
@@ -1926,8 +2077,28 @@ See also `auto-save-file-name-p'."
              "#"
              (file-name-nondirectory buffer-file-name)
              "#")
-    ;; For non-file bfr, use bfr name and Emacs pid.
-    (expand-file-name (format "#%s#%s#" (buffer-name) (make-temp-name "")))))
+
+    ;; Deal with buffers that don't have any associated files.  (Mail
+    ;; mode tends to create a good number of these.)
+
+    (let ((buffer-name (buffer-name))
+         (limit 0))
+      ;; Use technique from Sebastian Kremer's auto-save
+      ;; package to turn slashes into \\!.  This ensures that
+      ;; the auto-save buffer name is unique.
+
+      (while (string-match "[/\\]" buffer-name limit)
+       (setq buffer-name (concat (substring buffer-name 0 (match-beginning 0))
+                       (if (string= (substring buffer-name
+                                               (match-beginning 0)
+                                               (match-end 0))
+                                    "/")
+                           "\\!"
+                         "\\\\")
+                       (substring buffer-name (match-end 0))))
+       (setq limit (1+ (match-end 0))))
+
+      (expand-file-name (format "#%s#%s#" buffer-name (make-temp-name ""))))))
 
 (defun auto-save-file-name-p (filename)
   "Return non-nil if FILENAME can be yielded by `make-auto-save-file-name'.
@@ -1993,6 +2164,7 @@ and `list-directory-verbose-switches'."
 (defun insert-directory (file switches &optional wildcard full-directory-p)
   "Insert directory listing for FILE, formatted according to SWITCHES.
 Leaves point after the inserted text.
+SWITCHES may be a string of options, or a list of strings.
 Optional third arg WILDCARD means treat FILE as shell wildcard.
 Optional fourth arg FULL-DIRECTORY-P means file is a directory and
 switches do not contain `d', so that a full listing is expected.
@@ -2000,7 +2172,9 @@ switches do not contain `d', so that a full listing is expected.
 This works by running a directory listing program
 whose name is in the variable `insert-directory-program'.
 If WILDCARD, it also runs the shell specified by `shell-file-name'."
-  (let ((handler (find-file-name-handler file)))
+  ;; We need the directory in order to find the right handler.
+  (let ((handler (find-file-name-handler (expand-file-name file)
+                                        'insert-directory)))
     (if handler
        (funcall handler 'insert-directory file switches
                 wildcard full-directory-p)
@@ -2019,7 +2193,7 @@ If WILDCARD, it also runs the shell specified by `shell-file-name'."
              ;; We also currently don't quote the quoting characters
              ;; in case people want to use them explicitly to quote
              ;; wildcard characters.
-             (while (string-match "[ \t\n;<>&|{}()#$]" pattern beg)
+             (while (string-match "[ \t\n;<>&|()#$]" pattern beg)
                (setq pattern
                      (concat (substring pattern 0 (match-beginning 0))
                              "\\"
@@ -2027,17 +2201,40 @@ If WILDCARD, it also runs the shell specified by `shell-file-name'."
                      beg (1+ (match-end 0))))
              (call-process shell-file-name nil t nil
                            "-c" (concat insert-directory-program
-                                        " -d " switches " "
+                                        " -d "
+                                        (if (stringp switches)
+                                            switches
+                                          (mapconcat 'identity switches " "))
+                                        " "
                                         pattern)))
          ;; SunOS 4.1.3, SVr4 and others need the "." to list the
          ;; directory if FILE is a symbolic link.
-         (call-process insert-directory-program nil t nil switches
-                       (if full-directory-p
-                           (concat (file-name-as-directory file) ".")
-                         file)))))))
+         (apply 'call-process
+                insert-directory-program nil t nil
+                (let (list)
+                  (if (listp switches)
+                      (setq list switches)
+                    (if (not (equal switches ""))
+                        (progn
+                          ;; Split the switches at any spaces
+                          ;; so we can pass separate options as separate args.
+                          (while (string-match " " switches)
+                            (setq list (cons (substring switches 0 (match-beginning 0))
+                                             list)
+                                  switches (substring switches (match-end 0))))
+                          (setq list (cons switches list)))))
+                  (append list
+                          (list
+                           (if full-directory-p
+                               (concat (file-name-as-directory file) ".")
+                             file))))))))))
 
 (defvar kill-emacs-query-functions nil
-  "Functions to call with no arguments to query about killing Emacs.")
+  "Functions to call with no arguments to query about killing Emacs.
+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.
@@ -2063,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)