(exec-suffixes): Initialize to a system-dependent value.
[bpt/emacs.git] / lisp / diff-mode.el
index 8527377..2c86b1f 100644 (file)
@@ -1,11 +1,9 @@
 ;;; diff-mode.el --- A mode for viewing/editing context diffs
 
-;; Copyright (C) 1998-1999  Free Software Foundation, Inc.
+;; Copyright (C) 1998, 1999, 2000, 2001  Free Software Foundation, Inc.
 
 ;; Author: Stefan Monnier <monnier@cs.yale.edu>
 ;; Keywords: patch diff
-;; Version: v1_8
-;; Revision: diff-mode.el,v 1.11 1999/10/09 23:38:29 monnier Exp
 
 ;; This file is part of GNU Emacs.
 
 
 ;;; Commentary:
 
-;; Provides support for font-lock patterns, outline-regexps, navigation
+;; Provides support for font-lock, outline, navigation
 ;; commands, editing and various conversions as well as jumping
 ;; to the corresponding source file.
 
-;; History:
-
 ;; inspired by Pavel Machek's patch-mode.el (<pavel@atrey.karlin.mff.cuni.cz>)
 ;; some efforts were spent to have it somewhat compatible with XEmacs'
 ;; diff-mode as well as with compilation-minor-mode
 
 ;; Bugs:
 
-;; - reverse doesn't work with normal diffs.
-;; - (nitpick) the mark is not always quite right in diff-goto-source.
+;; - Reverse doesn't work with normal diffs.
 
 ;; Todo:
 
-;; - improve narrowed-view support.
-;; - improve diff-find-file-name.
-;; - improve the `compile' support.
+;; - Improve narrowed-view support.
+;; - re-enable (conditionally) the `compile' support after improving it to use
+;;   the same code as diff-goto-source.
+;; - Support for # comments in context->unified.
+;; - Do a fuzzy search in diff-goto-source.
+;; - Allow diff.el to use diff-mode.
+;;   This mostly means ability to jump from half-hunk to half-hunk
+;;   in context (and normal) diffs and to jump to the corresponding
+;;   (i.e. new or old) file.
+;; - Handle `diff -b' output in context->unified.
+
+;; Low priority:
+;; - Spice up the minor-mode with font-lock support.
+;; - Recognize pcl-cvs' special string for `cvs-execute-single'.
 
 ;;; Code:
 
 
 
 (defgroup diff-mode ()
-  "Major-mode for viewing/editing diffs"
+  "Major mode for viewing/editing diffs"
+  :version "21.1"
   :group 'tools
   :group 'diff)
 
-(defcustom diff-jump-to-old-file-flag nil
+(defcustom diff-default-read-only t
+  "If non-nil, `diff-mode' buffers default to being read-only."
+  :type 'boolean
+  :group 'diff-mode)
+
+(defcustom diff-jump-to-old-file nil
   "*Non-nil means `diff-goto-source' jumps to the old file.
 Else, it jumps to the new file."
   :group 'diff-mode
   :type '(boolean))
 
-(defcustom diff-update-on-the-fly-flag t
+(defcustom diff-update-on-the-fly t
   "*Non-nil means hunk headers are kept up-to-date on-the-fly.
 When editing a diff file, the line numbers in the hunk headers
 need to be kept consistent with the actual diff.  This can
@@ -78,37 +90,36 @@ when editing big diffs)."
   :group 'diff-mode
   :type '(boolean))
 
+(defcustom diff-advance-after-apply-hunk t
+  "*Non-nil means `diff-apply-hunk' will move to the next hunk after applying."
+  :group 'diff-mode
+  :type 'boolean)
+
+
 (defvar diff-mode-hook nil
   "Run after setting up the `diff-mode' major mode.")
 
 (defvar diff-outline-regexp
   "\\([*+][*+][*+] [^0-9]\\|@@ ...\\|\\*\\*\\* [0-9].\\|--- [0-9]..\\)")
 
-;;;; 
+;;;;
 ;;;; keymap, menu, ...
-;;;; 
+;;;;
 
-(defmacro diff-defmap (var bindings doc)
-  `(defvar ,var
-     (let ((m (make-keymap)))
-       (dolist (b ,bindings)
-        (define-key m (car b) (cdr b)))
-       m)
-     ,doc))
-
-(diff-defmap diff-mode-shared-map
-  '(;; from Pavel Machek's patch-mode
-    ("n" . diff-next-hunk)
-    ("N" . diff-next-file)
-    ("p" . diff-prev-hunk)
-    ("P" . diff-prev-file)
-    ("k" . diff-kill-hunk)
-    ("K" . diff-kill-file)
-    ;; from compilation-minor-mode
-    ("}" . diff-next-file)
-    ("{" . diff-prev-file)
+(easy-mmode-defmap diff-mode-shared-map
+  '(;; From Pavel Machek's patch-mode.
+    ("n" . diff-hunk-next)
+    ("N" . diff-file-next)
+    ("p" . diff-hunk-prev)
+    ("P" . diff-file-prev)
+    ("k" . diff-hunk-kill)
+    ("K" . diff-file-kill)
+    ;; From compilation-minor-mode.
+    ("}" . diff-file-next)
+    ("{" . diff-file-prev)
     ("\C-m" . diff-goto-source)
-    ;; from XEmacs' diff-mode
+    ([mouse-2] . diff-mouse-goto-source)
+    ;; From XEmacs' diff-mode.
     ("W" . widen)
     ;;("." . diff-goto-source)         ;display-buffer
     ;;("f" . diff-goto-source)         ;find-file
@@ -120,25 +131,30 @@ when editing big diffs)."
     ;;("q" . diff-quit)
     (" " . scroll-up)
     ("\177" . scroll-down)
-    ;; our very own bindings
+    ;; Our very own bindings.
     ("A" . diff-ediff-patch)
     ("r" . diff-restrict-view)
     ("R" . diff-reverse-direction)
     ("U" . diff-context->unified)
     ("C" . diff-unified->context))
-  "Keymap for read-only `diff-mode'. Only active in read-only mode.")
+  "Basic keymap for `diff-mode', bound to various prefix keys.")
 
-(diff-defmap diff-mode-map
+(easy-mmode-defmap diff-mode-map
   `(("\e" . ,diff-mode-shared-map)
-    ;; from compilation-minor-mode
-    ("\C-c\C-c" . diff-goto-source))
+    ;; From compilation-minor-mode.
+    ("\C-c\C-c" . diff-goto-source)
+    ;; Misc operations.
+    ("\C-c\C-s" . diff-split-hunk)
+    ("\C-c\C-a" . diff-apply-hunk)
+    ("\C-c\C-t" . diff-test-hunk))
   "Keymap for `diff-mode'.  See also `diff-mode-shared-map'.")
 
 (easy-menu-define diff-mode-menu diff-mode-map
   "Menu for `diff-mode'."
   '("Diff"
     ["Jump to Source"          diff-goto-source        t]
-    ["Apply with Ediff"                diff-ediff-patch        t]
+    ["Apply hunk"              diff-apply-hunk         t]
+    ["Apply diff with Ediff"   diff-ediff-patch        t]
     ["-----" nil nil]
     ["Reverse direction"       diff-reverse-direction  t]
     ["Context -> Unified"      diff-context->unified   t]
@@ -146,67 +162,133 @@ when editing big diffs)."
     ;;["Fixup Headers"         diff-fixup-modifs       (not buffer-read-only)]
     ))
 
+(defcustom diff-minor-mode-prefix "\C-c="
+  "Prefix key for `diff-minor-mode' commands."
+  :group 'diff-mode
+  :type '(choice (string "\e") (string "C-c=") string))
+
+(easy-mmode-defmap diff-minor-mode-map
+  `((,diff-minor-mode-prefix . ,diff-mode-shared-map))
+  "Keymap for `diff-minor-mode'.  See also `diff-mode-shared-map'.")
 
-;;;; 
+
+;;;;
 ;;;; font-lock support
-;;;; 
+;;;;
+
+(defface diff-header-face
+  '((((type tty pc) (class color) (background light))
+     (:foreground "blue1" :bold t))
+    (((type tty pc) (class color) (background dark))
+     (:foreground "green" :bold t))
+    (((class color) (background light))
+     (:background "grey85"))
+    (((class color) (background dark))
+     (:background "grey45"))
+    (t (:bold t)))
+  "`diff-mode' face inherited by hunk and index header faces."
+  :group 'diff-mode)
+(defvar diff-header-face 'diff-header-face)
 
 (defface diff-file-header-face
-  '((((class color) (background light))
+  '((((type tty pc) (class color) (background light))
+     (:foreground "yellow" :bold t))
+    (((type tty pc) (class color) (background dark))
+     (:foreground "cyan" :bold t))
+    (((class color) (background light))
      (:background "grey70" :bold t))
-    (t (:bold t)))
-  "diff-mode face used to highlight file header lines."
+    (((class color) (background dark))
+     (:background "grey60" :bold t))
+    (t (:bold t)))                     ; :height 1.3
+  "`diff-mode' face used to highlight file header lines."
   :group 'diff-mode)
 (defvar diff-file-header-face 'diff-file-header-face)
 
 (defface diff-index-face
-  '((((class color) (background light))
-     (:background "grey70" :bold t))
-    (t (:bold t)))
-  "diff-mode face used to highlight index header lines."
+  '((t (:inherit diff-file-header-face)))
+  "`diff-mode' face used to highlight index header lines."
   :group 'diff-mode)
 (defvar diff-index-face 'diff-index-face)
 
 (defface diff-hunk-header-face
-  '((((class color) (background light))
-     (:background "grey85"))
-    (t (:bold t)))
-  "diff-mode face used to highlight hunk header lines."
+  '((t (:inherit diff-header-face)))
+  "`diff-mode' face used to highlight hunk header lines."
   :group 'diff-mode)
 (defvar diff-hunk-header-face 'diff-hunk-header-face)
 
 (defface diff-removed-face
-  '((t ()))
-  "diff-mode face used to highlight removed lines."
+  '((t (:inherit diff-changed-face)))
+  "`diff-mode' face used to highlight removed lines."
   :group 'diff-mode)
 (defvar diff-removed-face 'diff-removed-face)
 
 (defface diff-added-face
-  '((t ()))
-  "diff-mode face used to highlight added lines."
+  '((t (:inherit diff-changed-face)))
+  "`diff-mode' face used to highlight added lines."
   :group 'diff-mode)
 (defvar diff-added-face 'diff-added-face)
 
 (defface diff-changed-face
-  '((t ()))
-  "diff-mode face used to highlight changed lines."
+  '((((type tty pc) (class color) (background light))
+     (:foreground "magenta" :bold t :italic t))
+    (((type tty pc) (class color) (background dark))
+     (:foreground "yellow" :bold t :italic t))
+    (t ()))
+  "`diff-mode' face used to highlight changed lines."
   :group 'diff-mode)
 (defvar diff-changed-face 'diff-changed-face)
 
+(defface diff-function-face
+  '((t (:inherit diff-context-face)))
+  "`diff-mode' face used to highlight function names produced by \"diff -p\"."
+  :group 'diff-mode)
+(defvar diff-function-face 'diff-function-face)
+
+(defface diff-context-face
+  '((((class color) (background light))
+     (:foreground "grey50"))
+    (((class color) (background dark))
+     (:foreground "grey70"))
+    (t ))
+  "`diff-mode' face used to highlight context and other side-information."
+  :group 'diff-mode)
+(defvar diff-context-face 'diff-context-face)
+
+(defface diff-nonexistent-face
+  '((t (:inherit diff-file-header-face)))
+  "`diff-mode' face used to highlight nonexistent files in recursive diffs."
+  :group 'diff-mode)
+(defvar diff-nonexistent-face 'diff-nonexistent-face)
+
 (defvar diff-font-lock-keywords
-  '(("^@@ .+ @@$" . diff-hunk-header-face) ;unified
+  '(("^\\(@@ -[0-9,]+ \\+[0-9,]+ @@\\)\\(.*\\)$" ;unified
+     (1 diff-hunk-header-face)
+     (2 diff-function-face))
     ("^--- .+ ----$" . diff-hunk-header-face) ;context
-    ("^\\*\\*\\*.+\\*\\*\\*\n" . diff-hunk-header-face) ;context
-    ("^\\(---\\|\\+\\+\\+\\|\\*\\*\\*\\) .*\n" . diff-file-header-face)
+    ("^\\(\\*\\{15\\}\\)\\(.*\\)$"     ;context
+     (1 diff-hunk-header-face)
+     (2 diff-function-face))
+    ("^\\*\\*\\* .+ \\*\\*\\*\\*". diff-hunk-header-face) ;context
+    ("^\\(---\\|\\+\\+\\+\\|\\*\\*\\*\\) \\(\\S-+\\)\\(.*[^*-]\\)?\n"
+     (0 diff-header-face) (2 diff-file-header-face prepend))
     ("^[0-9,]+[acd][0-9,]+$" . diff-hunk-header-face)
     ("^!.*\n" . diff-changed-face)     ;context
     ("^[+>].*\n" . diff-added-face)
     ("^[-<].*\n" . diff-removed-face)
-    ("^Index: .*\n" . diff-index-face)
-    ("^[^-=+*!<>].*\n" . font-lock-comment-face)))
+    ("^Index: \\(.+\\).*\n" (0 diff-header-face) (1 diff-index-face prepend))
+    ("^Only in .*\n" . diff-nonexistent-face)
+    ("^#.*" . font-lock-string-face)
+    ("^[^-=+*!<>].*\n" . diff-context-face)))
 
 (defconst diff-font-lock-defaults
-  '(diff-font-lock-keywords t nil nil nil))
+  '(diff-font-lock-keywords t nil nil nil (font-lock-multiline . nil)))
+
+(defvar diff-imenu-generic-expression
+  ;; Prefer second name as first is most likely to be a backup or
+  ;; version-control name.  The [\t\n] at the end of the unidiff pattern
+  ;; catches Debian source diff files (which lack the trailing date).
+  '((nil "\\+\\+\\+\\ \\([^\t\n]+\\)[\t\n]" 1) ; unidiffs
+    (nil "^--- \\([^\t\n]+\\)\t.*\n\\*" 1))) ; context diffs
 
 ;;;;
 ;;;; Compile support
@@ -220,23 +302,25 @@ when editing big diffs)."
     ("--- \\([0-9]+\\),[0-9]+ ----" nil 1)
     ("\\([0-9]+\\)\\(,[0-9]+\\)?[adc]\\([0-9]+\\)" nil 3)))
 
-;;;; 
+;;;;
 ;;;; Movement
-;;;; 
+;;;;
 
-(defconst diff-hunk-header-re "^\\(@@ .+ @@\\|\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\n\\*\\*\\* .+ \\*\\*\\*\\*\\|[0-9]+\\(,[0-9]+\\)?[acd][0-9]+\\(,[0-9]+\\)?\\)$")
+(defconst diff-hunk-header-re "^\\(@@ -[0-9,]+ \\+[0-9,]+ @@.*\\|\\*\\{15\\}.*\n\\*\\*\\* .+ \\*\\*\\*\\*\\|[0-9]+\\(,[0-9]+\\)?[acd][0-9]+\\(,[0-9]+\\)?\\)$")
 (defconst diff-file-header-re (concat "^\\(--- .+\n\\+\\+\\+\\|\\*\\*\\* .+\n---\\|[^-+!<>0-9@* ]\\).+\n" (substring diff-hunk-header-re 1)))
 (defvar diff-narrowed-to nil)
 
 (defun diff-end-of-hunk (&optional style)
   (if (looking-at diff-hunk-header-re) (goto-char (match-end 0)))
-  (re-search-forward (case style
-                      (unified "^[^-+ \\]")
-                      (context "^\\([^-+! \\][ \t]\\|--- .+ ----\\)")
-                      (normal "^\\([<>\\][ \t]\\|---\\)")
-                      (t "^[^-+!<> \\]"))
-                    nil 'move)
-  (beginning-of-line))
+  (let ((end (and (re-search-forward (case style
+                                      (unified "^[^-+# \\]")
+                                      (context "^[^-+#! \\]")
+                                      (normal "^[^<>#\\]")
+                                      (t "^[^-+#!<> \\]"))
+                                    nil t)
+                 (match-beginning 0))))
+    ;; The return value is used by easy-mmode-define-navigation.
+    (goto-char (or end (point-max)))))
 
 (defun diff-beginning-of-hunk ()
   (beginning-of-line)
@@ -255,49 +339,15 @@ when editing big diffs)."
       (error (error "Can't find the beginning of the file")))))
 
 (defun diff-end-of-file ()
-  (re-search-forward "^[-+!<>0-9@* \\]" nil t)
-  (re-search-forward "^[^-+!<>0-9@* \\]" nil 'move)
+  (re-search-forward "^[-+#!<>0-9@* \\]" nil t)
+  (re-search-forward "^[^-+#!<>0-9@* \\]" nil 'move)
   (beginning-of-line))
 
-(defun diff-next-hunk (&optional count)
-  "Move to next (COUNT'th) hunk."
-  (interactive "p")
-  (unless count (setq count 1))
-  (if (< count 0) (diff-prev-hunk (- count))
-    (when (looking-at diff-hunk-header-re) (incf count))
-    (condition-case ()
-       (re-search-forward diff-hunk-header-re nil nil count)
-      (error (error "Can't find next hunk")))
-    (goto-char (match-beginning 0))))
-
-(defun diff-prev-hunk (&optional count)
-  "Move to previous (COUNT'th) hunk."
-  (interactive "p")
-  (unless count (setq count 1))
-  (if (< count 0) (diff-next-hunk (- count))
-    (condition-case ()
-       (re-search-backward diff-hunk-header-re nil nil count)
-      (error (error "Can't find previous hunk")))))
-
-(defun diff-next-file (&optional count)
-  "Move to next (COUNT'th) file header."
-  (interactive "p")
-  (unless count (setq count 1))
-  (if (< count 0) (diff-prev-file (- count))
-    (when (looking-at diff-file-header-re) (incf count))
-    (condition-case ()
-       (re-search-forward diff-file-header-re nil nil count)
-      (error (error "Can't find next file")))
-    (goto-char (match-beginning 0))))
-
-(defun diff-prev-file (&optional count)
-  "Move to (COUNT'th) previous file header."
-  (interactive "p")
-  (unless count (setq count 1))
-  (if (< count 0) (diff-next-file (- count))
-    (condition-case ()
-       (re-search-backward diff-file-header-re nil nil count)
-      (error (error "Can't find previous file")))))
+;; Define diff-{hunk,file}-{prev,next}
+(easy-mmode-define-navigation
+ diff-hunk diff-hunk-header-re "hunk" diff-end-of-hunk)
+(easy-mmode-define-navigation
+ diff-file diff-file-header-re "file" diff-end-of-hunk)
 
 (defun diff-restrict-view (&optional arg)
   "Restrict the view to the current hunk.
@@ -311,49 +361,116 @@ If the prefix ARG is given, restrict the view to the current file instead."
     (set (make-local-variable 'diff-narrowed-to) (if arg 'file 'hunk))))
 
 
-(defun diff-kill-hunk ()
+(defun diff-hunk-kill ()
   "Kill current hunk."
   (interactive)
   (diff-beginning-of-hunk)
   (let ((start (point))
        (firsthunk (save-excursion
                     (ignore-errors
-                      (diff-beginning-of-file) (diff-next-hunk) (point))))
+                      (diff-beginning-of-file) (diff-hunk-next) (point))))
        (nexthunk  (save-excursion
                     (ignore-errors
-                      (diff-next-hunk) (point))))
+                      (diff-hunk-next) (point))))
        (nextfile (save-excursion
                    (ignore-errors
-                     (diff-next-file) (point)))))
+                     (diff-file-next) (point)))))
     (if (and firsthunk (= firsthunk start)
             (or (null nexthunk)
                 (and nextfile (> nexthunk nextfile))))
        ;; we're the only hunk for this file, so kill the file
-       (diff-kill-file)
+       (diff-file-kill)
       (diff-end-of-hunk)
       (kill-region start (point)))))
 
-(defun diff-kill-file ()
+(defun diff-file-kill ()
   "Kill current file's hunks."
   (interactive)
   (diff-beginning-of-file)
   (let* ((start (point))
         (prevhunk (save-excursion
                     (ignore-errors
-                      (diff-prev-hunk) (point))))
+                      (diff-hunk-prev) (point))))
         (index (save-excursion
                  (re-search-backward "^Index: " prevhunk t))))
     (when index (setq start index))
     (diff-end-of-file)
     (kill-region start (point))))
 
+(defun diff-kill-junk ()
+  "Kill spurious empty diffs."
+  (interactive)
+  (save-excursion
+    (let ((inhibit-read-only t))
+      (goto-char (point-min))
+      (while (re-search-forward (concat "^\\(Index: .*\n\\)"
+                                       "\\([^-+!* <>].*\n\\)*?"
+                                       "\\(\\(Index:\\) \\|"
+                                       diff-file-header-re "\\)")
+                               nil t)
+       (delete-region (if (match-end 4) (match-beginning 0) (match-end 1))
+                      (match-beginning 3))
+       (beginning-of-line)))))
+
+(defun diff-count-matches (re start end)
+  (save-excursion
+    (let ((n 0))
+      (goto-char start)
+      (while (re-search-forward re end t) (incf n))
+      n)))
+
+(defun diff-split-hunk ()
+  "Split the current (unified diff) hunk at point into two hunks."
+  (interactive)
+  (beginning-of-line)
+  (let ((pos (point))
+       (start (progn (diff-beginning-of-hunk) (point))))
+    (unless (looking-at "@@ -\\([0-9]+\\),[0-9]+ \\+\\([0-9]+\\),[0-9]+ @@")
+      (error "diff-split-hunk only works on unified context diffs"))
+    (forward-line 1)
+    (let* ((start1 (string-to-number (match-string 1)))
+          (start2 (string-to-number (match-string 2)))
+          (newstart1 (+ start1 (diff-count-matches "^[- \t]" (point) pos)))
+          (newstart2 (+ start2 (diff-count-matches "^[+ \t]" (point) pos))))
+      (goto-char pos)
+      ;; Hopefully the after-change-function will not screw us over.
+      (insert "@@ -" (number-to-string newstart1) ",1 +"
+             (number-to-string newstart2) ",1 @@\n")
+      ;; Fix the original hunk-header.
+      (diff-fixup-modifs start pos))))
+      
+
 ;;;;
 ;;;; jump to other buffers
 ;;;;
 
+(defvar diff-remembered-files-alist nil)
+
 (defun diff-filename-drop-dir (file)
   (when (string-match "/" file) (substring file (match-end 0))))
 
+(defun diff-merge-strings (ancestor from to)
+  "Merge the diff between ANCESTOR and FROM into TO.
+Returns the merged string if successful or nil otherwise.
+The strings are assumed not to contain any \"\\n\" (i.e. end of line).
+If ANCESTOR = FROM, returns TO.
+If ANCESTOR = TO, returns FROM.
+The heuristic is simplistic and only really works for cases
+like \(diff-merge-strings \"b/foo\" \"b/bar\" \"/a/c/foo\")."
+  ;; Ideally, we want:
+  ;;   AMB ANB CMD -> CND
+  ;; but that's ambiguous if `foo' or `bar' is empty:
+  ;; a/foo a/foo1 b/foo.c -> b/foo1.c but not 1b/foo.c or b/foo.c1
+  (let ((str (concat ancestor "\n" from "\n" to)))
+    (when (and (string-match (concat
+                             "\\`\\(.*?\\)\\(.*\\)\\(.*\\)\n"
+                             "\\1\\(.*\\)\\3\n"
+                             "\\(.*\\(\\2\\).*\\)\\'") str)
+              (equal to (match-string 5 str)))
+      (concat (substring str (match-beginning 5) (match-beginning 6))
+             (match-string 4 str)
+             (substring str (match-end 6) (match-end 5))))))
+
 (defun diff-find-file-name (&optional old)
   "Return the file corresponding to the current patch.
 Non-nil OLD means that we want the old file."
@@ -363,12 +480,12 @@ Non-nil OLD means that we want the old file."
          (re-search-forward diff-file-header-re nil t)))
     (let* ((limit (save-excursion
                   (condition-case ()
-                      (progn (diff-prev-hunk) (point))
+                      (progn (diff-hunk-prev) (point))
                     (error (point-min)))))
           (header-files
-           (if (looking-at "[-*][-*][-*] \\(\\S-+\\)\\s-.*\n[-+][-+][-+] \\(\\S-+\\)\\s-.*$")
-               (list (if old (match-string 1) (match-string 2))
-                     (if old (match-string 2) (match-string 1)))
+           (if (looking-at "[-*][-*][-*] \\(\\S-+\\)\\(\\s-.*\\)?\n[-+][-+][-+] \\(\\S-+\\)")
+               (list (if old (match-string 1) (match-string 3))
+                     (if old (match-string 3) (match-string 1)))
              (forward-line 1) nil))
           (fs (append
                (when (save-excursion
@@ -378,71 +495,49 @@ Non-nil OLD means that we want the old file."
                (when (re-search-backward "^diff \\(-\\S-+ +\\)*\\(\\S-+\\)\\( +\\(\\S-+\\)\\)?" nil t)
                  (list (if old (match-string 2) (match-string 4))
                        (if old (match-string 4) (match-string 2))))))
-          (fs (delq nil fs))
-          (file
-           ;; look for each file in turn.  If none found, try again but
-           ;; ignoring the first level of directory, ...
-           (do* ((files fs (delq nil (mapcar 'diff-filename-drop-dir files)))
-                 (file nil nil))
-               ((or (null files)
-                    (setq file (do* ((files files (cdr files))
-                                     (file (car files) (car files)))
-                                   ((or (null file) (file-exists-p file))
-                                    file))))
-                file))))
+          (fs (delq nil fs)))
       (or
-       file
+       ;; use any previously used preference
+       (cdr (assoc fs diff-remembered-files-alist))
+       ;; try to be clever and use previous choices as an inspiration
+       (dolist (rf diff-remembered-files-alist)
+        (let ((newfile (diff-merge-strings (caar rf) (car fs) (cdr rf))))
+          (if (and newfile (file-exists-p newfile)) (return newfile))))
+       ;; look for each file in turn.  If none found, try again but
+       ;; ignoring the first level of directory, ...
+       (do* ((files fs (delq nil (mapcar 'diff-filename-drop-dir files)))
+            (file nil nil))
+          ((or (null files)
+               (setq file (do* ((files files (cdr files))
+                                (file (car files) (car files)))
+                              ((or (null file) (file-exists-p file))
+                               file))))
+           file))
+       ;; <foo>.rej patches implicitly apply to <foo>
        (and (string-match "\\.rej\\'" (or buffer-file-name ""))
            (let ((file (substring buffer-file-name 0 (match-beginning 0))))
              (when (file-exists-p file) file)))
-       ;; FIXME: use a more informative prompt
-       (let ((file (read-file-name "File: " nil (first fs) nil (first fs))))
-        ;; FIXME: remember for the next invocation
+       ;; if all else fails, ask the user
+       (let ((file (read-file-name (format "Use file %s: " (or (first fs) ""))
+                                  nil (first fs) t (first fs))))
+        (set (make-local-variable 'diff-remembered-files-alist)
+             (cons (cons fs file) diff-remembered-files-alist))
         file)))))
 
-(defun diff-goto-source (&optional other-file)
-  "Jump to the corresponding source line.
-`diff-jump-to-old-file-flag' (or its opposite if the OTHER-FILE prefix arg
-is give) determines whether to jump to the old or the new file.
-If the prefix arg is bigger than 8 (for example with \\[universal-argument] \\[universal-argument])
-  then `diff-jump-to-old-file-flag' is also set, for the next invokations."
-  (interactive "P")
+
+(defun diff-mouse-goto-source (event)
+  "Run `diff-goto-source' for the diff at a mouse click."
+  (interactive "e")
   (save-excursion
-    (let ((old (if (not other-file) diff-jump-to-old-file-flag
-                (not diff-jump-to-old-file-flag))))
-      (when (> (prefix-numeric-value other-file) 8)
-       (setq diff-jump-to-old-file-flag old))
-      (diff-beginning-of-hunk)
-      (let* ((loc (if (not (looking-at "[-@*\n ]*\\([0-9,]+\\)\\([ acd+]+\\([0-9,]+\\)\\)?"))
-                     (error "Can't find the hunk header")
-                   (if old (match-string 1)
-                     (if (match-end 3) (match-string 3)
-                       (unless (re-search-forward "^--- \\([0-9,]+\\)" nil t)
-                         (error "Can't find the hunk separator"))
-                       (match-string 1)))))
-            (lines (if (string-match "^\\([0-9]*\\),\\([0-9]*\\)" loc)
-                       (cons (string-to-number (match-string 1 loc))
-                             (string-to-number (match-string 2 loc)))
-                     (cons (string-to-number loc) nil)))
-            (file (diff-find-file-name old)))
-       (unless file (error "Can't find the file"))
-       (pop-to-buffer (find-file-noselect file))
-       (let* ((line (car lines))
-              (span (if (or (null (cdr lines)) (< (cdr lines) 0)) 0
-                      (if (< (cdr lines) line) (cdr lines)
-                        (- (cdr lines) line)))))
-         (ignore-errors
-           (goto-line line)
-           (forward-line span)
-           (push-mark (point) t t)
-           (goto-line line)))))))
+    (mouse-set-point event)
+    (diff-goto-source)))
 
 
 (defun diff-ediff-patch ()
   "Call `ediff-patch-file' on the current buffer."
   (interactive)
   (condition-case err
-      (ediff-patch-file (current-buffer))
+      (ediff-patch-file nil (current-buffer))
     (wrong-number-of-arguments (ediff-patch-file))))
 
 ;;;; 
@@ -464,11 +559,11 @@ else cover the whole bufer."
        (inhibit-read-only t))
     (save-excursion
       (goto-char start)
-      (while (and (re-search-forward "^\\(\\(---\\) .+\n\\(\\+\\+\\+\\) .+\\|@@ -\\([0-9]+\\),\\([0-9]+\\) \\+\\([0-9]+\\),\\([0-9]+\\) @@\\)$" nil t)
+      (while (and (re-search-forward "^\\(\\(---\\) .+\n\\(\\+\\+\\+\\) .+\\|@@ -\\([0-9]+\\),\\([0-9]+\\) \\+\\([0-9]+\\),\\([0-9]+\\) @@.*\\)$" nil t)
                  (< (point) end))
        (combine-after-change-calls
          (if (match-beginning 2)
-             ;;we matched a file header
+             ;; we matched a file header
              (progn
                ;; use reverse order to make sure the indices are kept valid
                (replace-match "---" t t nil 3)
@@ -551,7 +646,7 @@ else cover the whole bufer."
        (inhibit-read-only t))
     (save-excursion
       (goto-char start)
-      (while (and (re-search-forward "^\\(\\(\\*\\*\\*\\) .+\n\\(---\\) .+\\|\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\n\\*\\*\\* \\([0-9]+\\),\\(-?[0-9]+\\) \\*\\*\\*\\*\\)$" nil t)
+      (while (and (re-search-forward "^\\(\\(\\*\\*\\*\\) .+\n\\(---\\) .+\\|\\*\\{15\\}.*\n\\*\\*\\* \\([0-9]+\\),\\(-?[0-9]+\\) \\*\\*\\*\\*\\)$" nil t)
                  (< (point) end))
        (combine-after-change-calls
          (if (match-beginning 2)
@@ -623,7 +718,7 @@ else cover the whole bufer."
        (inhibit-read-only t))
     (save-excursion
       (goto-char start)
-      (while (and (re-search-forward "^\\(\\([-*][-*][-*] \\)\\(.+\\)\n\\([-+][-+][-+] \\)\\(.+\\)\\|\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\\*\n\\*\\*\\* \\(.+\\) \\*\\*\\*\\*\\|@@ -\\(.+\\) \\+\\(.+\\) @@\\)$" nil t)
+      (while (and (re-search-forward "^\\(\\([-*][-*][-*] \\)\\(.+\\)\n\\([-+][-+][-+] \\)\\(.+\\)\\|\\*\\{15\\}.*\n\\*\\*\\* \\(.+\\) \\*\\*\\*\\*\\|@@ -\\([0-9,]+\\) \\+\\([0-9,]+\\) @@.*\\)$" nil t)
                  (< (point) end))
        (combine-after-change-calls
          (cond
@@ -636,11 +731,10 @@ else cover the whole bufer."
              (replace-match "" nil nil nil 6)
              (forward-line 1)
              (let ((half1s (point)))
-               (while (looking-at "[-! \\][ \t]")
+               (while (looking-at "[-! \\][ \t]\\|#")
                  (when (= (char-after) ?-) (delete-char 1) (insert "+"))
                  (forward-line 1))
-               (let ((half1 (buffer-substring half1s (point))))
-                 (delete-region half1s (point))
+               (let ((half1 (delete-and-extract-region half1s (point))))
                  (unless (looking-at "^--- \\([0-9]+,-?[0-9]+\\) ----$")
                    (insert half1)
                    (error "Can't find matching `--- n1,n2 ----' line"))
@@ -648,14 +742,13 @@ else cover the whole bufer."
                    (replace-match lines1 nil nil nil 1)
                    (forward-line 1)
                    (let ((half2s (point)))
-                     (while (looking-at "[!+ \\][ \t]")
+                     (while (looking-at "[!+ \\][ \t]\\|#")
                        (when (= (char-after) ?+) (delete-char 1) (insert "-"))
                        (forward-line 1))
-                     (let ((half2 (buffer-substring half2s (point))))
-                       (delete-region half2s (point))
-                       (insert half1)
+                     (let ((half2 (delete-and-extract-region half2s (point))))
+                       (insert (or half1 ""))
                        (goto-char half1s)
-                       (insert half2)))
+                       (insert (or half2 ""))))
                    (goto-char pt-lines1)
                    (insert str1))))))
           ;; a unified-diff hunk header
@@ -668,10 +761,11 @@ else cover the whole bufer."
                           (delete-char 1) (insert "+") t)
                       (?+ (setq last (or last (point)))
                           (delete-char 1) (insert "-") t)
-                      (?\\ t)
+                      ((?\\ ?#) t)
                       (t (when (and first last (< first last))
-                           (let ((str (buffer-substring first last)))
-                             (save-excursion (delete-region first last))
+                           (let ((str
+                                  (save-excursion
+                                    (delete-and-extract-region first last))))
                              (insert str)))
                          (setq first nil last nil)
                          (equal ?\  c)))
@@ -689,16 +783,16 @@ else cover the whole bufer."
       (goto-char end) (diff-end-of-hunk)
       (let ((plus 0) (minus 0) (space 0) (bang 0))
        (while (and (= (forward-line -1) 0) (<= start (point)))
-         (if (not (looking-at "\\(@@ .+ @@\\|[-*][-*][-*] .+ [-*][-*][-*][-*]\\)$"))
+         (if (not (looking-at "\\(@@ -[0-9,]+ \\+[0-9,]+ @@.*\\|[-*][-*][-*] .+ [-*][-*][-*][-*]\\)$"))
              (case (char-after)
                (?\  (incf space))
                (?+ (incf plus))
                (?- (incf minus))
                (?! (incf bang))
-               (?\\ nil)
+               ((?\\ ?#) nil)
                (t  (setq space 0 plus 0 minus 0 bang 0)))
            (cond
-            ((looking-at "@@ -[0-9]+,\\([0-9]*\\) \\+[0-9]+,\\([0-9]*\\) @@$")
+            ((looking-at "@@ -[0-9]+,\\([0-9]*\\) \\+[0-9]+,\\([0-9]*\\) @@.*$")
              (let* ((old1 (match-string 1))
                     (old2 (match-string 2))
                     (new1 (number-to-string (+ space minus)))
@@ -730,9 +824,6 @@ else cover the whole bufer."
   (if (buffer-modified-p) (diff-fixup-modifs (point-min) (point-max)))
   nil)
 
-;; XEmacs doesn't seem to have this feature
-(defvar undo-in-progress nil)
-
 ;; It turns out that making changes in the buffer from within an
 ;; *-change-function is asking for trouble, whereas making them
 ;; from a post-command-hook doesn't pose much problems
@@ -740,6 +831,11 @@ else cover the whole bufer."
 (defun diff-after-change-function (beg end len)
   "Remember to fixup the hunk header.
 See `after-change-functions' for the meaning of BEG, END and LEN."
+  ;; Ignoring changes when inhibit-read-only is set is strictly speaking
+  ;; incorrect, but it turns out that inhibit-read-only is normally not set
+  ;; inside editing commands, while it tends to be set when the buffer gets
+  ;; updated by an async process or by a conversion function, both of which
+  ;; would rather not be uselessly slowed down by this hook.
   (when (and (not undo-in-progress) (not inhibit-read-only))
     (if diff-unhandled-changes
        (setq diff-unhandled-changes
@@ -752,7 +848,15 @@ See `after-change-functions' for the meaning of BEG, END and LEN."
   (when (consp diff-unhandled-changes)
     (ignore-errors
       (save-excursion
-       (goto-char (car diff-unhandled-changes)) (diff-beginning-of-hunk)
+       (goto-char (car diff-unhandled-changes))
+       (unless (ignore-errors
+                 (diff-beginning-of-hunk)
+                 (save-excursion
+                   (diff-end-of-hunk)
+                   (> (point) (car diff-unhandled-changes))))
+         (goto-char (car diff-unhandled-changes))
+         (re-search-forward diff-hunk-header-re (cdr diff-unhandled-changes))
+         (diff-beginning-of-hunk))
        (diff-fixup-modifs (point) (cdr diff-unhandled-changes))))
     (setq diff-unhandled-changes nil)))
 
@@ -760,53 +864,319 @@ See `after-change-functions' for the meaning of BEG, END and LEN."
 ;;;; The main function
 ;;;; 
 
-;;(autoload 'diff-mode "diff-mode" "Major mode for viewing context diffs." t)
 ;;;###autoload
-(defun diff-mode ()
-  "Major mode for viewing context diffs.
-Supports unified and context diffs as well as (to a lesser extent) normal diffs.
-When the buffer is read-only, the ESC prefix is not necessary.
-This mode runs `diff-mode-hook'.
-\\{diff-mode-map}"
-  (interactive)
-  (kill-all-local-variables)
-  (setq major-mode 'diff-mode)
-  (setq mode-name "Diff")
-  (use-local-map diff-mode-map)
+(define-derived-mode diff-mode fundamental-mode "Diff"
+  "Major mode for viewing/editing context diffs.
+Supports unified and context diffs as well as (to a lesser extent)
+normal diffs.
+When the buffer is read-only, the ESC prefix is not necessary."
   (set (make-local-variable 'font-lock-defaults) diff-font-lock-defaults)
   (set (make-local-variable 'outline-regexp) diff-outline-regexp)
+  (set (make-local-variable 'imenu-generic-expression)
+       diff-imenu-generic-expression)
+  ;; These are not perfect.  They would be better done separately for
+  ;; context diffs and unidiffs.
+  ;; (set (make-local-variable 'paragraph-start)
+  ;;        (concat "@@ "                      ; unidiff hunk
+  ;;          "\\|\\*\\*\\* "          ; context diff hunk or file start
+  ;;          "\\|--- [^\t]+\t"))      ; context or unidiff file
+  ;;                                   ; start (first or second line)
+  ;;   (set (make-local-variable 'paragraph-separate) paragraph-start)
+  ;;   (set (make-local-variable 'page-delimiter) "--- [^\t]+\t")
   ;; compile support
-  (set (make-local-variable 'compilation-file-regexp-alist)
-       diff-file-regexp-alist)
-  (set (make-local-variable 'compilation-error-regexp-alist)
-       diff-error-regexp-alist)
-  (when (string-match "\\.rej\\'" (or buffer-file-name ""))
-    (set (make-local-variable 'compilation-current-file)
-        (substring buffer-file-name 0 (match-beginning 0))))
-  (compilation-shell-minor-mode 1)
-  ;; 
-  (setq buffer-read-only t)
-  (if (not diff-update-on-the-fly-flag)
+
+  ;;;; compile support is not good enough yet.  Also it can be annoying
+  ;; and should thus only be enabled conditionally.
+  ;; (set (make-local-variable 'compilation-file-regexp-alist)
+  ;;      diff-file-regexp-alist)
+  ;; (set (make-local-variable 'compilation-error-regexp-alist)
+  ;;      diff-error-regexp-alist)
+  ;; (when (string-match "\\.rej\\'" (or buffer-file-name ""))
+  ;;   (set (make-local-variable 'compilation-current-file)
+  ;;    (substring buffer-file-name 0 (match-beginning 0))))
+  ;; (compilation-shell-minor-mode 1)
+
+  (when (and (> (point-max) (point-min)) diff-default-read-only)
+    (toggle-read-only t))
+  ;; setup change hooks
+  (if (not diff-update-on-the-fly)
       (add-hook 'write-contents-hooks 'diff-write-contents-hooks)
     (make-local-variable 'diff-unhandled-changes)
-    (make-local-hook 'after-change-functions)
     (add-hook 'after-change-functions 'diff-after-change-function nil t)
-    (make-local-hook 'post-command-hook)
     (add-hook 'post-command-hook 'diff-post-command-hook nil t))
   ;; Neat trick from Dave Love to add more bindings in read-only mode:
-  (add-to-list (make-local-variable 'minor-mode-map-alist)
-              (cons 'buffer-read-only diff-mode-shared-map))
-  ;;
-  (run-hooks 'diff-mode-hook))
+  (add-to-list (make-local-variable 'minor-mode-overriding-map-alist)
+              (cons 'buffer-read-only diff-mode-shared-map))
+  ;; add-log support
+  (set (make-local-variable 'add-log-current-defun-function)
+       'diff-current-defun)
+  (set (make-local-variable 'add-log-buffer-file-name-function)
+       'diff-find-file-name))
 
 ;;;###autoload
-(add-to-list 'auto-mode-alist '("\\.\\(diffs?\\|patch\\|rej\\)\\'" . diff-mode))
+(define-minor-mode diff-minor-mode
+  "Minor mode for viewing/editing context diffs.
+\\{diff-minor-mode-map}"
+  nil " Diff" nil
+  ;; FIXME: setup font-lock
+  ;; setup change hooks
+  (if (not diff-update-on-the-fly)
+      (add-hook 'write-contents-hooks 'diff-write-contents-hooks)
+    (make-local-variable 'diff-unhandled-changes)
+    (add-hook 'after-change-functions 'diff-after-change-function nil t)
+    (add-hook 'post-command-hook 'diff-post-command-hook nil t)))
+
+
+;;;
+;;; Misc operations that have proved useful at some point.
+;;;
+
+(defun diff-next-complex-hunk ()
+  "Jump to the next \"complex\" hunk.
+\"Complex\" is approximated by \"the hunk changes the number of lines\".
+Only works for unified diffs."
+  (interactive)
+  (while
+      (and (re-search-forward "^@@ [-0-9]+,\\([0-9]+\\) [+0-9]+,\\([0-9]+\\) @@"
+                             nil t)
+          (equal (match-string 1) (match-string 2)))))
+
+(defun diff-hunk-text (hunk destp &optional char-offset)
+  "Return the literal source text from HUNK.
+if DESTP is nil return the source, otherwise the destination text.
+If CHAR-OFFSET is non-nil, it should be a char-offset in
+HUNK, and instead of a string, a cons cell is returned whose car is the
+appropriate text, and whose cdr is the corresponding char-offset in that text."
+  (with-temp-buffer
+    (insert hunk)
+    (goto-char (point-min))
+    (let ((src-pos nil)
+         (dst-pos nil)
+         (divider-pos nil)
+         (num-pfx-chars 2))
+      ;; Set the following variables:
+      ;;  SRC-POS     buffer pos of the source part of the hunk or nil if none
+      ;;  DST-POS     buffer pos of the destination part of the hunk or nil
+      ;;  DIVIDER-POS buffer pos of any divider line separating the src & dst
+      ;;  NUM-PFX-CHARS  number of line-prefix characters used by this format"
+      (cond ((looking-at "^@@")
+            ;; unified diff
+            (setq num-pfx-chars 1)
+            (forward-line 1)
+            (setq src-pos (point) dst-pos (point)))
+           ((looking-at "^\\*\\*")
+            ;; context diff
+            (forward-line 2)
+            (setq src-pos (point))
+            (re-search-forward "^--- " nil t)
+            (forward-line 0)
+            (setq divider-pos (point))
+            (forward-line 1)
+            (setq dst-pos (point)))
+           ((looking-at "^[0-9]+a[0-9,]+$")
+            ;; normal diff, insert
+            (forward-line 1)
+            (setq dst-pos (point)))
+           ((looking-at "^[0-9,]+d[0-9]+$")
+            ;; normal diff, delete
+            (forward-line 1)
+            (setq src-pos (point)))
+           ((looking-at "^[0-9,]+c[0-9,]+$")
+            ;; normal diff, change
+            (forward-line 1)
+            (setq src-pos (point))
+            (re-search-forward "^---$" nil t)
+            (forward-line 0)
+            (setq divider-pos (point))
+            (forward-line 1)
+            (setq dst-pos (point)))
+           (t
+            (error "Unknown diff hunk type")))
+
+      (if (if destp (null dst-pos) (null src-pos))
+         ;; Implied empty text
+         (if char-offset '("" . 0) "")
+
+       ;; For context diffs, either side can be empty, (if there's only
+       ;; added or only removed text).  We should then use the other side.
+       (cond ((equal src-pos divider-pos) (setq src-pos dst-pos))
+             ((equal dst-pos (point-max)) (setq dst-pos src-pos)))
+
+       (when char-offset (goto-char (+ (point-min) char-offset)))
+
+       ;; Get rid of anything except the desired text.
+       (save-excursion
+         ;; Delete unused text region
+         (let ((keep (if destp dst-pos src-pos)))
+           (when (and divider-pos (> divider-pos keep))
+             (delete-region divider-pos (point-max)))
+           (delete-region (point-min) keep))
+         ;; Remove line-prefix characters, and unneeded lines (unified diffs).
+         (let ((kill-char (if destp ?- ?+)))
+           (goto-char (point-min))
+           (while (not (eobp))
+             (if (eq (char-after) kill-char)
+                 (delete-region (point) (progn (forward-line 1) (point)))
+               (delete-char num-pfx-chars)
+               (forward-line 1)))))
+
+       (let ((text (buffer-substring-no-properties (point-min) (point-max))))
+         (if char-offset (cons text (- (point) (point-min))) text))))))
+
+
+(defun diff-find-text (text)
+  "Return the buffer position of the nearest occurrence of TEXT.
+If TEXT isn't found, nil is returned."
+  (let* ((orig (point))
+        (forw (and (search-forward text nil t)
+                         (match-beginning 0)))
+        (back (and (goto-char (+ orig (length text)))
+                   (search-backward text nil t)
+                         (match-beginning 0))))
+         ;; Choose the closest match.
+    (if (and forw back)
+       (if (> (- forw orig) (- orig back)) back forw)
+      (or back forw))))
+
+(defsubst diff-xor (a b) (if a (not b) b))
+
+(defun diff-find-source-location (&optional other-file reverse)
+  "Find out (BUF LINE-OFFSET POS SRC DST SWITCHED)."
+  (save-excursion
+    (let* ((other (diff-xor other-file diff-jump-to-old-file))
+          (char-offset (- (point) (progn (diff-beginning-of-hunk) (point))))
+          (hunk (buffer-substring (point)
+                                  (save-excursion (diff-end-of-hunk) (point))))
+          (old (diff-hunk-text hunk reverse char-offset))
+          (new (diff-hunk-text hunk (not reverse) char-offset))
+          ;; Find the location specification.
+          (line (if (not (looking-at "\\(?:\\*\\{15\\}.*\n\\)?[-@* ]*\\([0-9,]+\\)\\([ acd+]+\\([0-9,]+\\)\\)?"))
+                    (error "Can't find the hunk header")
+                  (if other (match-string 1)
+                    (if (match-end 3) (match-string 3)
+                      (unless (re-search-forward "^--- \\([0-9,]+\\)" nil t)
+                        (error "Can't find the hunk separator"))
+                      (match-string 1)))))
+          (file (or (diff-find-file-name other) (error "Can't find the file")))
+          (buf (find-file-noselect file)))
+      ;; Update the user preference if he so wished.
+      (when (> (prefix-numeric-value other-file) 8)
+       (setq diff-jump-to-old-file other))
+      (with-current-buffer buf
+       (goto-line (string-to-number line))
+       (let* ((orig-pos (point))
+              (pos (diff-find-text (car old)))
+              (switched nil))
+         (when (null pos)
+           (setq pos (diff-find-text (car new)) switched t))
+         (nconc
+          (list buf)
+          (if pos (list (count-lines orig-pos pos) pos) (list nil orig-pos))
+          (if switched (list new old t) (list old new))))))))
+
+
+(defun diff-hunk-status-msg (line-offset reversed dry-run)
+  (let ((msg (if dry-run
+                (if reversed "already applied" "not yet applied")
+              (if reversed "undone" "applied"))))
+    (message (cond ((null line-offset) "Hunk text not found")
+                  ((= line-offset 0) "Hunk %s")
+                  ((= line-offset 1) "Hunk %s at offset %d line")
+                  (t "Hunk %s at offset %d lines"))
+            msg line-offset)))
+
+
+(defun diff-apply-hunk (&optional reverse)
+  "Apply the current hunk to the source file and go to the next.
+By default, the new source file is patched, but if the variable
+`diff-jump-to-old-file' is non-nil, then the old source file is
+patched instead (some commands, such as `diff-goto-source' can change
+the value of this variable when given an appropriate prefix argument).
+
+With a prefix argument, REVERSE the hunk."
+  (interactive "P")
+  (destructuring-bind (buf line-offset pos old new &optional switched)
+      (diff-find-source-location nil reverse)
+    (cond
+     ((null line-offset)
+      (error "Can't find the text to patch"))
+     ((and switched
+          ;; A reversed patch was detected, perhaps apply it in reverse.
+          (not (save-window-excursion
+                 (pop-to-buffer buf)
+                 (goto-char (+ pos (cdr old)))
+                 (y-or-n-p
+                  (if reverse
+                      "Hunk hasn't been applied yet; apply it now? "
+                    "Hunk has already been applied; undo it? ")))))
+      (message "(Nothing done)"))
+     (t
+      ;; Apply the hunk
+      (with-current-buffer buf
+       (goto-char pos)
+       (delete-char (length (car old)))
+       (insert (car new)))
+      ;; Display BUF in a window
+      (set-window-point (display-buffer buf) (+ pos (cdr new)))
+      (diff-hunk-status-msg line-offset (diff-xor switched reverse) nil)
+      (when diff-advance-after-apply-hunk
+       (diff-hunk-next))))))
+
+
+(defun diff-test-hunk (&optional reverse)
+  "See whether it's possible to apply the current hunk.
+With a prefix argument, try to REVERSE the hunk."
+  (interactive "P")
+  (destructuring-bind (buf line-offset pos src dst &optional switched)
+      (diff-find-source-location nil reverse)
+    (set-window-point (display-buffer buf) (+ pos (cdr src)))
+    (diff-hunk-status-msg line-offset (diff-xor reverse switched) t)))
+
+
+(defun diff-goto-source (&optional other-file)
+  "Jump to the corresponding source line.
+`diff-jump-to-old-file' (or its opposite if the OTHER-FILE prefix arg
+is given) determines whether to jump to the old or the new file.
+If the prefix arg is bigger than 8 (for example with \\[universal-argument] \\[universal-argument])
+  then `diff-jump-to-old-file' is also set, for the next invocations."
+  (interactive "P")
+  ;; When pointing at a removal line, we probably want to jump to
+  ;; the old location, and else to the new (i.e. as if reverting).
+  ;; This is a convenient detail when using smerge-diff.
+  (let ((rev (not (save-excursion (beginning-of-line) (looking-at "[-<]")))))
+    (destructuring-bind (buf line-offset pos src dst &optional switched)
+       (diff-find-source-location other-file rev)
+      (pop-to-buffer buf)
+      (goto-char (+ pos (cdr src)))
+      (diff-hunk-status-msg line-offset (diff-xor rev switched) t))))
+
+
+(defun diff-current-defun ()
+  "Find the name of function at point.
+For use in `add-log-current-defun-function'."
+  (destructuring-bind (buf line-offset pos src dst &optional switched)
+      (diff-find-source-location)
+    (save-excursion
+      (beginning-of-line)
+      (or (when (memq (char-after) '(?< ?-))
+           ;; Cursor is pointing at removed text.  This could be a removed
+           ;; function, in which case, going to the source buffer will
+           ;; not help since the function is now removed.  Instead,
+           ;; try to figure out the function name just from the code-fragment.
+           (let ((old (if switched dst src)))
+             (with-temp-buffer
+               (insert (car old))
+               (goto-char (cdr old))
+               (funcall (with-current-buffer buf major-mode))
+               (add-log-current-defun))))
+         (with-current-buffer buf
+           (goto-char (+ pos (cdr src)))
+           (add-log-current-defun))))))
 
 ;; provide the package
 (provide 'diff-mode)
 
-;;; Change Log:
-;; diff-mode.el,v
+;;; Old Change Log from when diff-mode wasn't part of Emacs:
 ;; Revision 1.11  1999/10/09 23:38:29  monnier
 ;; (diff-mode-load-hook): dropped.
 ;; (auto-mode-alist): also catch *.diffs.