(fill-region-as-paragraph): Refine last change.
[bpt/emacs.git] / lisp / ediff-ptch.el
index dc63304..b911c33 100644 (file)
@@ -1,8 +1,9 @@
 ;;; ediff-ptch.el --- Ediff's  patch support
 
-;; Copyright (C) 1996, 1997 Free Software Foundation, Inc.
+;; Copyright (C) 1996, 1997, 1998, 1999, 2000, 2001, 2002,
+;;   2003, 2004, 2005, 2006 Free Software Foundation, Inc.
 
-;; Author: Michael Kifer <kifer@cs.sunysb.edu>
+;; Author: Michael Kifer <kifer@cs.stonybrook.edu>
 
 ;; This file is part of GNU Emacs.
 
 
 ;; You should have received a copy of the GNU General Public License
 ;; along with GNU Emacs; see the file COPYING.  If not, write to the
-;; Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-;; Boston, MA 02111-1307, USA.
+;; Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+;; Boston, MA 02110-1301, USA.
 
+;;; Commentary:
 
 ;;; Code:
-        
-(provide 'ediff-ptch)
+
 
 (defgroup ediff-ptch nil
-  "Ediff patch support"
+  "Ediff patch support."
   :tag "Patch"
   :prefix "ediff-"
   :group 'ediff)
@@ -43,6 +44,8 @@
   (let ((load-path (cons (expand-file-name ".") load-path)))
     (or (featurep 'ediff-init)
        (load "ediff-init.el" nil nil 'nosuffix))
+    (or (featurep 'ediff-mult)
+       (load "ediff-mult.el" nil nil 'nosuffix))
     (or (featurep 'ediff)
        (load "ediff.el" nil nil 'nosuffix))
     ))
@@ -61,7 +64,7 @@ It is recommended to use GNU-compatible versions."
 Note: the `-b' option should be specified in `ediff-backup-specs'.
 
 It is recommended to pass the `-f' option to the patch program, so it won't ask
-questions. However, some implementations don't accept this option, in which
+questions.  However, some implementations don't accept this option, in which
 case the default value for this variable should be changed."
   :type 'string
   :group 'ediff-ptch)
@@ -73,7 +76,7 @@ case the default value for this variable should be changed."
 (defconst ediff-default-backup-extension
   (if (memq system-type '(vax-vms axp-vms emx ms-dos))
       "_orig" ".orig"))
-  
+
 
 (defcustom ediff-backup-extension ediff-default-backup-extension
   "Backup extension used by the patch program.
@@ -82,14 +85,16 @@ See also `ediff-backup-specs'."
   :group 'ediff-ptch)
 
 (defun ediff-test-patch-utility ()
-  (cond ((zerop (call-process ediff-patch-program nil nil nil "-z." "-b"))
-        ;; GNU `patch' v. >= 2.2
-        'gnu)
-       ((zerop (call-process ediff-patch-program nil nil nil "-b"))
-        'posix)
-       (t 'traditional)))
-
-(defcustom ediff-backup-specs 
+  (condition-case nil
+      (cond ((eq 0 (call-process ediff-patch-program nil nil nil "-z." "-b"))
+            ;; GNU `patch' v. >= 2.2
+            'gnu)
+           ((eq 0 (call-process ediff-patch-program nil nil nil "-b"))
+            'posix)
+           (t 'traditional))
+    (file-error nil)))
+
+(defcustom ediff-backup-specs
   (let ((type (ediff-test-patch-utility)))
     (cond ((eq type 'gnu)
           ;; GNU `patch' v. >= 2.2
@@ -103,23 +108,23 @@ See also `ediff-backup-specs'."
           (format "-b %s" ediff-backup-extension))))
   "*Backup directives to pass to the patch program.
 Ediff requires that the old version of the file \(before applying the patch\)
-be saved in a file named `the-patch-file.extension'. Usually `extension' is
+be saved in a file named `the-patch-file.extension'.  Usually `extension' is
 `.orig', but this can be changed by the user and may depend on the system.
 Therefore, Ediff needs to know the backup extension used by the patch program.
 
 Some versions of the patch program let you specify `-b backup-extension'.
 Other versions only permit `-b', which assumes the extension `.orig'
-\(in which case ediff-backup-extension MUST be also `.orig'\). The latest
+\(in which case ediff-backup-extension MUST be also `.orig'\).  The latest
 versions of GNU patch require `-b -z backup-extension'.
 
 Note that both `ediff-backup-extension' and `ediff-backup-specs'
-must be set properly. If your patch program takes the option `-b',
+must be set properly.  If your patch program takes the option `-b',
 but not `-b extension', the variable `ediff-backup-extension' must
 still be set so Ediff will know which extension to use.
 
-Ediff tries to guess the appropriate value for this variables. It is believed
+Ediff tries to guess the appropriate value for this variables.  It is believed
 to be working for `traditional' patch, all versions of GNU patch, and for POSIX
-patch. So, don't change these variables, unless the default doesn't work."
+patch.  So, don't change these variables, unless the default doesn't work."
   :type 'string
   :group 'ediff-ptch)
 
@@ -141,13 +146,13 @@ program."
   :type 'regexp
   :group 'ediff-ptch)
 
-;; The buffer of the patch file. Local to control buffer.
+;; The buffer of the patch file.  Local to control buffer.
 (ediff-defvar-local ediff-patchbufer nil "")
 
 ;; The buffer where patch displays its diagnostics.
 (ediff-defvar-local ediff-patch-diagnostics nil "")
 
-;; Map of patch buffer. Has the form:
+;; Map of patch buffer.  Has the form:
 ;;    ((filename1 marker1 marker2) (filename2 marker1 marker2) ...)
 ;; where filenames are files to which patch would have applied the patch;
 ;; marker1 delimits the beginning of the corresponding patch and marker2 does
@@ -157,10 +162,16 @@ program."
 ;; strip prefix from filename
 ;; returns /dev/null, if can't strip prefix
 (defsubst ediff-file-name-sans-prefix (filename prefix)
-  (save-match-data
-    (if (string-match (concat "^" prefix) filename)
-       (substring filename (match-end 0))
-      (concat "/null/" filename))))
+  (if prefix
+      (save-match-data
+       (if (string-match (concat "^" (if (stringp prefix)
+                                         (regexp-quote prefix)
+                                       ""))
+                         filename)
+           (substring filename (match-end 0))
+         (concat "/null/" filename)))
+    filename)
+  )
 
 
 
@@ -179,12 +190,22 @@ program."
            (setq count (1+ count)))))
       count)))
 
-;; Scan BUF (which is supposed to contain a patch) and make a list of the form 
-;;    ((filename1 marker1 marker2) (filename2 marker1 marker2) ...)
-;; where filenames are files to which patch would have applied the patch;
-;; marker1 delimits the beginning of the corresponding patch and marker2 does
-;; it for the end. This list is then assigned to ediff-patch-map.
-;; Returns the number of elements in the list ediff-patch-map
+;; Scan BUF (which is supposed to contain a patch) and make a list of the form
+;;    ((nil nil filename-spec1 marker1 marker2)
+;;          (nil nil filename-spec2 marker1 marker2) ...)
+;; where filename-spec[12] are files to which the `patch' program would
+;; have applied the patch.
+;; nin, nil are placeholders. See ediff-make-new-meta-list-element in
+;;    ediff-meta.el for the explanations.
+;; In the beginning we don't know exactly which files need to be patched.
+;; We usually come up with two candidates and ediff-file-name-sans-prefix
+;;    resolves this later.
+;;
+;; The marker `marker1' delimits the beginning of the corresponding patch and
+;;    `marker2' does it for the end.
+;; The result of ediff-map-patch-buffer is a list, which is then assigned
+;; to ediff-patch-map.
+;; The function returns the number of elements in the list ediff-patch-map
 (defun ediff-map-patch-buffer (buf)
   (ediff-with-current-buffer buf
     (let ((count 0)
@@ -209,7 +230,8 @@ program."
                  end2 (or (match-end 3) (match-end 5)))
            ;; possible-file-names is holding the new file names until we
            ;; insert the old file name in the patch map
-           ;; It is a pair (filename from 1st header line . fn from 2nd line)
+           ;; It is a pair
+           ;;     (filename-from-1st-header-line . fn from 2nd line)
            (setq possible-file-names
                  (cons (if (and beg1 end1)
                            (buffer-substring beg1 end1)
@@ -224,27 +246,35 @@ program."
                (move-marker mark2 (match-beginning 0)))
 
            (goto-char mark2-end)
-           
+
            (if filenames
-               (setq patch-map (cons (list filenames mark1 mark2) patch-map)))
+               (setq patch-map
+                     (cons (ediff-make-new-meta-list-element
+                            filenames mark1 mark2)
+                           patch-map)))
            (setq mark1 mark2
                  mark1-end mark2-end
                  filenames possible-file-names))
          (setq opoint (point)
                count (1+ count))))
       (setq mark2 (point-max-marker)
-           patch-map (cons (list possible-file-names mark1 mark2) patch-map))
+           patch-map (cons (ediff-make-new-meta-list-element
+                            possible-file-names mark1 mark2)
+                           patch-map))
       (setq ediff-patch-map (nreverse patch-map))
       count)))
 
 ;; Fix up the file names in the list using the argument FILENAME
-;; Algorithm: find the first file's directory and cut it out from each file
-;; name in the patch. Prepend the directory of FILENAME to each file in the
-;; patch. In addition, the first file in the patch is replaced by FILENAME.
-;; Each file is actually a file-pair of files found in the context diff header
-;; In the end, for each pair, we select the shortest existing file.
+;; Algorithm: find the files' directories in the patch and, if a directory is
+;; absolute, cut it out from the corresponding file name in the patch.
+;; Relative directories are not cut out.
+;; Prepend the directory of FILENAME to each resulting file (which came
+;; originally from the patch).
+;; In addition, the first file in the patch document is replaced by FILENAME.
+;; Each file is actually a pair of files found in the context diff header
+;; In the end, for each pair, we ask the user which file to patch.
 ;; Note: Ediff doesn't recognize multi-file patches that are separated
-;; with the `Index:' line. It treats them as a single-file patch.
+;; with the `Index:' line.  It treats them as a single-file patch.
 ;;
 ;; Executes inside the patch buffer
 (defun ediff-fixup-patch-map (filename)
@@ -253,96 +283,144 @@ program."
                        ;; directory part of filename
                        (file-name-as-directory filename)
                      (file-name-directory filename)))
-       ;; directory part of the first file in the patch
-       (base-dir1 (file-name-directory (car (car (car ediff-patch-map)))))
-       (base-dir2 (file-name-directory (cdr (car (car ediff-patch-map)))))
+       ;; In case 2 files are possible patch targets, the user will be offered
+       ;; to choose file1 or file2.  In a multifile patch, if the user chooses
+       ;; 1 or 2, this choice is preserved to decide future alternatives.
+       chosen-alternative
        )
 
     ;; chop off base-dirs
-    (mapcar (function (lambda (triple)
-                       (or (string= (car (car triple)) "/dev/null")
-                           (setcar (car triple)
-                                   (ediff-file-name-sans-prefix
-                                    (car (car triple)) base-dir1)))
-                       (or (string= (cdr (car triple)) "/dev/null")
-                           (setcdr (car triple)
-                                   (ediff-file-name-sans-prefix
-                                    (cdr (car triple)) base-dir2)))
-                       ))
+    (mapcar (lambda (session-info)
+             (let* ((proposed-file-names
+                     ;; Filename-spec is objA; it is represented as
+                     ;; (file1 . file2). Get it using ediff-get-session-objA.
+                     (ediff-get-session-objA-name session-info))
+                    ;; base-dir1 is  the dir part of the 1st file in the patch
+                    (base-dir1
+                     (or (file-name-directory (car proposed-file-names))
+                         ""))
+                    ;; directory part of the 2nd file in the patch
+                    (base-dir2
+                     (or (file-name-directory (cdr proposed-file-names))
+                         ""))
+                    )
+               ;; If both base-dir1 and base-dir2 are relative and exist,
+               ;; assume that
+               ;; these dirs lead to the actual files starting at the present
+               ;; directory. So, we don't strip these relative dirs from the
+               ;; file names. This is a heuristic intended to improve guessing
+               (unless (or (file-name-absolute-p base-dir1)
+                           (file-name-absolute-p base-dir2)
+                           (not (file-exists-p base-dir1))
+                           (not (file-exists-p base-dir2)))
+                 (setq base-dir1 ""
+                       base-dir2 ""))
+               (or (string= (car proposed-file-names) "/dev/null")
+                   (setcar proposed-file-names
+                           (ediff-file-name-sans-prefix
+                            (car proposed-file-names) base-dir1)))
+               (or (string=
+                    (cdr proposed-file-names) "/dev/null")
+                   (setcdr proposed-file-names
+                           (ediff-file-name-sans-prefix
+                            (cdr proposed-file-names) base-dir2)))
+               ))
            ediff-patch-map)
 
     ;; take the given file name into account
     (or (file-directory-p filename)
        (string= "/dev/null" filename)
-       (progn
-         (setcar (car ediff-patch-map)
-                 (cons (file-name-nondirectory filename)
-                       (file-name-nondirectory filename)))))
+       (setcar (ediff-get-session-objA (car ediff-patch-map))
+               (cons (file-name-nondirectory filename)
+                     (file-name-nondirectory filename))))
 
     ;; prepend actual-dir
-    (mapcar (function (lambda (triple)
-                        (if (and (string-match "^/null/" (car (car triple)))
-                                 (string-match "^/null/" (cdr (car triple))))
-                            ;; couldn't strip base-dir1 and base-dir2
-                            ;; hence, something wrong
-                            (progn
-                              (with-output-to-temp-buffer ediff-msg-buffer
-                                (princ
-                                 (format "
+    (mapcar (lambda (session-info)
+             (let ((proposed-file-names
+                    (ediff-get-session-objA-name session-info)))
+               (if (and (string-match "^/null/" (car proposed-file-names))
+                        (string-match "^/null/" (cdr proposed-file-names)))
+                   ;; couldn't intuit the file name to patch, so
+                   ;; something is amiss
+                   (progn
+                     (with-output-to-temp-buffer ediff-msg-buffer
+                       (ediff-with-current-buffer standard-output
+                         (fundamental-mode))
+                       (princ
+                        (format "
 The patch file contains a context diff for
        %s
        %s
 However, Ediff cannot infer the name of the actual file
-to be patched on your system. If you know the correct file name,
+to be patched on your system.  If you know the correct file name,
 please enter it now.
 
 If you don't know and still would like to apply patches to
 other files, enter /dev/null
 "
-                                         (substring (car (car triple)) 6)
-                                         (substring (cdr (car triple)) 6))))
-                              (let ((directory t)
-                                    user-file)
-                                (while directory
-                                  (setq user-file
-                                        (read-file-name
-                                         "Please enter file name: "
-                                         actual-dir actual-dir t))
-                                  (if (not (file-directory-p user-file))
-                                      (setq directory nil)
-                                    (setq directory t)
-                                    (beep)
-                                    (message "%s is a directory" user-file)
-                                    (sit-for 2)))
-                                (setcar triple (cons user-file user-file))))
-                          (setcar (car triple)
-                                  (expand-file-name 
-                                   (concat actual-dir (car (car triple)))))
-                          (setcdr (car triple)
-                                  (expand-file-name 
-                                   (concat actual-dir (cdr (car triple))))))
-                        ))
+                                (substring (car proposed-file-names) 6)
+                                (substring (cdr proposed-file-names) 6))))
+                     (let ((directory t)
+                           user-file)
+                       (while directory
+                         (setq user-file
+                               (read-file-name
+                                "Please enter file name: "
+                                actual-dir actual-dir t))
+                         (if (not (file-directory-p user-file))
+                             (setq directory nil)
+                           (setq directory t)
+                           (beep)
+                           (message "%s is a directory" user-file)
+                           (sit-for 2)))
+                       (setcar (ediff-get-session-objA session-info)
+                               (cons user-file user-file))))
+                 (setcar proposed-file-names
+                         (expand-file-name
+                          (concat actual-dir (car proposed-file-names))))
+                 (setcdr proposed-file-names
+                         (expand-file-name
+                          (concat actual-dir (cdr proposed-file-names)))))
+               ))
            ediff-patch-map)
-    ;; check for the shorter existing file in each pair and discard the other
-    ;; one
-    (mapcar (function (lambda (triple)
-                       (let* ((file1 (car (car triple)))
-                              (file2 (cdr (car triple)))
-                              (f1-exists (file-exists-p file1))
-                              (f2-exists (file-exists-p file2)))
-                         (cond
-                          ((and (< (length file2) (length file1))
-                                f2-exists)
-                           (setcar triple file2))
-                          ((and (< (length file1) (length file2))
-                                f1-exists)
-                           (setcar triple file1))
-                          ((and f1-exists f2-exists
-                                (string= file1 file2))
-                           (setcar triple file1))
-                          ((and f1-exists f2-exists)
-                           (with-output-to-temp-buffer ediff-msg-buffer
-                             (princ (format "
+    ;; Check for the existing files in each pair and discard the nonexisting
+    ;; ones. If both exist, ask the user.
+    (mapcar (lambda (session-info)
+             (let* ((file1 (car (ediff-get-session-objA-name session-info)))
+                    (file2 (cdr (ediff-get-session-objA-name session-info)))
+                    (session-file-object
+                     (ediff-get-session-objA session-info))
+                    (f1-exists (file-exists-p file1))
+                    (f2-exists (file-exists-p file2)))
+               (cond
+                ((and
+                  ;; The patch program prefers the shortest file as the patch
+                  ;; target. However, this is a questionable heuristic. In an
+                  ;; interactive program, like ediff, we can offer the user a
+                  ;; choice.
+                  ;; (< (length file2) (length file1))
+                  (not f1-exists)
+                  f2-exists)
+                 ;; replace file-pair with the winning file2
+                 (setcar session-file-object file2))
+                ((and
+                  ;; (< (length file1) (length file2))
+                  (not f2-exists)
+                  f1-exists)
+                 ;; replace file-pair with the winning file1
+                 (setcar session-file-object file1))
+                ((and f1-exists f2-exists
+                      (string= file1 file2))
+                 (setcar session-file-object file1))
+                ((and f1-exists f2-exists (eq chosen-alternative 1))
+                 (setcar session-file-object file1))
+                ((and f1-exists f2-exists (eq chosen-alternative 2))
+                 (setcar session-file-object file2))
+                ((and f1-exists f2-exists)
+                 (with-output-to-temp-buffer ediff-msg-buffer
+                   (ediff-with-current-buffer standard-output
+                     (fundamental-mode))
+                   (princ (format "
 Ediff has inferred that
        %s
        %s
@@ -353,39 +431,46 @@ Please advice:
     Type `y' to use %s as the target;
     Type `n' to use %s as the target.
 "
-                                            file1 file2 file2 file1)))
-                           (setcar triple
-                                   (if (y-or-n-p (format "Use %s ? " file2))
-                                       file2 file1)))
-                          (f2-exists (setcar triple file2))
-                          (f1-exists (setcar triple file1))
-                          (t
-                           (with-output-to-temp-buffer ediff-msg-buffer
-                             (princ "\nEdiff has inferred that")
-                             (if (string= file1 file2)
-                                 (princ (format "
+                                  file1 file2 file1 file2)))
+                 (setcar session-file-object
+                         (if (y-or-n-p (format "Use %s ? " file1))
+                             (progn
+                               (setq chosen-alternative 1)
+                               file1)
+                           (setq chosen-alternative 2)
+                           file2))
+                 )
+                (f2-exists (setcar session-file-object file2))
+                (f1-exists (setcar session-file-object file1))
+                (t
+                 (with-output-to-temp-buffer ediff-msg-buffer
+                   (ediff-with-current-buffer standard-output
+                     (fundamental-mode))
+                   (princ "\nEdiff has inferred that")
+                   (if (string= file1 file2)
+                       (princ (format "
        %s
-is the target for this patch. However, this file does not exist."
-                                                file1))
-                               (princ (format "
+is assumed to be the target for this patch.  However, this file does not exist."
+                                      file1))
+                     (princ (format "
        %s
        %s
-are two possible targets for this patch. However, these files do not exist."
-                                              file1 file2)))
-                             (princ "
+are two possible targets for this patch.  However, these files do not exist."
+                                    file1 file2)))
+                   (princ "
 \nPlease enter an alternative patch target ...\n"))
-                           (let ((directory t)
-                                 target)
-                             (while directory
-                               (setq target (read-file-name 
-                                             "Please enter a patch target: "
-                                             actual-dir actual-dir t))
-                               (if (not (file-directory-p target))
-                                   (setq directory nil)
-                                 (beep)
-                                 (message "%s is a directory" target)
-                                 (sit-for 2)))
-                             (setcar triple target)))))))
+                 (let ((directory t)
+                       target)
+                   (while directory
+                     (setq target (read-file-name
+                                   "Please enter a patch target: "
+                                   actual-dir actual-dir t))
+                     (if (not (file-directory-p target))
+                         (setq directory nil)
+                       (beep)
+                       (message "%s is a directory" target)
+                       (sit-for 2)))
+                   (setcar session-file-object target))))))
            ediff-patch-map)
     ))
 
@@ -397,33 +482,75 @@ are two possible targets for this patch. However, these files do not exist."
         (set-window-buffer ediff-window-B ediff-patch-diagnostics))
        (t (display-buffer ediff-patch-diagnostics 'not-this-window))))
 
-(defun ediff-get-patch-buffer ()
-  "Obtain patch buffer.  If patch is already in a buffer---use it.
-Else, read patch file into a new buffer."
-  (let ((dir (cond (ediff-patch-default-directory) ; try patch default dir
-                  (ediff-use-last-dir ediff-last-dir-patch)
+;; prompt for file, get the buffer
+(defun ediff-prompt-for-patch-file ()
+  (let ((dir (cond (ediff-use-last-dir ediff-last-dir-patch)
+                  (ediff-patch-default-directory) ; try patch default dir
                   (t default-directory)))
-       patch-buf)
-    (if (let ((last-nonmenu-event t) ; Emacs: don't use dialog box
-             last-command-event)    ; XEmacs: don't use dialog box
-         (y-or-n-p "Is the patch already in a buffer? "))
-       (setq patch-buf
-             (get-buffer
-              (read-buffer
-               "Which buffer contains the patch? "
-               (ediff-other-buffer
-                (if (eq (next-window (selected-window)) (selected-window))
-                    ;; only one window in frame --- don't skip current buff
-                    ""
-                  ;; >1 window --- skip current buff, assuming this is the one
-                  ;; to patch, not the one that has the patch
-                  (current-buffer)))
-               'must-match)))
-      (setq patch-buf
-           (find-file-noselect
-            (read-file-name "Which file contains the patch? "
-                            dir nil 'must-match))))
-    
+       (coding-system-for-read ediff-coding-system-for-read)
+       patch-file-name)
+    (setq patch-file-name
+         (read-file-name
+          (format "Patch is in file%s: "
+                  (cond ((and buffer-file-name
+                              (equal (expand-file-name dir)
+                                     (file-name-directory buffer-file-name)))
+                         (concat
+                          " (default "
+                          (file-name-nondirectory buffer-file-name)
+                          ")"))
+                        (t "")))
+          dir buffer-file-name 'must-match))
+    (if (file-directory-p patch-file-name)
+       (error "Patch file cannot be a directory: %s" patch-file-name)
+      (find-file-noselect patch-file-name))
+    ))
+
+
+;; Try current buffer, then the other window's buffer. Else, give up.
+(defun ediff-prompt-for-patch-buffer ()
+  (get-buffer
+   (read-buffer
+    "Buffer that holds the patch: "
+    (cond ((save-excursion
+            (goto-char (point-min))
+            (re-search-forward ediff-context-diff-label-regexp nil t))
+          (current-buffer))
+         ((save-window-excursion
+            (other-window 1)
+            (save-excursion
+              (goto-char (point-min))
+              (and (re-search-forward ediff-context-diff-label-regexp nil t)
+                   (current-buffer)))))
+         ((save-window-excursion
+            (other-window -1)
+            (save-excursion
+              (goto-char (point-min))
+              (and (re-search-forward ediff-context-diff-label-regexp nil t)
+                   (current-buffer)))))
+         (t (ediff-other-buffer (current-buffer))))
+    'must-match)))
+
+
+(defun ediff-get-patch-buffer (&optional arg patch-buf)
+  "Obtain patch buffer.  If patch is already in a buffer---use it.
+Else, read patch file into a new buffer. If patch buffer is passed as an
+optional argument, then use it."
+  (let ((last-nonmenu-event t) ; Emacs: don't use dialog box
+       last-command-event)    ; XEmacs: don't use dialog box
+
+    (cond ((ediff-buffer-live-p patch-buf))
+         ;; even prefix arg: patch in buffer
+         ((and (integerp arg) (eq 0 (mod arg 2)))
+          (setq patch-buf (ediff-prompt-for-patch-buffer)))
+         ;; odd prefix arg: get patch from a file
+         ((and (integerp arg) (eq 1 (mod arg 2)))
+          (setq patch-buf (ediff-prompt-for-patch-file)))
+         (t (setq patch-buf
+                  (if (y-or-n-p "Is the patch already in a buffer? ")
+                      (ediff-prompt-for-patch-buffer)
+                    (ediff-prompt-for-patch-file)))))
+
     (ediff-with-current-buffer patch-buf
       (goto-char (point-min))
       (or (ediff-get-visible-buffer-window patch-buf)
@@ -445,16 +572,22 @@ Else, read patch file into a new buffer."
     (if (< (length ediff-patch-map) 2)
        (ediff-patch-file-internal
         patch-buf
-        (if (and (not (string-match "^/dev/null" (car (car ediff-patch-map))))
-                 (> (length (car (car ediff-patch-map))) 1))
-            (car (car ediff-patch-map))
+        (if (and ediff-patch-map
+                 (not (string-match
+                       "^/dev/null"
+                       ;; this is the file to patch
+                       (ediff-get-session-objA-name (car ediff-patch-map))))
+                 (> (length
+                     (ediff-get-session-objA-name (car ediff-patch-map)))
+                    1))
+            (ediff-get-session-objA-name (car ediff-patch-map))
           filename)
         startup-hooks)
       (ediff-multi-patch-internal patch-buf startup-hooks))
     ))
 
 
-;; When patching a buffer, never change the orig file. Instead, create a new
+;; When patching a buffer, never change the orig file.  Instead, create a new
 ;; buffer, ***_patched, even if the buff visits a file.
 ;; Users who want to actually patch the buffer should use
 ;; ediff-patch-file, not ediff-patch-buffer.
@@ -469,7 +602,7 @@ Else, read patch file into a new buffer."
         default-dir file-name ctl-buf)
     (if multifile-patch-p
        (error
-        "Can't apply multi-file patches to buffers that visit no files"))
+        "To apply multi-file patches, please use `ediff-patch-file'"))
 
     ;; create a temp file to patch
     (ediff-with-current-buffer buf-to-patch
@@ -486,11 +619,11 @@ Else, read patch file into a new buffer."
       (set-visited-file-modtime) ; sync buffer and temp file
       (setq default-directory default-dir)
       )
-  
+
     ;; dispatch a patch function
     (setq ctl-buf (ediff-dispatch-file-patching-job
                   patch-buf file-name startup-hooks))
-    
+
     (ediff-with-current-buffer ctl-buf
       (delete-file (buffer-file-name ediff-buffer-A))
       (delete-file (buffer-file-name ediff-buffer-B))
@@ -503,7 +636,7 @@ Else, read patch file into a new buffer."
        (setq buffer-auto-save-file-name nil) ; don't create auto-save file
        (if default-dir (setq default-directory default-dir))
        (set-visited-file-name nil)
-       (rename-buffer (ediff-unique-buffer-name 
+       (rename-buffer (ediff-unique-buffer-name
                        (concat buf-to-patch-name "_patched") ""))
        (set-buffer-modified-p t)))
     ))
@@ -522,7 +655,7 @@ Else, read patch file into a new buffer."
 (defun ediff-patch-file-internal (patch-buf source-filename
                                            &optional startup-hooks)
   (setq source-filename (expand-file-name source-filename))
-  
+
   (let* ((shell-file-name ediff-shell)
         (patch-diagnostics (get-buffer-create "*ediff patch diagnostics*"))
         ;; ediff-find-file may use a temp file to do the patch
@@ -531,13 +664,17 @@ Else, read patch file into a new buffer."
         ;; file for the purpose of patching.
         (true-source-filename source-filename)
         (target-filename source-filename)
-        target-buf buf-to-patch file-name-magic-p 
+        ;; this ensures that the patch process gets patch buffer in the
+        ;; encoding that Emacs thinks is right for that type of text
+        (coding-system-for-write
+         (if (boundp 'buffer-file-coding-system) buffer-file-coding-system))
+        target-buf buf-to-patch file-name-magic-p
         patch-return-code ctl-buf backup-style aux-wind)
-         
+
     (if (string-match "V" ediff-patch-options)
        (error
         "Ediff doesn't take the -V option in `ediff-patch-options'--sorry"))
-                                       
+
     ;; Make a temp file, if source-filename has a magic file handler (or if
     ;; it is handled via auto-mode-alist and similar magic).
     ;; Check if there is a buffer visiting source-filename and if they are in
@@ -551,13 +688,13 @@ Else, read patch file into a new buffer."
     ;; temporary file where we put the after-product of the file handler.
     (setq file-name-magic-p (not (equal (file-truename true-source-filename)
                                        (file-truename source-filename))))
-    
-    ;; Checkout orig file, if necessary, so that the patched file 
+
+    ;; Checkout orig file, if necessary, so that the patched file
     ;; could be checked back in.
     (ediff-maybe-checkout buf-to-patch)
 
     (ediff-with-current-buffer patch-diagnostics
-      (insert-buffer patch-buf)
+      (insert-buffer-substring patch-buf)
       (message "Applying patch ... ")
       ;; fix environment for gnu patch, so it won't make numbered extensions
       (setq backup-style (getenv "VERSION_CONTROL"))
@@ -585,13 +722,15 @@ Else, read patch file into a new buffer."
 
     (switch-to-buffer patch-diagnostics)
     (sit-for 0) ; synchronize - let the user see diagnostics
-    
+
     (or (and (ediff-patch-return-code-ok patch-return-code)
             (file-exists-p
              (concat true-source-filename ediff-backup-extension)))
        (progn
          (with-output-to-temp-buffer ediff-msg-buffer
-           (princ (format 
+           (ediff-with-current-buffer standard-output
+             (fundamental-mode))
+           (princ (format
                    "Patch program has failed due to a bad patch file,
 it couldn't apply all hunks, OR
 it couldn't create the backup for the file being patched.
@@ -604,17 +743,17 @@ The second problem might be due to an incompatibility among these settings:
     ediff-backup-extension = %S             ediff-backup-specs     = %S
 
 See Ediff on-line manual for more details on these variables.
-In particular, check the documentation for `ediff-backup-specs'. 
+In particular, check the documentation for `ediff-backup-specs'.
 
 In any of the above cases, Ediff doesn't compare files automatically.
 However, if the patch was applied partially and the backup file was created,
 you can still examine the changes via M-x ediff-files"
-                          ediff-patch-program
-                          ediff-patch-program
-                          ediff-patch-options
-                          ediff-backup-extension
-                          ediff-backup-specs
-                          )))
+                   ediff-patch-program
+                   ediff-patch-program
+                   ediff-patch-options
+                   ediff-backup-extension
+                   ediff-backup-specs
+                   )))
          (beep 1)
          (if (setq aux-wind (get-buffer-window ediff-msg-buffer))
              (progn
@@ -622,11 +761,11 @@ you can still examine the changes via M-x ediff-files"
                (goto-char (point-max))))
          (switch-to-buffer-other-window patch-diagnostics)
          (error "Patch appears to have failed")))
-    
+
     ;; If black magic is involved, apply patch to a temp copy of the
-    ;; file. Otherwise, apply patch to the orig copy.  If patch is applied
+    ;; file.  Otherwise, apply patch to the orig copy.  If patch is applied
     ;; to temp copy, we name the result old-name_patched for local files
-    ;; and temp-copy_patched for remote files. The orig file name isn't
+    ;; and temp-copy_patched for remote files.  The orig file name isn't
     ;; changed, and the temp copy of the original is later deleted.
     ;; Without magic, the original file is renamed (usually into
     ;; old-name_orig) and the result of patching will have the same name as
@@ -636,7 +775,7 @@ you can still examine the changes via M-x ediff-files"
          (set-visited-file-name
           (concat source-filename ediff-backup-extension))
          (set-buffer-modified-p nil))
-      
+
       ;; Black magic in effect.
       ;; If orig file was remote, put the patched file in the temp directory.
       ;; If orig file is local, put the patched file in the directory of
@@ -647,20 +786,20 @@ you can still examine the changes via M-x ediff-files"
                 true-source-filename
               source-filename)
             "_patched"))
-      
+
       (rename-file true-source-filename target-filename t)
-      
+
       ;; arrange that the temp copy of orig will be deleted
       (rename-file (concat true-source-filename ediff-backup-extension)
                   true-source-filename t))
-    
+
     ;; make orig buffer read-only
     (setq startup-hooks
          (cons 'ediff-set-read-only-in-buf-A startup-hooks))
-    
+
     ;; set up a buf for the patched file
     (setq target-buf (find-file-noselect target-filename))
-    
+
     (setq ctl-buf
          (ediff-buffers-internal
           buf-to-patch target-buf nil
@@ -668,7 +807,7 @@ you can still examine the changes via M-x ediff-files"
     (ediff-with-current-buffer ctl-buf
       (setq ediff-patchbufer patch-buf
            ediff-patch-diagnostics patch-diagnostics))
-  
+
     (bury-buffer patch-diagnostics)
     (message "Type `P', if you need to see patch diagnostics")
     ctl-buf))
@@ -678,18 +817,22 @@ you can still examine the changes via M-x ediff-files"
     (setq startup-hooks
          ;; this sets various vars in the meta buffer inside
          ;; ediff-prepare-meta-buffer
-         (cons (` (lambda ()
-                    ;; tell what to do if the user clicks on a session record
-                    (setq ediff-session-action-function
-                          'ediff-patch-file-form-meta
-                          ediff-meta-patchbufer patch-buf)
-                    ))
+         (cons `(lambda ()
+                  ;; tell what to do if the user clicks on a session record
+                  (setq ediff-session-action-function
+                        'ediff-patch-file-form-meta
+                        ediff-meta-patchbufer patch-buf) )
                startup-hooks))
-    (setq meta-buf (ediff-prepare-meta-buffer 
+    (setq meta-buf (ediff-prepare-meta-buffer
                    'ediff-filegroup-action
                    (ediff-with-current-buffer patch-buf
-                     ;; nil replaces a regular expression
-                     (cons (list nil (format "%S" patch-buf))
+                     (cons (ediff-make-new-meta-list-header
+                            nil                     ; regexp
+                            (format "%S" patch-buf) ; obj A
+                            nil nil                 ; objects B,C
+                            nil                     ; merge-auto-store-dir
+                            nil                     ; comparison-func
+                            )
                            ediff-patch-map))
                    "*Ediff Session Group Panel"
                    'ediff-redraw-directory-group-buffer
@@ -698,8 +841,10 @@ you can still examine the changes via M-x ediff-files"
     (ediff-show-meta-buffer meta-buf)
     ))
 
-  
-      
+
+
+(provide 'ediff-ptch)
+
 
 ;;; Local Variables:
 ;;; eval: (put 'ediff-defvar-local 'lisp-indent-hook 'defun)
@@ -707,4 +852,5 @@ you can still examine the changes via M-x ediff-files"
 ;;; eval: (put 'ediff-with-current-buffer 'edebug-form-spec '(form body))
 ;;; End:
 
+;;; arch-tag: 2fe2161e-e116-469b-90fa-5cbb44c1bd1b
 ;;; ediff-ptch.el ends here