[Emacs-diffs] scratch/backtrace-mode adddc31 1/3: Initialize backtrace display options in backtrace-mode

[Gemini Lasswell]

branch: scratch/backtrace-mode
commit adddc3160c744ab5a1cff4069709818b0c4dea4f
Author: Gemini Lasswell <address@hidden>
Commit: Gemini Lasswell <address@hidden>

    Initialize backtrace display options in backtrace-mode
    
    * lisp/emacs-lisp/backtrace.el (backtrace-mode): Initialize
    backtrace-view.
    * lisp/emacs-lisp/debug.el (debugger-setup-buffer): Modify
    backtrace-view instead of setting it.
    * lisp/emacs-lisp/edebug.el (edebug-backtrace): Remove
    initialization of backtrace-view.
    * lisp/emacs-lisp/ert.el
    (ert-results-pop-to-backtrace-for-test-at-point): Remove
    initialization of backtrace-view.
---
 lisp/emacs-lisp/backtrace.el | 8 +++++---
 lisp/emacs-lisp/debug.el     | 4 ++--
 lisp/emacs-lisp/edebug.el    | 3 +--
 lisp/emacs-lisp/ert.el       | 3 +--
 4 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/lisp/emacs-lisp/backtrace.el b/lisp/emacs-lisp/backtrace.el
index 6b7b26d..ed44c1c 100644
--- a/lisp/emacs-lisp/backtrace.el
+++ b/lisp/emacs-lisp/backtrace.el
@@ -737,15 +737,16 @@ creates a backtrace-mode buffer, should usually do the 
following:
  - Maybe set `backtrace-insert-header-function' to a function to create
    header text for the buffer.
  - Set `backtrace-frames' (see below).
- - Set `backtrace-view' if desired (see below).
+ - Maybe modify `backtrace-view' (see below).
  - Maybe set `backtrace-print-function'.
 
 A command which creates or switches to a Backtrace mode buffer,
 such as `ert-results-pop-to-backtrace-for-test-at-point', should
 initialize `backtrace-frames' to a list of `backtrace-frame'
 objects (`backtrace-get-frames' is provided for that purpose, if
-desired), and `backtrace-view' to a plist describing how it wants
-the backtrace to appear.  Finally, it should call `backtrace-print'.
+desired), and may optionally modify `backtrace-view', which is a
+plist describing the appearance of the backtrace.  Finally, it
+should call `backtrace-print'.
 
 `backtrace-print' calls `backtrace-insert-header-function'
 followed by `backtrace-print-frame', once for each stack frame."
@@ -766,6 +767,7 @@ followed by `backtrace-print-frame', once for each stack 
frame."
   ;; was because of bytecode. Since 2009 it's been set to t, but the
   ;; default is t so I think this isn't necessary.
   ;; (set-buffer-multibyte t)
+  (setq backtrace-view (list :do-xrefs t))
   (setq-local revert-buffer-function #'backtrace-revert)
   (setq-local filter-buffer-substring-function #'backtrace--filter-visible)
   (add-hook 'xref-backend-functions #'backtrace--xref-backend nil t))
diff --git a/lisp/emacs-lisp/debug.el b/lisp/emacs-lisp/debug.el
index 707e0cf..2d8a838 100644
--- a/lisp/emacs-lisp/debug.el
+++ b/lisp/emacs-lisp/debug.el
@@ -335,10 +335,10 @@ That buffer should be current already and in 
debugger-mode."
                    :debug-on-exit)
           nil))
 
-  (setq backtrace-view '(:do-xrefs t :show-flags t)
-        backtrace-insert-header-function (lambda ()
+  (setq backtrace-insert-header-function (lambda ()
                                            (debugger--insert-header args))
         backtrace-print-function debugger-print-function)
+  (plist-put backtrace-view :show-flags t)
   (backtrace-print)
   ;; Place point on "stack frame 0" (bug#15101).
   (goto-char (point-min))
diff --git a/lisp/emacs-lisp/edebug.el b/lisp/emacs-lisp/edebug.el
index 465ba1b..15bf05b 100644
--- a/lisp/emacs-lisp/edebug.el
+++ b/lisp/emacs-lisp/edebug.el
@@ -3959,8 +3959,7 @@ Otherwise call `debug' normally."
     (with-current-buffer edebug-backtrace-buffer
       (unless (derived-mode-p 'backtrace-mode)
         (backtrace-mode))
-      (setq backtrace-frames (edebug--backtrace-frames)
-            backtrace-view '(:do-xrefs t))
+      (setq backtrace-frames (edebug--backtrace-frames))
       (backtrace-print)
       (goto-char (point-min)))))
 
diff --git a/lisp/emacs-lisp/ert.el b/lisp/emacs-lisp/ert.el
index 3af31e1..d18c36d 100644
--- a/lisp/emacs-lisp/ert.el
+++ b/lisp/emacs-lisp/ert.el
@@ -2461,8 +2461,7 @@ To be used in the ERT results buffer."
            (backtrace-mode))
          (setq backtrace-insert-header-function
                (lambda () (ert--insert-backtrace-header (ert-test-name test)))
-               backtrace-frames (ert-test-result-with-condition-backtrace 
result)
-               backtrace-view '(:do-xrefs t))
+               backtrace-frames (ert-test-result-with-condition-backtrace 
result))
          (backtrace-print)
          (goto-char (point-min)))))))