Add arch taglines
[bpt/emacs.git] / lisp / emacs-lisp / debug.el
index a768d39..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-"
@@ -75,6 +77,9 @@ This is to optimize `debugger-make-xrefs'.")
 (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)
@@ -99,127 +104,134 @@ 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)
-  (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 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)
-             (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)
+  (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
+             (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))
-                 (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)
-             (message "")
-             (let ((inhibit-trace t)
-                   (standard-output nil)
-                   (buffer-read-only t))
+                 (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 "")
-               ;; 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)
-    (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))
+               (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.
@@ -289,7 +301,7 @@ That buffer should be current already."
     (set-buffer (or buffer (current-buffer)))
     (setq buffer (current-buffer))
     (let ((buffer-read-only nil)
-         (old-end 1) (new-end 1))
+         (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,
@@ -315,7 +327,7 @@ That buffer should be current already."
            ;; 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 1 since strings are
+           ;; backtrace.  But we must subtract (point-min) since strings are
            ;; indexed in origin 0.
 
            ;; Replace the unchanged part of the backtrace
@@ -325,11 +337,12 @@ That buffer should be current already."
            ;; the changed part of the backtrace.
            (delete-region new-end (point-max))
            (goto-char (point-max))
-           (insert (substring debugger-previous-backtrace (1- old-end)))
+           (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
@@ -478,7 +491,6 @@ Applies to the frame whose line point is on in the backtrace."
           (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)
@@ -491,7 +503,17 @@ Applies to the frame whose line point is on in the backtrace."
           (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)
+      (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
@@ -500,7 +522,6 @@ Applies to the frame whose line point is on in the backtrace."
         (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)
@@ -526,6 +547,7 @@ Applies to the frame whose line point is on in the backtrace."
 (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)
@@ -607,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
@@ -629,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")
@@ -670,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."
@@ -703,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