Add arch taglines
[bpt/emacs.git] / lisp / emacs-lisp / debug.el
index ed5f1ae..6e10b59 100644 (file)
@@ -1,6 +1,6 @@
 ;;; debug.el --- debuggers and related commands for Emacs
 
-;; Copyright (C) 1985, 1986, 1994 Free Software Foundation, Inc.
+;; Copyright (C) 1985, 1986, 1994, 2001, 2003  Free Software Foundation, Inc.
 
 ;; Maintainer: FSF
 ;; Keywords: lisp, tools, maint
@@ -28,6 +28,8 @@
 
 ;;; Code:
 
+(require 'button)
+
 (defgroup debugger nil
   "Debuggers and related commands for Emacs."
   :prefix "debugger-"
   :group 'debugger
   :version "20.3")
 
+(defcustom debugger-batch-max-lines 40
+  "*Maximum lines to show in debugger buffer in a noninteractive Emacs.
+When the debugger is entered and Emacs is running in batch mode,
+if the backtrace text has more than this many lines,
+the middle is discarded, and just the beginning and end are displayed."
+  :type 'integer
+  :group 'debugger
+  :version "21.1")
 
 (defcustom debug-function-list nil
   "List of functions currently set for debug on entry."
 (defvar debugger-old-buffer nil
   "This is the buffer that was current when the debugger was entered.")
 
+(defvar debugger-previous-backtrace nil
+  "The contents of the previous backtrace (including text properties).
+This is to optimize `debugger-make-xrefs'.")
+
 (defvar debugger-outer-match-data)
 (defvar debugger-outer-load-read-function)
 (defvar debugger-outer-overriding-local-map)
 (defvar debugger-outer-track-mouse)
 (defvar debugger-outer-last-command)
 (defvar debugger-outer-this-command)
+;; unread-command-char is obsolete,
+;; but we still save and restore it
+;; in case some user program still tries to set it.
 (defvar debugger-outer-unread-command-char)
 (defvar debugger-outer-unread-command-events)
+(defvar debugger-outer-unread-post-input-method-events)
 (defvar debugger-outer-last-input-event)
 (defvar debugger-outer-last-command-event)
 (defvar debugger-outer-last-nonmenu-event)
 (defvar debugger-outer-last-event-frame)
 (defvar debugger-outer-standard-input)
 (defvar debugger-outer-standard-output)
+(defvar debugger-outer-inhibit-redisplay)
 (defvar debugger-outer-cursor-in-echo-area)
 
 ;;;###autoload
@@ -85,144 +104,264 @@ You may call with no args, or you may pass nil as the first arg and
 any other args you like.  In that case, the list of args after the
 first will be printed into the backtrace buffer."
   (interactive)
-  (message "Entering debugger...")
-  (let (debugger-value
-       (debug-on-error nil)
-       (debug-on-quit nil)
-       (debugger-buffer (let ((default-major-mode 'fundamental-mode))
-                          (get-buffer-create "*Backtrace*")))
-       (debugger-old-buffer (current-buffer))
-       (debugger-step-after-exit nil)
-       ;; Don't keep reading from an executing kbd macro!
-       (executing-kbd-macro nil)
-       ;; Save the outer values of these vars for the `e' command
-       ;; before we replace the values.
-       (debugger-outer-match-data (match-data))
-       (debugger-outer-load-read-function load-read-function)
-       (debugger-outer-overriding-local-map overriding-local-map)
-       (debugger-outer-overriding-terminal-local-map
-        overriding-terminal-local-map)
-       (debugger-outer-track-mouse track-mouse)
-       (debugger-outer-last-command last-command)
-       (debugger-outer-this-command this-command)
-       (debugger-outer-unread-command-char unread-command-char)
-       (debugger-outer-unread-command-events unread-command-events)
-       (debugger-outer-last-input-event last-input-event)
-       (debugger-outer-last-command-event last-command-event)
-       (debugger-outer-last-nonmenu-event last-nonmenu-event)
-       (debugger-outer-last-event-frame last-event-frame)
-       (debugger-outer-standard-input standard-input)
-       (debugger-outer-standard-output standard-output)
-       (debugger-outer-cursor-in-echo-area cursor-in-echo-area))
-    ;; Set this instead of binding it, so that `q'
-    ;; will not restore it.
-    (setq overriding-terminal-local-map nil) 
-    ;; Don't let these magic variables affect the debugger itself.
-    (let ((last-command nil) this-command track-mouse
-         (unread-command-char -1) unread-command-events
-         last-input-event last-command-event last-nonmenu-event
-         last-event-frame
-         overriding-local-map
-         load-read-function
-         ;; If we are inside a minibuffer, allow nesting
-         ;; so that we don't get an error from the `e' command.
-         (enable-recursive-minibuffers (> (minibuffer-depth) 0))
-         (standard-input t) (standard-output t)
-         (cursor-in-echo-area nil))
-      (unwind-protect
-         (save-excursion
-           (save-window-excursion
-             (pop-to-buffer debugger-buffer)
-             (erase-buffer)
-             (set-buffer-multibyte nil)
-             (let ((standard-output (current-buffer))
-                   (print-escape-newlines t)
-                   (print-length 50))
-               (backtrace))
-             (goto-char (point-min))
-             (debugger-mode)
-             (delete-region (point)
-                            (progn
-                              (search-forward "\n  debug(")
-                              (forward-line 1)
-                              (point)))
-             (debugger-reenable)
-             ;; lambda is for debug-on-call when a function call is next.
-             ;; debug is for debug-on-entry function called.
-             (cond ((memq (car debugger-args) '(lambda debug))
-                    (insert "Entering:\n")
-                    (if (eq (car debugger-args) 'debug)
-                        (progn
-                          ;; Skip the frames for backtrace-debug, byte-code,
-                          ;; and debug.
-                          (backtrace-debug 3 t)
-                          (delete-char 1)
-                          (insert ?*)
-                          (beginning-of-line))))
-                   ;; Exiting a function.
-                   ((eq (car debugger-args) 'exit)
-                    (insert "Return value: ")
-                    (setq debugger-value (nth 1 debugger-args))
-                    (prin1 debugger-value (current-buffer))
-                    (insert ?\n)
-                    (delete-char 1)
-                    (insert ? )
-                    (beginning-of-line))
-                   ;; Debugger entered for an error.
-                   ((eq (car debugger-args) 'error)
-                    (insert "Signaling: ")
-                    (prin1 (nth 1 debugger-args) (current-buffer))
-                    (insert ?\n))
-                   ;; debug-on-call, when the next thing is an eval.
-                   ((eq (car debugger-args) t)
-                    (insert "Beginning evaluation of function call form:\n"))
-                   ;; User calls debug directly.
-                   (t
-                    (prin1 (if (eq (car debugger-args) 'nil)
-                               (cdr debugger-args) debugger-args)
-                           (current-buffer))
-                    (insert ?\n)))
-             (message "")
-             (let ((inhibit-trace t)
-                   (standard-output nil)
-                   (buffer-read-only t))
-               (message "")
-               ;; Make sure we unbind buffer-read-only in the right buffer.
-               (save-excursion
-                 (recursive-edit)))))
-       ;; Kill or at least neuter the backtrace buffer, so that users
-       ;; don't try to execute debugger commands in an invalid context.
-       (if (get-buffer-window debugger-buffer 'visible)
-           ;; Still visible despite the save-window-excursion?  Maybe it
-           ;; it's in a pop-up frame.  It would be annoying to delete and
-           ;; recreate it every time the debugger stops, so instead we'll
-           ;; erase it but leave it visible.
+  (if inhibit-redisplay
+      ;; Don't really try to enter debugger within an eval from redisplay.
+      debugger-value
+    (unless noninteractive
+      (message "Entering debugger..."))
+    (let (debugger-value
+         (debug-on-error nil)
+         (debug-on-quit nil)
+         (debugger-buffer (let ((default-major-mode 'fundamental-mode))
+                            (get-buffer-create "*Backtrace*")))
+         (debugger-old-buffer (current-buffer))
+         (debugger-step-after-exit nil)
+         ;; Don't keep reading from an executing kbd macro!
+         (executing-kbd-macro nil)
+         ;; Save the outer values of these vars for the `e' command
+         ;; before we replace the values.
+         (debugger-outer-match-data (match-data))
+         (debugger-outer-load-read-function load-read-function)
+         (debugger-outer-overriding-local-map overriding-local-map)
+         (debugger-outer-overriding-terminal-local-map
+          overriding-terminal-local-map)
+         (debugger-outer-track-mouse track-mouse)
+         (debugger-outer-last-command last-command)
+         (debugger-outer-this-command this-command)
+         (debugger-outer-unread-command-char
+          (with-no-warnings unread-command-char))
+         (debugger-outer-unread-command-events unread-command-events)
+         (debugger-outer-unread-post-input-method-events
+          unread-post-input-method-events)
+         (debugger-outer-last-input-event last-input-event)
+         (debugger-outer-last-command-event last-command-event)
+         (debugger-outer-last-nonmenu-event last-nonmenu-event)
+         (debugger-outer-last-event-frame last-event-frame)
+         (debugger-outer-standard-input standard-input)
+         (debugger-outer-standard-output standard-output)
+         (debugger-outer-inhibit-redisplay inhibit-redisplay)
+         (debugger-outer-cursor-in-echo-area cursor-in-echo-area))
+      ;; Set this instead of binding it, so that `q'
+      ;; will not restore it.
+      (setq overriding-terminal-local-map nil)
+      ;; Don't let these magic variables affect the debugger itself.
+      (let ((last-command nil) this-command track-mouse
+           unread-command-events
+           unread-post-input-method-events
+           last-input-event last-command-event last-nonmenu-event
+           last-event-frame
+           overriding-local-map
+           load-read-function
+           ;; If we are inside a minibuffer, allow nesting
+           ;; so that we don't get an error from the `e' command.
+           (enable-recursive-minibuffers
+            (or enable-recursive-minibuffers (> (minibuffer-depth) 0)))
+           (standard-input t) (standard-output t)
+           inhibit-redisplay
+           (cursor-in-echo-area nil))
+       (unwind-protect
            (save-excursion
-             (set-buffer debugger-buffer)
-             (erase-buffer)
-             (fundamental-mode))
-         (kill-buffer debugger-buffer))
-       (set-match-data debugger-outer-match-data)))
-    ;; Put into effect the modified values of these variables
-    ;; in case the user set them with the `e' command.
-    (setq load-read-function debugger-outer-load-read-function)
-    (setq overriding-local-map debugger-outer-overriding-local-map)
-    (setq overriding-terminal-local-map
-         debugger-outer-overriding-terminal-local-map)
-    (setq track-mouse debugger-outer-track-mouse)
-    (setq last-command debugger-outer-last-command)
-    (setq this-command debugger-outer-this-command)
-    (setq unread-command-char debugger-outer-unread-command-char)
-    (setq unread-command-events debugger-outer-unread-command-events)
-    (setq last-input-event debugger-outer-last-input-event)
-    (setq last-command-event debugger-outer-last-command-event)
-    (setq last-nonmenu-event debugger-outer-last-nonmenu-event)
-    (setq last-event-frame debugger-outer-last-event-frame)
-    (setq standard-input debugger-outer-standard-input)
-    (setq standard-output debugger-outer-standard-output)
-    (setq cursor-in-echo-area debugger-outer-cursor-in-echo-area)
-    (setq debug-on-next-call debugger-step-after-exit)
-    debugger-value))
+             (save-window-excursion
+               (with-no-warnings
+                (setq unread-command-char -1))
+               (pop-to-buffer debugger-buffer)
+               (debugger-mode)
+               (debugger-setup-buffer debugger-args)
+               (when noninteractive
+                 ;; If the backtrace is long, save the beginning
+                 ;; and the end, but discard the middle.
+                 (when (> (count-lines (point-min) (point-max))
+                          debugger-batch-max-lines)
+                   (goto-char (point-min))
+                   (forward-line (/ 2 debugger-batch-max-lines))
+                   (let ((middlestart (point)))
+                     (goto-char (point-max))
+                     (forward-line (- (/ 2 debugger-batch-max-lines)
+                                      debugger-batch-max-lines))
+                     (delete-region middlestart (point)))
+                   (insert "...\n"))
+                 (goto-char (point-min))
+                 (message "%s" (buffer-string))
+                 (kill-emacs))
+               (if (eq (car debugger-args) 'debug)
+                   ;; Skip the frames for backtrace-debug, byte-code, and debug.
+                   (backtrace-debug 3 t))
+               (debugger-reenable)
+               (message "")
+               (let ((inhibit-trace t)
+                     (standard-output nil)
+                     (buffer-read-only t))
+                 (message "")
+                 ;; Make sure we unbind buffer-read-only in the right buffer.
+                 (save-excursion
+                   (recursive-edit)))))
+         ;; Kill or at least neuter the backtrace buffer, so that users
+         ;; don't try to execute debugger commands in an invalid context.
+         (if (get-buffer-window debugger-buffer 'visible)
+             ;; Still visible despite the save-window-excursion?  Maybe it
+             ;; it's in a pop-up frame.  It would be annoying to delete and
+             ;; recreate it every time the debugger stops, so instead we'll
+             ;; erase it but leave it visible.
+             (save-excursion
+               (set-buffer debugger-buffer)
+               (erase-buffer)
+               (fundamental-mode))
+           (kill-buffer debugger-buffer))
+         (set-match-data debugger-outer-match-data)))
+      ;; Put into effect the modified values of these variables
+      ;; in case the user set them with the `e' command.
+      (setq load-read-function debugger-outer-load-read-function)
+      (setq overriding-local-map debugger-outer-overriding-local-map)
+      (setq overriding-terminal-local-map
+           debugger-outer-overriding-terminal-local-map)
+      (setq track-mouse debugger-outer-track-mouse)
+      (setq last-command debugger-outer-last-command)
+      (setq this-command debugger-outer-this-command)
+      (with-no-warnings
+       (setq unread-command-char debugger-outer-unread-command-char))
+      (setq unread-command-events debugger-outer-unread-command-events)
+      (setq unread-post-input-method-events
+           debugger-outer-unread-post-input-method-events)
+      (setq last-input-event debugger-outer-last-input-event)
+      (setq last-command-event debugger-outer-last-command-event)
+      (setq last-nonmenu-event debugger-outer-last-nonmenu-event)
+      (setq last-event-frame debugger-outer-last-event-frame)
+      (setq standard-input debugger-outer-standard-input)
+      (setq standard-output debugger-outer-standard-output)
+      (setq inhibit-redisplay debugger-outer-inhibit-redisplay)
+      (setq cursor-in-echo-area debugger-outer-cursor-in-echo-area)
+      (setq debug-on-next-call debugger-step-after-exit)
+      debugger-value)))
+\f
+(defun debugger-setup-buffer (debugger-args)
+  "Initialize the `*Backtrace*' buffer for entry to the debugger.
+That buffer should be current already."
+  (setq buffer-read-only nil)
+  (erase-buffer)
+  (set-buffer-multibyte nil)
+  (let ((standard-output (current-buffer))
+       (print-escape-newlines t)
+       (print-level 8)
+       (print-length 50))
+    (backtrace))
+  (goto-char (point-min))
+  (delete-region (point)
+                (progn
+                  (search-forward "\n  debug(")
+                  (forward-line 1)
+                  (point)))
+  (insert "Debugger entered")
+  ;; lambda is for debug-on-call when a function call is next.
+  ;; debug is for debug-on-entry function called.
+  (cond ((memq (car debugger-args) '(lambda debug))
+        (insert "--entering a function:\n")
+        (if (eq (car debugger-args) 'debug)
+            (progn
+              (delete-char 1)
+              (insert ?*)
+              (beginning-of-line))))
+       ;; Exiting a function.
+       ((eq (car debugger-args) 'exit)
+        (insert "--returning value: ")
+        (setq debugger-value (nth 1 debugger-args))
+        (prin1 debugger-value (current-buffer))
+        (insert ?\n)
+        (delete-char 1)
+        (insert ? )
+        (beginning-of-line))
+       ;; Debugger entered for an error.
+       ((eq (car debugger-args) 'error)
+        (insert "--Lisp error: ")
+        (prin1 (nth 1 debugger-args) (current-buffer))
+        (insert ?\n))
+       ;; debug-on-call, when the next thing is an eval.
+       ((eq (car debugger-args) t)
+        (insert "--beginning evaluation of function call form:\n"))
+       ;; User calls debug directly.
+       (t
+        (insert ": ")
+        (prin1 (if (eq (car debugger-args) 'nil)
+                   (cdr debugger-args) debugger-args)
+               (current-buffer))
+        (insert ?\n)))
+  ;; After any frame that uses eval-buffer,
+  ;; insert a line that states the buffer position it's reading at.
+  (save-excursion
+    (while (re-search-forward "^  eval-buffer(" nil t)
+      (end-of-line)
+      (insert (format "\n  ;;; Reading at buffer position %d"
+                     (with-current-buffer (nth 2 (backtrace-frame (debugger-frame-number)))
+                       (point))))))
+  (debugger-make-xrefs))
+
+(defun debugger-make-xrefs (&optional buffer)
+  "Attach cross-references to symbol names in the `*Backtrace*' buffer."
+  (interactive "b")
+  (save-excursion
+    (set-buffer (or buffer (current-buffer)))
+    (setq buffer (current-buffer))
+    (let ((buffer-read-only nil)
+         (old-end (point-min)) (new-end (point-min)))
+      ;; If we saved an old backtrace, find the common part
+      ;; between the new and the old.
+      ;; Compare line by line, starting from the end,
+      ;; because that's the part that is likely to be unchanged.
+      (if debugger-previous-backtrace
+         (let (old-start new-start (all-match t))
+           (goto-char (point-max))
+           (with-temp-buffer
+             (insert debugger-previous-backtrace)
+             (while (and all-match (not (bobp)))
+               (setq old-end (point))
+               (forward-line -1)
+               (setq old-start (point))
+               (with-current-buffer buffer
+                 (setq new-end (point))
+                 (forward-line -1)
+                 (setq new-start (point)))
+               (if (not (zerop
+                         (compare-buffer-substrings
+                          (current-buffer) old-start old-end
+                          buffer new-start new-end)))
+                   (setq all-match nil))))
+           ;; Now new-end is the position of the start of the
+           ;; unchanged part in the current buffer, and old-end is
+           ;; the position of that same text in the saved old
+           ;; backtrace.  But we must subtract (point-min) since strings are
+           ;; indexed in origin 0.
+
+           ;; Replace the unchanged part of the backtrace
+           ;; with the text from debugger-previous-backtrace,
+           ;; since that already has the proper xrefs.
+           ;; With this optimization, we only need to scan
+           ;; the changed part of the backtrace.
+           (delete-region new-end (point-max))
+           (goto-char (point-max))
+           (insert (substring debugger-previous-backtrace
+                              (- old-end (point-min))))
+           ;; Make the unchanged part of the backtrace inaccessible
+           ;; so it won't be scanned.
+           (narrow-to-region (point-min) new-end)))
+
+      ;; Scan the new part of the backtrace, inserting xrefs.
+      (goto-char (point-min))
+      (while (progn
+              (skip-syntax-forward "^w_")
+              (not (eobp)))
+       (let* ((beg (point))
+              (end (progn (skip-syntax-forward "w_") (point)))
+              (sym (intern-soft (buffer-substring-no-properties
+                                 beg end)))
+              (file (and sym (symbol-file sym))))
+         (when file
+           (goto-char beg)
+           ;; help-xref-button needs to operate on something matched
+           ;; by a regexp, so set that up for it.
+           (re-search-forward "\\(\\(\\sw\\|\\s_\\)+\\)")
+           (help-xref-button 1 'help-function-def sym file)))
+       (forward-line 1))
+      (widen))
+    (setq debugger-previous-backtrace (buffer-string))))
 \f
 (defun debugger-step-through ()
   "Proceed, stepping through subexpressions of this expression.
@@ -235,6 +374,8 @@ Enter another debugger on next entry to eval, apply or funcall."
 (defun debugger-continue ()
   "Continue, evaluating this expression without stopping."
   (interactive)
+  (unless debugger-may-continue
+    (error "Cannot continue"))
   (message "Continuing.")
   (exit-recursive-edit))
 
@@ -251,9 +392,7 @@ will be used, such as in a debug on exit from a frame."
 (defun debugger-jump ()
   "Continue to exit from this frame, with all debug-on-entry suspended."
   (interactive)
-  ;; Compensate for the two extra stack frames for debugger-jump.
-  (let ((debugger-frame-offset (+ debugger-frame-offset 2)))
-    (debugger-frame))
+  (debugger-frame)
   ;; Turn off all debug-on-entry functions
   ;; but leave them in the list.
   (let ((list debug-function-list))
@@ -280,6 +419,8 @@ will be used, such as in a debug on exit from a frame."
     (beginning-of-line)
     (let ((opoint (point))
          (count 0))
+      (while (not (eq (cadr (backtrace-frame count)) 'debug))
+       (setq count (1+ count)))
       (goto-char (point-min))
       (if (or (equal (buffer-substring (point) (+ (point) 6))
                     "Signal")
@@ -296,23 +437,21 @@ will be used, such as in a debug on exit from a frame."
                 (forward-sexp 2))
               (forward-line 1)
               (<= (point) opoint))
+       (if (looking-at " *;;;")
+           (forward-line 1))
        (setq count (1+ count)))
       count)))
 
-;; Chosen empirically to account for all the frames
-;; that will exist when debugger-frame is called
-;; within the first one that appears in the backtrace buffer.
-;; Assumes debugger-frame is called from a key;
-;; will be wrong if it is called with Meta-x.
-(defconst debugger-frame-offset 8 "")
-
 (defun debugger-frame ()
   "Request entry to debugger when this frame exits.
 Applies to the frame whose line point is on in the backtrace."
   (interactive)
+  (save-excursion
+    (beginning-of-line)
+    (if (looking-at " *;;;\\|[a-z]")
+       (error "This line is not a function call")))
   (beginning-of-line)
-  (let ((level (debugger-frame-number)))
-    (backtrace-debug (+ level debugger-frame-offset) t))
+  (backtrace-debug (debugger-frame-number) t)
   (if (= (following-char) ? )
       (let ((buffer-read-only nil))
        (delete-char 1)
@@ -323,9 +462,12 @@ Applies to the frame whose line point is on in the backtrace."
   "Do not enter debugger when this frame exits.
 Applies to the frame whose line point is on in the backtrace."
   (interactive)
+  (save-excursion
+    (beginning-of-line)
+    (if (looking-at " *;;;\\|[a-z]")
+       (error "This line is not a function call")))
   (beginning-of-line)
-  (let ((level (debugger-frame-number)))
-    (backtrace-debug (+ level debugger-frame-offset) nil))
+  (backtrace-debug (debugger-frame-number) nil)
   (if (= (following-char) ?*)
       (let ((buffer-read-only nil))
        (delete-char 1)
@@ -337,48 +479,61 @@ Applies to the frame whose line point is on in the backtrace."
 (put 'debugger-env-macro 'lisp-indent-function 0)
 (defmacro debugger-env-macro (&rest body)
   "Run BODY in original environment."
-  (`
-   (save-excursion
-     (if (null (buffer-name debugger-old-buffer))
-        ;; old buffer deleted
-        (setq debugger-old-buffer (current-buffer)))
-     (set-buffer debugger-old-buffer)
-     (let ((track-mouse debugger-outer-track-mouse)
-          (last-command debugger-outer-last-command)
-          (this-command debugger-outer-this-command)
-          (unread-command-char debugger-outer-unread-command-char)
-          (unread-command-events debugger-outer-unread-command-events)
-          (last-input-event debugger-outer-last-input-event)
-          (last-command-event debugger-outer-last-command-event)
-          (last-nonmenu-event debugger-outer-last-nonmenu-event)
-          (last-event-frame debugger-outer-last-event-frame)
-          (standard-input debugger-outer-standard-input)
-          (standard-output debugger-outer-standard-output)
-          (cursor-in-echo-area debugger-outer-cursor-in-echo-area)
-          (overriding-local-map debugger-outer-overriding-local-map)
-          (overriding-terminal-local-map
-           debugger-outer-overriding-terminal-local-map)
-          (load-read-function debugger-outer-load-read-function))
-       (set-match-data debugger-outer-match-data)
-       (prog1 (progn (,@ body))
-        (setq debugger-outer-match-data (match-data))
-        (setq debugger-outer-load-read-function load-read-function)
-        (setq debugger-outer-overriding-terminal-local-map
-              overriding-terminal-local-map)
-        (setq debugger-outer-overriding-local-map overriding-local-map)
-        (setq debugger-outer-track-mouse track-mouse)
-        (setq debugger-outer-last-command last-command)
-        (setq debugger-outer-this-command this-command)
-        (setq debugger-outer-unread-command-char unread-command-char)
-        (setq debugger-outer-unread-command-events unread-command-events)
-        (setq debugger-outer-last-input-event last-input-event)
-        (setq debugger-outer-last-command-event last-command-event)
-        (setq debugger-outer-last-nonmenu-event last-nonmenu-event)
-        (setq debugger-outer-last-event-frame last-event-frame)
-        (setq debugger-outer-standard-input standard-input)
-        (setq debugger-outer-standard-output standard-output)
-        (setq debugger-outer-cursor-in-echo-area cursor-in-echo-area)
-        )))))
+  `(save-excursion
+    (if (null (buffer-name debugger-old-buffer))
+        ;; old buffer deleted
+        (setq debugger-old-buffer (current-buffer)))
+    (set-buffer debugger-old-buffer)
+    (let ((load-read-function debugger-outer-load-read-function)
+          (overriding-terminal-local-map
+           debugger-outer-overriding-terminal-local-map)
+          (overriding-local-map debugger-outer-overriding-local-map)
+          (track-mouse debugger-outer-track-mouse)
+          (last-command debugger-outer-last-command)
+          (this-command debugger-outer-this-command)
+          (unread-command-events debugger-outer-unread-command-events)
+          (unread-post-input-method-events
+           debugger-outer-unread-post-input-method-events)
+          (last-input-event debugger-outer-last-input-event)
+          (last-command-event debugger-outer-last-command-event)
+          (last-nonmenu-event debugger-outer-last-nonmenu-event)
+          (last-event-frame debugger-outer-last-event-frame)
+          (standard-input debugger-outer-standard-input)
+          (standard-output debugger-outer-standard-output)
+          (inhibit-redisplay debugger-outer-inhibit-redisplay)
+          (cursor-in-echo-area debugger-outer-cursor-in-echo-area))
+      (set-match-data debugger-outer-match-data)
+      (prog1
+         (let ((save-ucc (with-no-warnings unread-command-char)))
+           (unwind-protect
+               (progn
+                 (with-no-warnings
+                  (setq unread-command-char debugger-outer-unread-command-char))
+                 (prog1 (progn ,@body)
+                   (with-no-warnings
+                    (setq debugger-outer-unread-command-char unread-command-char))))
+             (with-no-warnings
+              (setq unread-command-char save-ucc))))
+        (setq debugger-outer-match-data (match-data))
+        (setq debugger-outer-load-read-function load-read-function)
+        (setq debugger-outer-overriding-terminal-local-map
+              overriding-terminal-local-map)
+        (setq debugger-outer-overriding-local-map overriding-local-map)
+        (setq debugger-outer-track-mouse track-mouse)
+        (setq debugger-outer-last-command last-command)
+        (setq debugger-outer-this-command this-command)
+        (setq debugger-outer-unread-command-events unread-command-events)
+        (setq debugger-outer-unread-post-input-method-events
+              unread-post-input-method-events)
+        (setq debugger-outer-last-input-event last-input-event)
+        (setq debugger-outer-last-command-event last-command-event)
+        (setq debugger-outer-last-nonmenu-event last-nonmenu-event)
+        (setq debugger-outer-last-event-frame last-event-frame)
+        (setq debugger-outer-standard-input standard-input)
+        (setq debugger-outer-standard-output standard-output)
+        (setq debugger-outer-inhibit-redisplay inhibit-redisplay)
+        (setq debugger-outer-cursor-in-echo-area cursor-in-echo-area)
+        ))))
 
 (defun debugger-eval-expression (exp)
   "Eval an expression, in an environment like that outside the debugger."
@@ -389,10 +544,10 @@ Applies to the frame whose line point is on in the backtrace."
   (debugger-env-macro (eval-expression exp)))
 \f
 (defvar debugger-mode-map nil)
-(if debugger-mode-map
-    nil
+(unless debugger-mode-map
   (let ((loop ? ))
     (setq debugger-mode-map (make-keymap))
+    (set-keymap-parent debugger-mode-map button-buffer-map)
     (suppress-keymap debugger-mode-map)
     (define-key debugger-mode-map "-" 'negative-argument)
     (define-key debugger-mode-map "b" 'debugger-frame)
@@ -407,6 +562,8 @@ Applies to the frame whose line point is on in the backtrace."
     (define-key debugger-mode-map "e" 'debugger-eval-expression)
     (define-key debugger-mode-map " " 'next-line)
     (define-key debugger-mode-map "R" 'debugger-record-expression)
+    (define-key debugger-mode-map "\C-m" 'help-follow)
+    (define-key debugger-mode-map [mouse-2] 'push-button)
     ))
 
 
@@ -472,19 +629,23 @@ Redefining FUNCTION also cancels it."
   (interactive "aDebug on entry (to function): ")
   (debugger-reenable)
   ;; Handle a function that has been aliased to some other function.
-  (if (symbolp (symbol-function function))
+  (if (and (subrp (symbol-function function))
+          (eq (cdr (subr-arity (symbol-function function))) 'unevalled))
+      (error "Function %s is a special form" function))
+  (if (or (symbolp (symbol-function function))
+         (subrp (symbol-function function)))
+      ;; Create a wrapper in which we can then add the necessary debug call.
       (fset function `(lambda (&rest debug-on-entry-args)
+                       ,(interactive-form (symbol-function function))
                        (apply ',(symbol-function function)
                               debug-on-entry-args))))
-  (if (subrp (symbol-function function))
-      (error "Function %s is a primitive" function))
   (or (consp (symbol-function function))
       (debug-convert-byte-code function))
   (or (consp (symbol-function function))
       (error "Definition of %s is not a list" function))
   (fset function (debug-on-entry-1 function (symbol-function function) t))
   (or (memq function debug-function-list)
-      (setq debug-function-list (cons function debug-function-list)))
+      (push function debug-function-list))
   function)
 
 ;;;###autoload
@@ -494,18 +655,21 @@ If argument is nil or an empty string, cancel for all functions."
   (interactive
    (list (let ((name
                (completing-read "Cancel debug on entry (to function): "
-                                ;; Make an "alist" of the functions
-                                ;; that now have debug on entry.
-                                (mapcar 'list
-                                        (mapcar 'symbol-name
-                                                debug-function-list))
+                                (mapcar 'symbol-name debug-function-list)
                                 nil t nil)))
           (if name (intern name)))))
   (debugger-reenable)
   (if (and function (not (string= function "")))
       (progn
-       (fset function
-             (debug-on-entry-1 function (symbol-function function) nil))
+       (let ((f (debug-on-entry-1 function (symbol-function function) nil)))
+         (condition-case nil
+             (if (and (equal (nth 1 f) '(&rest debug-on-entry-args))
+                      (eq (car (nth 3 f)) 'apply))
+                 ;; `f' is a wrapper introduced in debug-on-entry.
+                 ;; Get rid of it since we don't need it any more.
+                 (setq f (nth 1 (nth 1 (nth 3 f)))))
+           (error nil))
+         (fset function f))
        (setq debug-function-list (delq function debug-function-list))
        function)
     (message "Cancelling debug-on-entry for all functions")
@@ -535,18 +699,19 @@ If argument is nil or an empty string, cancel for all functions."
        (debug-on-entry-1 function (cdr defn) flag)
       (or (eq (car defn) 'lambda)
          (error "%s not user-defined Lisp function" function))
-      (let (tail prec)
-       (if (stringp (car (nthcdr 2 defn)))
-           (setq tail (nthcdr 3 defn)
-                 prec (list (car defn) (car (cdr defn))
-                            (car (cdr (cdr defn)))))
-         (setq tail (nthcdr 2 defn)
-               prec (list (car defn) (car (cdr defn)))))
-       (if (eq flag (equal (car tail) '(debug 'debug)))
-           defn
-         (if flag
-             (nconc prec (cons '(debug 'debug) tail))
-           (nconc prec (cdr tail))))))))
+      (let ((tail (cddr defn)))
+       ;; Skip the docstring.
+       (if (stringp (car tail)) (setq tail (cdr tail)))
+       ;; Skip the interactive form.
+       (if (eq 'interactive (car-safe (car tail))) (setq tail (cdr tail)))
+       (unless (eq flag (equal (car tail) '(debug 'debug)))
+         ;; Add/remove debug statement as needed.
+         (if (not flag)
+             (progn (setcar tail (cadr tail))
+                    (setcdr tail (cddr tail)))
+           (setcdr tail (cons (car tail) (cdr tail)))
+           (setcar tail '(debug 'debug))))
+       defn))))
 
 (defun debugger-list-functions ()
   "Display a list of all the functions now set to debug on entry."
@@ -568,4 +733,5 @@ If argument is nil or an empty string, cancel for all functions."
 
 (provide 'debug)
 
+;;; arch-tag: b6ec7047-f801-4103-9c63-d69322db9d3b
 ;;; debug.el ends here