*** empty log message ***
[bpt/emacs.git] / lisp / ediff-diff.el
index 20a0d9f..e6864bf 100644 (file)
@@ -1,5 +1,6 @@
 ;;; ediff-diff.el --- diff-related utilities
-;;; Copyright (C) 1994, 1995 Free Software Foundation, Inc.
+
+;; Copyright (C) 1994, 1995, 1996, 1997 Free Software Foundation, Inc.
 
 ;; Author: Michael Kifer <kifer@cs.sunysb.edu>
 
 ;; GNU General Public License for more details.
 
 ;; 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, 675 Mass Ave, Cambridge, MA 02139, USA.
+;; 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.
+
+;;; Code:
+
+(provide 'ediff-diff)
+
+;; compiler pacifier
+(defvar ediff-default-variant)
+
+(eval-when-compile
+  (let ((load-path (cons (expand-file-name ".") load-path)))
+    (or (featurep 'ediff-init)
+       (load "ediff-init.el" nil nil 'nosuffix))
+    (or (featurep 'ediff-util)
+       (load "ediff-util.el" nil nil 'nosuffix))
+    ))
+;; end pacifier
 
 (require 'ediff-init)
 
+(defgroup ediff-diff nil
+  "Diff related utilities"
+  :prefix "ediff-"
+  :group 'ediff)
+
+
+;; The following functions needed for setting diff/diff3 options
+;; test if diff supports the --binary option
+(defsubst ediff-test-utility (diff-util option &optional files)
+  (zerop (apply 'call-process
+               (append (list diff-util nil nil nil option) files))))
+
+(defun ediff-diff-mandatory-option (diff-util)
+  (let ((file (if (boundp 'null-device) null-device "/dev/null")))
+    (cond  ((not (memq system-type '(ms-dos windows-nt windows-95)))
+           "")
+          ((and (string= diff-util ediff-diff-program)
+                (ediff-test-utility
+                 ediff-diff-program "--binary" (list file file)))
+           "--binary")
+          ((and (string= diff-util ediff-diff3-program)
+                (ediff-test-utility
+                 ediff-diff3-program "--binary" (list file file file)))
+           "--binary")
+          (t ""))))
 
-(defvar ediff-shell
+;; make sure that mandatory options are added even if the user changes
+;; ediff-diff-options or ediff-diff3-options in the customization widget
+(defun ediff-reset-diff-options (symb val)
+  (let* ((diff-program 
+         (if (eq symb 'ediff-diff-options) 
+             ediff-diff-program
+           ediff-diff3-program))
+        (mandatory-option (ediff-diff-mandatory-option diff-program))
+        (spacer (if (string-equal mandatory-option "") "" " ")))
+    (set symb 
+        (if (string-match mandatory-option val)
+            val
+          (concat mandatory-option spacer val)))
+    ))
+
+
+(defcustom ediff-shell
   (cond ((eq system-type 'emx) "cmd") ; OS/2
+       ((memq system-type '(ms-dos windows-nt windows-95))
+        shell-file-name) ; no standard name on MS-DOS
        ((memq system-type '(vax-vms axp-vms)) "*dcl*") ; VMS
        (t  "sh")) ; UNIX
   "*The shell used to run diff and patch.  If user's .profile or
 .cshrc files are set up correctly, any shell will do.  However, some people
 set $prompt or other things incorrectly, which leads to undesirable output
 messages.  These may cause Ediff to fail.  In such a case, set ediff-shell
-to a shell that you are not using or, better, fix your shell's startup file.")
+to a shell that you are not using or, better, fix your shell's startup file."
+  :type 'string
+  :group 'ediff-diff)
 
+(defcustom ediff-cmp-program "cmp"
+  "*Utility to use to determine if two files are identical.
+It must return code 0, if its arguments are identical files."
+  :type 'string
+  :group 'ediff-diff)
 
-(defvar ediff-diff-program "diff"
-  "*Program to use for generating the differential of the two files.")
-(defvar ediff-diff-options ""  
+(defcustom ediff-cmp-options nil
+  "*Options to pass to `ediff-cmp-program'.  If GNUS diff is used as
+`ediff-cmp-program', then the most useful options are `-I' RE, to
+ignore changes whose lines all match RE."
+  :type '(repeat string)
+  :group 'ediff-diff)
+
+(defcustom ediff-diff-program "diff"
+  "*Program to use for generating the differential of the two files."
+  :type 'string
+  :group 'ediff-diff)
+(defcustom ediff-diff-options ""
   "*Options to pass to `ediff-diff-program'. 
 If diff\(1\) is used as `ediff-diff-program', then the most useful options are
 `-w', to ignore space, and `-i', to ignore case of letters.
-At present, the option `-c' is ignored, since Ediff doesn't understand this
-type of output.")
+At present, the option `-c' is not allowed."
+  :set 'ediff-reset-diff-options
+  :type 'string
+  :group 'ediff-diff)
 
-(defvar ediff-custom-diff-program ediff-diff-program
+(defcustom ediff-custom-diff-program ediff-diff-program
   "*Program to use for generating custom diff output for saving it in a file.
-This output is not used by Ediff internally.")
-(defvar ediff-custom-diff-options "-c"
-  "*Options to pass to `ediff-custom-diff-program'.")
+This output is not used by Ediff internally."
+  :type 'string
+  :group 'ediff-diff)
+(defcustom ediff-custom-diff-options "-c"
+  "*Options to pass to `ediff-custom-diff-program'."
+  :type 'string
+  :group 'ediff-diff)
 
 ;;; Support for diff3
 
 (defvar ediff-match-diff3-line "^====\\(.?\\)$"
   "Pattern to match lines produced by diff3 that describe differences.")
-(defvar ediff-diff3-program "diff3"
+(defcustom ediff-diff3-program "diff3"
   "*Program to be used for three-way comparison.
-Must produce output compatible with Unix's diff3 program.")
-(defvar ediff-diff3-options ""  
-  "*Options to pass to `ediff-diff3-program'.")
-(defvar ediff-diff3-ok-lines-regexp
-  "^\\([1-3]:\\|====\\|  \\|.*Warning *:\\|.*No newline\\|.*missing newline\\)"
+Must produce output compatible with Unix's diff3 program."
+  :type 'string
+  :group 'ediff-diff)
+(defcustom ediff-diff3-options ""  
+  "*Options to pass to `ediff-diff3-program'."
+  :set 'ediff-reset-diff-options
+  :type 'string
+  :group 'ediff-diff)
+(defcustom ediff-diff3-ok-lines-regexp
+  "^\\([1-3]:\\|====\\|  \\|.*Warning *:\\|.*No newline\\|.*missing newline\\|^\C-m$\\)"
   "*Regexp that matches normal output lines from `ediff-diff3-program'.
-Lines that do not match are assumed to be error messages.")
+Lines that do not match are assumed to be error messages."
+  :type 'regexp
+  :group 'ediff-diff)
 
 ;; keeps the status of the current diff in 3-way jobs.
 ;; the status can be =diff(A), =diff(B), or =diff(A+B)
 (ediff-defvar-local ediff-diff-status "" "")
 
-;; Support for patch 
-
-(defvar ediff-patch-program "patch"
-  "*Name of the program that applies patches.")
-(defvar ediff-patch-options ""
-  "*Options to pass to ediff-patch-program.")
-
-;; The buffer of the patch file.
-(defvar ediff-patch-buf nil)
-;; The buffer where patch would display its diagnostics.
-(defvar ediff-patch-diagnostics nil)
-
   
 ;;; Fine differences 
 
-(ediff-defvar-local ediff-auto-refine (if (ediff-window-display-p) 'on 'nix)
+(ediff-defvar-local ediff-auto-refine (if (ediff-has-face-support-p) 'on 'nix)
   "If `on', Ediff auto-highlights fine diffs for the current diff region.
 If `off', auto-highlighting is not used. If `nix', no fine diffs are shown
 at all, unless the user force-refines the region by hitting `*'.
@@ -94,20 +172,30 @@ Use `setq-default' if setting it in .emacs")
 This variable can be set either in .emacs or toggled interactively.
 Use `setq-default' if setting it in .emacs")
 
-(ediff-defvar-local ediff-auto-refine-limit 700
-  "Auto-refine only those regions that are smaller than this number of bytes.")
+(ediff-defvar-local ediff-auto-refine-limit 1400
+  "*Auto-refine only the regions of this size \(in bytes\) or less.")
   
 ;;; General
 
 (defvar ediff-diff-ok-lines-regexp  
-  "^\\([0-9,]+[acd][0-9,]+$\\|[<>] \\|---\\|.*Warning *:\\|.*No newline\\|.*missing newline\\)"
+  (concat
+   "^\\("
+   "[0-9,]+[acd][0-9,]+\C-m?$"
+   "\\|[<>] "
+   "\\|---"
+   "\\|.*Warning *:"
+   "\\|.*No +newline"
+   "\\|.*missing +newline"
+   "\\|^\C-m?$"
+   "\\)")
   "Regexp that matches normal output lines from `ediff-diff-program'.
 This is mostly lifted from Emerge, except that Ediff also considers
 warnings and `Missing newline'-type messages to be normal output.
 Lines that do not match are assumed to be error messages.")
 
-(defvar ediff-match-diff-line (let ((x "\\([0-9]+\\)\\(\\|,\\([0-9]+\\)\\)"))
-                               (concat "^" x "\\([acd]\\)" x "$"))
+(defvar ediff-match-diff-line
+  (let ((x "\\([0-9]+\\)\\(\\|,\\([0-9]+\\)\\)"))
+    (concat "^" x "\\([acd]\\)" x "\C-m?$"))
   "Pattern to match lines produced by diff that describe differences.")
 
 (ediff-defvar-local ediff-setup-diff-regions-function nil
@@ -129,24 +217,62 @@ one optional arguments, diff-number to refine.")
 ;; ediff-setup-diff-regions-function, which can also have the value
 ;; ediff-setup-diff-regions3, which takes 4 arguments.
 (defun ediff-setup-diff-regions (file-A file-B file-C)
-  ;; Force all minibuffers to display ediff's messages.
-  ;; When xemacs implements minibufferless frames, this won't be necessary
-  (if ediff-xemacs-p (setq synchronize-minibuffers t))
+  (if (string-match "c" ediff-diff-options)
+      (error "Option `-c' is not allowed in `ediff-diff-options'"))
                                                  
+  ;; create, if it doesn't exist
   (or (ediff-buffer-live-p ediff-diff-buffer)
       (setq ediff-diff-buffer
            (get-buffer-create (ediff-unique-buffer-name "*ediff-diff" "*"))))
-  
-  (message "Computing differences ...")
-  (ediff-exec-process ediff-diff-program ediff-diff-buffer 'synchronize
-                     ediff-diff-options file-A file-B)
-  
+  (ediff-make-diff2-buffer ediff-diff-buffer file-A file-B)
   (ediff-prepare-error-list ediff-diff-ok-lines-regexp ediff-diff-buffer)
-  ;;(message "Computing differences ... done")
   (ediff-convert-diffs-to-overlays
    (ediff-extract-diffs
-    ediff-diff-buffer ediff-word-mode ediff-narrow-bounds))
-  )
+    ediff-diff-buffer ediff-word-mode ediff-narrow-bounds)))
+
+;; Run the diff program on FILE1 and FILE2 and put the output in DIFF-BUFFER
+;; Return the size of DIFF-BUFFER
+(defun ediff-make-diff2-buffer (diff-buffer file1 file2)
+  (let ((file1-size (ediff-file-size file1))
+       (file2-size (ediff-file-size file2)))
+    (cond ((not (numberp file1-size))
+          (message "Can't find file: %s"
+                   (ediff-abbreviate-file-name file1))
+          (sit-for 2)
+          ;; 1 is an error exit code
+          1)
+         ((not (numberp file2-size))
+          (message "Can't find file: %s"
+                   (ediff-abbreviate-file-name file2))
+          (sit-for 2)
+          ;; 1 is an error exit code
+          1)
+         ((< file1-size 0)
+          (message "Can't diff remote files: %s"
+                   (ediff-abbreviate-file-name file1))
+          (sit-for 2)
+          ;; 1 is an error exit code
+          1)
+         ((< file2-size 0)
+          (message "Can't diff remote file: %s"
+                   (ediff-abbreviate-file-name file2))
+          (sit-for 2)
+          (message "")
+          ;; 1 is an error exit code
+          1)
+         (t (message "Computing differences between %s and %s ..."
+                     (file-name-nondirectory file1)
+                     (file-name-nondirectory file2))
+            ;; this erases the diff buffer automatically
+            (ediff-exec-process ediff-diff-program
+                                diff-buffer
+                                'synchronize
+                                ediff-diff-options file1 file2)
+            (message "")
+            (ediff-with-current-buffer diff-buffer
+              (buffer-size))))))
+  
+
      
 ;; If file-A/B/C is nil, do 2-way comparison with the non-nil buffers
 ;; This function works for diff3 and diff2 jobs
@@ -156,15 +282,17 @@ one optional arguments, diff-number to refine.")
            (get-buffer-create
             (ediff-unique-buffer-name "*ediff-fine-diff" "*"))))
   
-  (let (diff3-job diff-program ok-regexp diff-list)
+  (let (diff3-job diff-program diff-options ok-regexp diff-list)
     (setq diff3-job ediff-3way-job
          diff-program (if diff3-job ediff-diff3-program ediff-diff-program)
+         diff-options (if diff3-job ediff-diff3-options ediff-diff-options)
          ok-regexp (if diff3-job
                        ediff-diff3-ok-lines-regexp
                        ediff-diff-ok-lines-regexp))
     
     (ediff-message-if-verbose "Refining difference region %d ..." (1+ reg-num))
     (ediff-exec-process diff-program ediff-fine-diff-buffer 'synchronize
+                       diff-options
                        ;; The shuffle below is because we can compare 3-way
                        ;; or in several 2-way fashions, like fA fC, fA fB,
                        ;; or fB fC.
@@ -175,7 +303,8 @@ one optional arguments, diff-number to refine.")
                        ) ; exec process
   
     (ediff-prepare-error-list ok-regexp ediff-fine-diff-buffer)
-    ;;(ediff-message-if-verbose
+    (ediff-message-if-verbose
+     "")
     ;; "Refining difference region %d ... done" (1+ reg-num))
     
     (setq diff-list
@@ -186,19 +315,19 @@ one optional arguments, diff-number to refine.")
     ;; fixup diff-list
     (if diff3-job
        (cond ((not file-A)
-              (mapcar (function (lambda (elt)
-                                  (aset elt 0 nil)
-                                  (aset elt 1 nil)))
+              (mapcar (lambda (elt)
+                        (aset elt 0 nil)
+                        (aset elt 1 nil))
                       (cdr diff-list)))
              ((not file-B)
-              (mapcar (function (lambda (elt)
-                                  (aset elt 2 nil)
-                                  (aset elt 3 nil)))
+              (mapcar (lambda (elt)
+                        (aset elt 2 nil)
+                        (aset elt 3 nil))
                       (cdr diff-list)))
              ((not file-C)
-              (mapcar (function (lambda (elt)
-                                  (aset elt 4 nil)
-                                  (aset elt 5 nil)))
+              (mapcar (lambda (elt)
+                        (aset elt 4 nil)
+                        (aset elt 5 nil))
                       (cdr diff-list)))
          ))
     
@@ -211,15 +340,15 @@ one optional arguments, diff-number to refine.")
       (setq ediff-error-buffer
            (get-buffer-create (ediff-unique-buffer-name
                                "*ediff-errors" "*"))))
-  (ediff-eval-in-buffer ediff-error-buffer
+  (ediff-with-current-buffer ediff-error-buffer
     (erase-buffer)
-    (insert (ediff-eval-in-buffer diff-buff (buffer-string)))
+    (insert (ediff-with-current-buffer diff-buff (buffer-string)))
     (goto-char (point-min))
     (delete-matching-lines ok-regexp)
     (if (memq system-type '(vax-vms axp-vms))
        (delete-matching-lines "^$")))
   ;; If diff reports errors, show them then quit.
-  (if (/= 0 (ediff-eval-in-buffer ediff-error-buffer (buffer-size)))
+  (if (/= 0 (ediff-with-current-buffer ediff-error-buffer (buffer-size)))
       (let ((ctl-buf ediff-control-buffer)
            (error-buf ediff-error-buffer))
        (ediff-skip-unsuitable-frames)
@@ -256,15 +385,15 @@ one optional arguments, diff-number to refine.")
               (ediff-get-value-according-to-buffer-type 'B bounds))))
     
     ;; reset point in buffers A/B/C
-    (ediff-eval-in-buffer A-buffer
+    (ediff-with-current-buffer A-buffer
       (goto-char (if shift-A shift-A (point-min))))
-    (ediff-eval-in-buffer B-buffer
+    (ediff-with-current-buffer B-buffer
       (goto-char (if shift-B shift-B (point-min))))
     (if (ediff-buffer-live-p C-buffer)
-       (ediff-eval-in-buffer C-buffer
+       (ediff-with-current-buffer C-buffer
          (goto-char (point-min))))
     
-    (ediff-eval-in-buffer diff-buffer
+    (ediff-with-current-buffer diff-buffer
       (goto-char (point-min))
       (while (re-search-forward ediff-match-diff-line nil t)
        (let* ((a-begin (string-to-int (buffer-substring (match-beginning 1)
@@ -332,20 +461,20 @@ one optional arguments, diff-number to refine.")
                   b-prev b-end
                   c-prev c-end)
           ;; else convert lines to points
-          (ediff-eval-in-buffer A-buffer
+          (ediff-with-current-buffer A-buffer
             (forward-line (- a-begin a-prev))
             (setq a-begin-pt (point))
             (forward-line (- a-end a-begin))
             (setq a-end-pt (point)
                   a-prev a-end))
-          (ediff-eval-in-buffer B-buffer
+          (ediff-with-current-buffer B-buffer
             (forward-line (- b-begin b-prev))
             (setq b-begin-pt (point))
             (forward-line (- b-end b-begin))
             (setq b-end-pt (point)
                   b-prev b-end))
           (if (ediff-buffer-live-p C-buffer)
-              (ediff-eval-in-buffer C-buffer
+              (ediff-with-current-buffer C-buffer
                 (forward-line (- c-begin c-prev))
                 (setq c-begin-pt (point))
                 (forward-line (- c-end c-begin))
@@ -375,7 +504,7 @@ one optional arguments, diff-number to refine.")
                             ))) 
                  )))
                  
-        ))) ; end ediff-eval-in-buffer
+        ))) ; end ediff-with-current-buffer
     diff-list
     ))
     
@@ -391,13 +520,14 @@ one optional arguments, diff-number to refine.")
   (if ediff-merge-job
       (setq ediff-state-of-merge
            (vconcat
-            (mapcar (function
-                     (lambda (elt)
-                       (let ((state-of-merge (aref elt 9))
-                             (state-of-ancestor (aref elt 10)))
-                         (vector
-                          (if state-of-merge (format "%S" state-of-merge))
-                          state-of-ancestor))))
+            (mapcar (lambda (elt)
+                      (let ((state-of-merge (aref elt 9))
+                            (state-of-ancestor (aref elt 10)))
+                        (vector
+                         ;; state of merge: prefers/default-A/B or combined
+                         (if state-of-merge (format "%S" state-of-merge))
+                         ;; whether the ancestor region is empty
+                         state-of-ancestor)))
                     ;; the first elt designates type of list
                     (cdr diff-list))
             )))
@@ -423,7 +553,7 @@ one optional arguments, diff-number to refine.")
     (setq total-diffs (length diff-list))
       
     ;; shift, if necessary
-    (ediff-eval-in-buffer buff (setq pt-saved shift))
+    (ediff-with-current-buffer buff (setq pt-saved shift))
           
     (while diff-list
       (setq current-diff (1+ current-diff)
@@ -452,23 +582,17 @@ one optional arguments, diff-number to refine.")
       ;; convert word numbers to points, if necessary
       (if (eq diff-list-type 'words)
          (progn
-           (ediff-eval-in-buffer buff (goto-char pt-saved))
+           (ediff-with-current-buffer buff (goto-char pt-saved))
            (setq begin (ediff-goto-word (1+ begin) buff)
                  end (ediff-goto-word end buff 'end))
            (if (> end limit) (setq end limit))
            (if (> begin end) (setq begin end))
-           (setq pt-saved (ediff-eval-in-buffer buff (point)))))
+           (setq pt-saved (ediff-with-current-buffer buff (point)))))
       (setq overlay (ediff-make-bullet-proof-overlay begin end buff))
       
-      ;; Priorities of overlays should be equal in all ediff control
-      ;; panel buffers. Otherwise it won't work due to Emacs
-      ;; bug, as insert-in-front-hooks will be called 
-      ;; only on behalf of the buffer with higher priority.
       (ediff-overlay-put overlay 'priority ediff-shadow-overlay-priority)
       (ediff-overlay-put overlay 'ediff-diff-num current-diff)
-      (ediff-overlay-put 
-       overlay 'insert-in-front-hooks '(ediff-insert-in-front))
-      (if (and (ediff-window-display-p)
+      (if (and (ediff-has-face-support-p)
               ediff-use-faces ediff-highlight-all-diffs)
          (ediff-set-overlay-face
           overlay (ediff-background-face buf-type current-diff)))
@@ -476,9 +600,11 @@ one optional arguments, diff-number to refine.")
       (if (= 0 (mod current-diff 10))
          (message "Buffer %S: Processing difference region %d of %d"
                   buf-type current-diff total-diffs))
-      ;; record all overlays for this difference
-      ;; the second elt, nil, is a place holder for the fine diff vector.
-      ;; the third elt, nil, is a place holder for no-fine-diffs flag.
+      ;; Record all overlays for this difference.
+      ;; The 2-d elt, nil, is a place holder for the fine diff vector.
+      ;; The 3-d elt, nil, is a place holder for no-fine-diffs flag.
+      ;; The 4-th elt says which diff region is different from the other two
+      ;; (3-way jobs only).
       (setq diff-overlay-list
            (nconc
             diff-overlay-list
@@ -487,7 +613,7 @@ one optional arguments, diff-number to refine.")
            (cdr diff-list))
       ) ; while
       
-    (set (intern (format "ediff-difference-vector-%S" buf-type))
+    (set (ediff-get-symbol-from-alist buf-type ediff-difference-vector-alist)
         (vconcat diff-overlay-list))
     ))
 
@@ -499,7 +625,7 @@ one optional arguments, diff-number to refine.")
   (or n  (setq n ediff-current-difference))
   
   (if (< ediff-number-of-differences 1)
-      (error "No differences found"))
+      (error ediff-NO-DIFFERENCES))
       
   (if ediff-word-mode
       (setq flag 'skip
@@ -520,12 +646,7 @@ one optional arguments, diff-number to refine.")
            (whitespace-C (ediff-whitespace-diff-region-p n 'C))
            cumulative-fine-diff-length)
        
-       (cond ((and (eq flag 'noforce) (ediff-get-fine-diff-vector n 'A))
-              ;; don't compute fine diffs if diff vector exists
-              (if (ediff-no-fine-diffs-p n)
-                  (ediff-message-if-verbose
-                   "Only white-space differences in region %d" (1+ n))))
-             ;; If one of the regions is empty (or 2 in 3way comparison)
+       (cond ;; If one of the regions is empty (or 2 in 3way comparison)
              ;; then don't refine.
              ;; If the region happens to be entirely whitespace or empty then
              ;; mark as such.
@@ -547,16 +668,33 @@ one optional arguments, diff-number to refine.")
                        (empty-B 'B)
                        (empty-C 'C)))
                 )
-              ;; if all regions happen to be whitespace, indicate this
+              ;; if all regions happen to be whitespace
               (if (and whitespace-A whitespace-B whitespace-C)
+                  ;; mark as space only
                   (ediff-mark-diff-as-space-only n t)
+                ;; if some regions are white and others don't, then mark as
+                ;; non-white-space-only
                 (ediff-mark-diff-as-space-only n nil)))
+
+             ;; don't compute fine diffs if diff vector exists
+             ((and (eq flag 'noforce) (ediff-get-fine-diff-vector n 'A))
+              (if (ediff-no-fine-diffs-p n)
+                  (message
+                   "Only white-space differences in region %d %s"
+                   (1+ n)
+                   (cond ((eq (ediff-no-fine-diffs-p n) 'A)
+                          "in buffers B & C")
+                         ((eq (ediff-no-fine-diffs-p n) 'B)
+                          "in buffers A & C")
+                         ((eq (ediff-no-fine-diffs-p n) 'C)
+                          "in buffers A & B")
+                         (t "")))))
              ;; don't compute fine diffs for this region
              ((eq flag 'skip)
               (or (ediff-get-fine-diff-vector n 'A)
                   (memq ediff-auto-refine '(off nix))
                   (ediff-message-if-verbose
-                   "Region %d exceeds auto-refine limit. `%s' force-refines"
+                   "Region %d exceeds the auto-refinement limit. Type `%s' to refine"
                    (1+ n)
                    (substitute-command-keys
                     "\\[ediff-make-or-kill-fine-diffs]")
@@ -604,20 +742,20 @@ one optional arguments, diff-number to refine.")
                     ((and ediff-3way-job whitespace-B)
                      (ediff-setup-fine-diff-regions file-A nil file-C n))
                     ((and ediff-3way-job
-                          (or whitespace-C
-                              (and ediff-merge-job
-                                   (ediff-looks-like-combined-merge n))))
+                          ;; In merge-jobs, whitespace-C is t, since
+                          ;; ediff-empty-diff-region-p returns t in this case
+                          whitespace-C)
                      (ediff-setup-fine-diff-regions file-A file-B nil n))
                     (t
                      (ediff-setup-fine-diff-regions file-A file-B file-C n)))
                      
               (setq cumulative-fine-diff-length
                     (+ (length (ediff-get-fine-diff-vector n 'A))
-                                  (length (ediff-get-fine-diff-vector n 'B))
-                                  (if file-C
-                                      (length
-                                       (ediff-get-fine-diff-vector n 'C))
-                                    0)))
+                       (length (ediff-get-fine-diff-vector n 'B))
+                       ;; in merge jobs, the merge buffer is never refined
+                       (if (and file-C (not ediff-merge-job))
+                           (length (ediff-get-fine-diff-vector n 'C))
+                         0)))
                      
               (cond ((or
                       ;; all regions are white space
@@ -631,13 +769,15 @@ one optional arguments, diff-number to refine.")
                      (ediff-message-if-verbose
                       "Only white-space differences in region %d" (1+ n)))
                     ((eq cumulative-fine-diff-length 0)
-                     (ediff-mark-diff-as-space-only n nil)
                      (ediff-message-if-verbose
                       "Only white-space differences in region %d %s"
                       (1+ n)
-                      (cond (whitespace-A "in buffers B & C")
-                            (whitespace-B "in buffers A & C")
-                            (whitespace-C "in buffers A & B"))))
+                      (cond (whitespace-A (ediff-mark-diff-as-space-only n 'A)
+                                          "in buffers B & C")
+                            (whitespace-B (ediff-mark-diff-as-space-only n 'B)
+                                          "in buffers A & C")
+                            (whitespace-C (ediff-mark-diff-as-space-only n 'C)
+                                          "in buffers A & B"))))
                     (t 
                      (ediff-mark-diff-as-space-only n nil)))
               )
@@ -665,10 +805,10 @@ one optional arguments, diff-number to refine.")
     
 ;; if fine diff vector is not set for diff N, then do nothing
 (defun ediff-set-fine-diff-properties (n &optional default)
-  (or (not (ediff-window-display-p))
+  (or (not (ediff-has-face-support-p))
       (< n 0)
       (>= n ediff-number-of-differences)
-      ;; in a window system, set faces and priorities of fine overlays
+      ;; when faces are supported, set faces and priorities of fine overlays
       (progn
        (ediff-set-fine-diff-properties-in-one-buffer 'A n default)
        (ediff-set-fine-diff-properties-in-one-buffer 'B n default)
@@ -681,36 +821,40 @@ one optional arguments, diff-number to refine.")
        (face (if default 
                  'default
                (face-name
-                (intern (format "ediff-fine-diff-face-%S" buf-type)))))
+                (ediff-get-symbol-from-alist
+                 buf-type ediff-fine-diff-face-alist))))
        (priority (if default
                      0
                    (1+ (or (ediff-overlay-get
                             (symbol-value
-                             (intern
-                              (format
-                               "ediff-current-diff-overlay-%S" buf-type)))
+                             (ediff-get-symbol-from-alist
+                              buf-type
+                              ediff-current-diff-overlay-alist))
                             'priority)
                            0)))))
-    (mapcar
-     (function (lambda (overl)
-                (ediff-set-overlay-face overl face)
-                (ediff-overlay-put overl 'priority priority)))
-     fine-diff-vector)))
+    (mapcar (lambda (overl)
+             (ediff-set-overlay-face overl face)
+             (ediff-overlay-put overl 'priority priority))
+           fine-diff-vector)))
      
-;; This assumes buffer C and that the region looks like a combination of
-;; regions in buffer A and C.
+;; Set overlays over the regions that denote delimiters
 (defun ediff-set-fine-overlays-for-combined-merge (diff-list reg-num)
-  (let (overlay1 overlay2 overlay3)
-    (setq overlay1 (ediff-make-bullet-proof-overlay (nth 0 diff-list)
-                                                   (nth 1 diff-list)
-                                                   ediff-buffer-C)
-         overlay2 (ediff-make-bullet-proof-overlay (nth 2 diff-list)
-                                                   (nth 3 diff-list)
-                                                   ediff-buffer-C)
-         overlay3 (ediff-make-bullet-proof-overlay (nth 4 diff-list)
-                                                   (nth 5 diff-list)
-                                                   ediff-buffer-C))
-    (ediff-set-fine-diff-vector reg-num 'C (vector overlay1 overlay2 overlay3))
+  (let (overlay overlay-list)
+    (while diff-list
+      (condition-case nil
+         (setq overlay
+               (ediff-make-bullet-proof-overlay
+                (nth 0 diff-list) (nth 1 diff-list) ediff-buffer-C))
+       (error ""))
+      (setq overlay-list (cons overlay overlay-list))
+      (if (> (length diff-list) 1)
+         (setq diff-list (cdr (cdr diff-list)))
+       (error "ediff-set-fine-overlays-for-combined-merge: corrupt list of
+delimiter regions"))
+      )
+    (setq overlay-list (reverse overlay-list))
+    (ediff-set-fine-diff-vector
+     reg-num 'C (apply 'vector overlay-list))
     ))
        
     
@@ -726,7 +870,7 @@ one optional arguments, diff-number to refine.")
 
     (ediff-clear-fine-differences-in-one-buffer region-num buf-type)
     (setq diff-list (cdr diff-list)) ; discard list type (words or points)
-    (ediff-eval-in-buffer buff (goto-char reg-start))
+    (ediff-with-current-buffer buff (goto-char reg-start))
     
     ;; if it is a combined merge then set overlays in buff C specially
     (if (and ediff-merge-job (eq buf-type 'C)
@@ -829,24 +973,24 @@ one optional arguments, diff-number to refine.")
                   (ediff-get-value-according-to-buffer-type 'C bounds)))))
     
     ;; reset point in buffers A, B, C
-    (ediff-eval-in-buffer A-buffer
+    (ediff-with-current-buffer A-buffer
       (goto-char (if shift-A shift-A (point-min))))
-    (ediff-eval-in-buffer B-buffer
+    (ediff-with-current-buffer B-buffer
       (goto-char (if shift-B shift-B (point-min))))
     (if three-way-comp
-       (ediff-eval-in-buffer C-buffer
+       (ediff-with-current-buffer C-buffer
          (goto-char (if shift-C shift-C (point-min)))))
     (if (ediff-buffer-live-p anc-buffer)
-       (ediff-eval-in-buffer anc-buffer
+       (ediff-with-current-buffer anc-buffer
          (goto-char (point-min))))
     
-    (ediff-eval-in-buffer diff-buffer
+    (ediff-with-current-buffer diff-buffer
       (goto-char (point-min))
       (while (re-search-forward ediff-match-diff3-line nil t)
        ;; leave point after matched line
        (beginning-of-line 2)
        (let ((agreement (buffer-substring (match-beginning 1) (match-end 1))))
-        ;; if the A and B files are the same and not 3way-comparison,
+        ;; if the files A and B are the same and not 3way-comparison,
         ;; ignore the difference
         (if (or three-way-comp (not (string-equal agreement "3")))
             (let* ((a-begin (car (ediff-get-diff3-group "1")))
@@ -903,26 +1047,26 @@ one optional arguments, diff-number to refine.")
                         b-prev b-end
                         c-prev c-end)
                 ;; else convert lines to points
-                (ediff-eval-in-buffer A-buffer
+                (ediff-with-current-buffer A-buffer
                   (forward-line (- a-begin a-prev))
                   (setq a-begin-pt (point))
                   (forward-line (- a-end a-begin))
                   (setq a-end-pt (point)
                         a-prev a-end))
-                (ediff-eval-in-buffer B-buffer
+                (ediff-with-current-buffer B-buffer
                   (forward-line (- b-begin b-prev))
                   (setq b-begin-pt (point))
                   (forward-line (- b-end b-begin))
                   (setq b-end-pt (point)
                         b-prev b-end))
-                (ediff-eval-in-buffer C-buffer
+                (ediff-with-current-buffer C-buffer
                   (forward-line (- c-begin c-prev))
                   (setq c-begin-pt (point))
                   (forward-line (- c-end c-begin))
                   (setq c-end-pt (point)
                         c-prev c-end))
                 (if (ediff-buffer-live-p anc-buffer)
-                    (ediff-eval-in-buffer anc-buffer
+                    (ediff-with-current-buffer anc-buffer
                       (forward-line (- c-or-anc-begin anc-prev))
                       (setq anc-begin-pt (point))
                       (forward-line (- c-or-anc-end c-or-anc-begin))
@@ -954,7 +1098,7 @@ one optional arguments, diff-number to refine.")
                        )))
               ))
               
-        ))) ; end ediff-eval-in-buffer
+        ))) ; end ediff-with-current-buffer
     diff-list
     ))
     
@@ -962,11 +1106,6 @@ one optional arguments, diff-number to refine.")
 ;; File-C is either the third file to compare (in case of 3-way comparison)
 ;; or it is the ancestor file.
 (defun ediff-setup-diff-regions3 (file-A file-B file-C)
-  
-  ;; force all minibuffers to display ediff's messages.
-  ;; when xemacs implements minibufferless frames, this won't be necessary
-  (if ediff-xemacs-p (setq synchronize-minibuffers t))
-                                                 
   (or (ediff-buffer-live-p ediff-diff-buffer)
       (setq ediff-diff-buffer
            (get-buffer-create (ediff-unique-buffer-name "*ediff-diff" "*"))))
@@ -984,16 +1123,22 @@ one optional arguments, diff-number to refine.")
    ))
    
 
-;; Execute PROGRAM asynchronously, unless OS/2 or unless SYNCH is non-nil.
-;; BUFFER must be a buffer object, and must be alive.
-;; All arguments in ARGS must be strings. The first arg may be a blank string,
-;; in which case we delete it from ARGS list. We also delete nil from args.
-(defun ediff-exec-process (program buffer synch &rest args)
-  (let ((data (match-data)))
-    (if (string-match "^[ \t]*$" (car args)) ; delete blank string
-       (setq args (cdr args)))
-    (setq args (delq nil args)) ; delete nil from arguments
-    (setq args (ediff-split-string (mapconcat 'identity args " ")))
+;; Execute PROGRAM asynchronously, unless OS/2, Windows-*, or DOS, or unless
+;; SYNCH is non-nil.  BUFFER must be a buffer object, and must be alive.  The
+;; OPTIONS arg is a list of options to pass to PROGRAM. It may be a blank
+;; string.  All elements in FILES must be strings.  We also delete nil from
+;; args.
+(defun ediff-exec-process (program buffer synch options &rest files)
+  (let ((data (match-data))
+       (coding-system-for-read 'no-conversion)
+       args)
+    (setq args (append (split-string options) files))
+    (setq args (delete "" (delq nil args))) ; delete nil and "" from arguments
+    ;; the --binary option, if present, should be used only for buffer jobs
+    ;; or for refining the differences
+    (or (string-match "buffer" (symbol-name ediff-job-name))
+       (eq buffer ediff-fine-diff-buffer)
+       (setq args (delete "--binary" args)))
     (unwind-protect
        (let ((directory default-directory)
              proc)
@@ -1001,13 +1146,17 @@ one optional arguments, diff-number to refine.")
            (set-buffer buffer)
            (erase-buffer)
            (setq default-directory directory)
-           (if (or (eq system-type 'emx) synch)
-               ;; In OS/2 do it synchronously, since OS/2 doesn't let us
+           (if (or (memq system-type '(emx ms-dos windows-nt windows-95))
+                   synch)
+               ;; In OS/2 (emx) do it synchronously, since OS/2 doesn't let us
                ;; delete files used by other processes. Thus, in ediff-buffers
                ;; and similar functions, we can't delete temp files because
                ;; they might be used by the asynch process that computes
                ;; custom diffs. So, we have to wait till custom diff
                ;; subprocess is done.
+               ;; Similarly for Windows-*
+               ;; In DOS, must synchronize because DOS doesn't have
+               ;; asynchronous processes.
                (apply 'call-process program nil buffer nil args)
              ;; On other systems, do it asynchronously.
              (setq proc (get-buffer-process buffer))
@@ -1020,7 +1169,7 @@ one optional arguments, diff-number to refine.")
              )))
       (store-match-data data))))
       
-;; This is shell-command-filter from simple.el in FSF Emacs.
+;; This is shell-command-filter from simple.el in Emacs.
 ;; Copied here because XEmacs doesn't have it.
 (defun ediff-process-filter (proc string)
   ;; Do save-excursion by hand so that we can leave point numerically unchanged
@@ -1067,7 +1216,6 @@ Used for splitting difference regions into individual words.")
   "*Characters constituting white space.
 These characters are ignored when differing regions are split into words.")
 
-;;(defvar ediff-word-1 "a-zA-Z---_`'.?!:"
 (defvar ediff-word-1 "a-zA-Z---_"
   "*Characters that constitute words of type 1.
 More precisely, [ediff-word-1] is a regexp that matches type 1 words.
@@ -1095,21 +1243,29 @@ See `ediff-forward-word' for more details.")
   "Move point one word forward.
 There are four types of words, each of which consists entirely of
 characters in `ediff-word-1', `ediff-word-2', `ediff-word-3', or
-`ediff-word-4'.  Words are recognized by passing these in turn as the
-argument to `skip-chars-forward'."
-  (or (> (skip-chars-forward ediff-word-1) 0)
+`ediff-word-4'.  Words are recognized by passing these one after another as
+arguments to `skip-chars-forward'."
+  (or (> (+ (skip-chars-forward ediff-word-1)
+           (skip-syntax-forward "w"))
+        0)
       (> (skip-chars-forward ediff-word-2) 0)
       (> (skip-chars-forward ediff-word-3) 0)
       (> (skip-chars-forward ediff-word-4) 0)
       ))
 
+
 (defun ediff-wordify (beg end in-buffer out-buffer &optional control-buf)
-  (let (sv-point string)
+  (let (inbuf-syntax-tbl sv-point string)
     (save-excursion
      (set-buffer in-buffer)
-     (setq string (buffer-substring beg end))
+     (setq inbuf-syntax-tbl (syntax-table))
+     (setq string (buffer-substring-no-properties beg end))
 
      (set-buffer out-buffer)
+     ;; Make sure that temp buff syntax table is the same a the original buf
+     ;; syntax tbl, because we use ediff-forward-word in both and
+     ;; ediff-forward-word depends on the syntax classes of characters.
+     (set-syntax-table inbuf-syntax-tbl)
      (erase-buffer)
      (insert string)
      (goto-char (point-min))
@@ -1121,7 +1277,8 @@ argument to `skip-chars-forward'."
        ;; different invocations
        (if control-buf
           (funcall 
-           (ediff-eval-in-buffer control-buf ediff-forward-word-function))
+           (ediff-with-current-buffer 
+               control-buf ediff-forward-word-function))
         (funcall ediff-forward-word-function))
        (setq sv-point (point))
        (skip-chars-forward ediff-whitespace)
@@ -1148,7 +1305,7 @@ argument to `skip-chars-forward'."
 (defun ediff-goto-word (n buf &optional flag)
   ;; remember val ediff-forward-word-function has in ctl buf
   (let ((fwd-word-fun ediff-forward-word-function))
-    (ediff-eval-in-buffer buf
+    (ediff-with-current-buffer buf
       (skip-chars-forward ediff-whitespace)
       (while (> n 1)
        (funcall fwd-word-fun)
@@ -1158,8 +1315,19 @@ argument to `skip-chars-forward'."
          (funcall fwd-word-fun))
       (point))))
 
+(defun ediff-same-file-contents (f1 f2)
+  "T if F1 and F2 have identical contents."
+  (let ((res 
+        (apply 'call-process ediff-cmp-program nil nil nil
+               (append ediff-cmp-options (list f1 f2)))))
+    (and (numberp res) (eq res 0))))
 
-(provide 'ediff-diff)
+
+;;; Local Variables:
+;;; eval: (put 'ediff-defvar-local 'lisp-indent-hook 'defun)
+;;; eval: (put 'ediff-with-current-buffer 'lisp-indent-hook 1)
+;;; eval: (put 'ediff-with-current-buffer 'edebug-form-spec '(form body))
+;;; End:
 
 
 ;; ediff-diff.el ends here