From db07304c71b3af84544700543c0dd82d1e68b351 Mon Sep 17 00:00:00 2001 From: Henrik Lissner Date: Sun, 11 Oct 2020 16:26:04 -0400 Subject: [PATCH] Don't enable explain-pause-mode at startup When starting Emacs in debug mode, explain-pause-mode is enabled. This pulls in other packages with it, which can taint results when testing package load order. Also, explain-pause-mode is for measuring pauses during interactive use, it isn't very useful for startup benchmarking. So we only toggle it if doom-debug-mode is toggled interactively. --- core/autoload/debug.el | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/core/autoload/debug.el b/core/autoload/debug.el index c05e5b897..4fce6f5e5 100644 --- a/core/autoload/debug.el +++ b/core/autoload/debug.el @@ -49,8 +49,9 @@ symbol and CDR is the value to set it to when `doom-debug-mode' is activated.") ((if (boundp var) (set-default var enabled) (add-to-list 'doom--debug-vars-undefined var))))) - (when (fboundp 'explain-pause-mode) - (explain-pause-mode (if enabled +1 -1))) + (when (called-interactively-p 'any) + (when (fboundp 'explain-pause-mode) + (explain-pause-mode (if enabled +1 -1)))) ;; Watch for changes in `doom-debug-variables', or when packages load (and ;; potentially define one of `doom-debug-variables'), in case some of them ;; aren't defined when `doom-debug-mode' is first loaded.