Re-enable code giving word syntax to certain japanese-jisx0208 characters.
[bpt/emacs.git] / lisp / emacs-lisp / debug.el
index fb2a132..a0165a8 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 Free Software Foundation, Inc.
 
 ;; Maintainer: FSF
 ;; Keywords: lisp, tools, maint
 
 ;;; Code:
 
-(defvar debug-function-list nil
-  "List of functions currently set for debug on entry.")
-
-(defvar debugger-step-after-exit nil
-  "Non-nil means \"single-step\" after the debugger exits.")
+(require 'button)
+
+(defgroup debugger nil
+  "Debuggers and related commands for Emacs."
+  :prefix "debugger-"
+  :group 'debug)
+
+(defcustom debugger-mode-hook nil
+  "*Hooks run when `debugger-mode' is turned on."
+  :type 'hook
+  :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."
+  :type '(repeat function)
+  :group 'debugger)
+
+(defcustom debugger-step-after-exit nil
+  "Non-nil means \"single-step\" after the debugger exits."
+  :type 'boolean
+  :group 'debugger)
 
 (defvar debugger-value nil
   "This is the value for the debugger to return, when it returns.")
 (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-overriding-terminal-local-map)
 (defvar debugger-outer-track-mouse)
 (defvar debugger-outer-last-command)
 (defvar debugger-outer-this-command)
 (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
@@ -68,7 +101,8 @@ 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...")
+  (unless noninteractive
+    (message "Entering debugger..."))
   (let (debugger-value
        (debug-on-error nil)
        (debug-on-quit nil)
@@ -83,85 +117,75 @@ first will be printed into the backtrace buffer."
        (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-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-char -1) 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
            (save-window-excursion
              (pop-to-buffer debugger-buffer)
-             (erase-buffer)
-             (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-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 (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)
-             ;; 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 "")
-               (recursive-edit))))
+               ;; 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)
@@ -174,26 +198,161 @@ first will be printed into the backtrace buffer."
              (erase-buffer)
              (fundamental-mode))
          (kill-buffer debugger-buffer))
-       (store-match-data debugger-outer-match-data)))
+       (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 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.
 Enter another debugger on next entry to eval, apply or funcall."
@@ -205,6 +364,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))
 
@@ -221,9 +382,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))
@@ -250,6 +409,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")
@@ -266,23 +427,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)
@@ -290,66 +449,87 @@ Applies to the frame whose line point is on in the backtrace."
   (beginning-of-line))
 
 (defun debugger-frame-clear ()
-  "Do not enter to debugger when this frame exits.
+  "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)
        (insert ? )))
   (beginning-of-line))
 
+
+
+(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 ((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-char debugger-outer-unread-command-char)
+          (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 (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-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."
   (interactive
    (list (read-from-minibuffer "Eval: "
                               nil read-expression-map t
                               'read-expression-history)))
-  (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)
-         (load-read-function debugger-outer-load-read-function))
-      (store-match-data debugger-outer-match-data)
-      (prog1 (eval-expression exp)
-       (setq debugger-outer-match-data (match-data))
-       (setq debugger-outer-load-read-function load-read-function)
-       (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)))))
+  (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)
@@ -362,7 +542,38 @@ Applies to the frame whose line point is on in the backtrace."
     (define-key debugger-mode-map "h" 'describe-mode)
     (define-key debugger-mode-map "q" 'top-level)
     (define-key debugger-mode-map "e" 'debugger-eval-expression)
-    (define-key debugger-mode-map " " 'next-line)))
+    (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)
+    ))
+
+
+(defcustom debugger-record-buffer "*Debugger-record*"
+  "*Buffer name for expression values, for \\[debugger-record-expression]."
+  :type 'string
+  :group 'debugger
+  :version "20.3")
+
+(defun debugger-record-expression  (exp)
+  "Display a variable's value and record it in `*Backtrace-record*' buffer."
+  (interactive
+   (list (read-from-minibuffer
+         "Record Eval: "
+         nil
+         read-expression-map t
+         'read-expression-history)))
+  (let* ((buffer (get-buffer-create debugger-record-buffer))
+        (standard-output buffer))
+    (princ (format "Debugger Eval (%s): " exp))
+    (princ (debugger-eval-expression exp))
+    (terpri))
+
+  (with-current-buffer (get-buffer debugger-record-buffer)
+    (save-excursion
+      (forward-line -1)
+      (message
+       (buffer-substring (point) (progn (end-of-line) (point)))))))
 
 (put 'debugger-mode 'mode-class 'special)
 
@@ -381,12 +592,13 @@ which functions will enter the debugger when called.
 
 Complete list of commands:
 \\{debugger-mode-map}"
-  (kill-all-local-variables)    
+  (kill-all-local-variables)
   (setq major-mode 'debugger-mode)
   (setq mode-name "Debugger")
   (setq truncate-lines t)
   (set-syntax-table emacs-lisp-mode-syntax-table)
-  (use-local-map debugger-mode-map))
+  (use-local-map debugger-mode-map)
+  (run-hooks 'debugger-mode-hook))
 \f
 ;;;###autoload
 (defun debug-on-entry (function)
@@ -398,6 +610,11 @@ Use \\[cancel-debug-on-entry] to cancel the effect of this command.
 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))
+      (fset function `(lambda (&rest debug-on-entry-args)
+                       (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))
@@ -406,7 +623,7 @@ Redefining FUNCTION also cancels it."
       (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
@@ -416,11 +633,7 @@ 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)
@@ -488,4 +701,6 @@ If argument is nil or an empty string, cancel for all functions."
       (set-buffer standard-output)
       (help-mode))))
 
+(provide 'debug)
+
 ;;; debug.el ends here