nit: minor refactors & comment/docstring revisions

core/autoload/debug.el

@@ -14,9 +14,9 @@
     gcmh-verbose
     init-file-debug
     jka-compr-verbose
+    (message-log-max . 16384)
     url-debug
-    use-package-verbose
+    use-package-verbose)
-    (message-log-max . 16384))
   "A list of variable to toggle on `doom-debug-mode'.
 
 Each entry can be a variable symbol or a cons cell whose CAR is the variable

core/autoload/themes.el

@@ -40,6 +40,7 @@ all themes. It will apply to all themes once they are loaded."
        (when (or (get 'doom-theme 'previous-themes)
                  (null doom-theme))
          (funcall #',fn))
+       ;; FIXME Prevent clobbering this on-the-fly
        (add-hook 'doom-customize-theme-hook #',fn 100))))
 
 ;;;###autoload

core/core-cli.el

@@ -51,9 +51,9 @@
         (setq dir (car tail))
         (let ((default-directory dir))
           (load (expand-file-name "subdirs.el") t inhibit-message t))
-        (or (string-prefix-p lispdir dir)
+        (unless (string-prefix-p lispdir dir)
-            (let ((default-directory dir))
+          (let ((default-directory dir))
-              (load (expand-file-name "leim-list.el") t inhibit-message t)))
+            (load (expand-file-name "leim-list.el") t inhibit-message t)))
         (setq tail (cdr tail)))
       (load site-run-file t inhibit-message))))
 

core/core-lib.el

@@ -262,8 +262,8 @@ NAME, ARGLIST, and BODY are the same as `defun', `defun*', `defmacro', and
   "Run FORMS without generating any output.
 
 This silences calls to `message', `load', `write-region' and anything that
-writes to `standard-output'. In interactive sessions this won't suppress writing
+writes to `standard-output'. In interactive sessions this inhibits output to the
-to *Messages*, only inhibit output in the echo area."
+echo-area, but not to *Messages*."
   `(if doom-debug-p
        (progn ,@forms)
      ,(if doom-interactive-p

modules/completion/vertico/autoload/workspaces.el

@@ -60,7 +60,7 @@
 (autoload 'consult--multi "consult")
 ;;;###autoload
 (defun +vertico/switch-workspace-buffer (&optional force-same-workspace)
-  "Switch to another buffer in the same workspace.
+  "Switch to another buffer in the same or a specified workspace.
 
 Type the workspace's number (starting from 1) followed by a space to display its
 buffer list. Selecting a buffer in another workspace will switch to that

modules/lang/org/config.el

@@ -1070,6 +1070,7 @@ between the two."
         (set-marker p nil)))))
 
 
+;; TODO Move to +encrypt flag
 (use-package! org-crypt ; built-in
   :commands org-encrypt-entries org-encrypt-entry org-decrypt-entries org-decrypt-entry
   :hook (org-reveal-start . org-decrypt-entry)

modules/term/vterm/config.el

@@ -5,7 +5,7 @@
   :commands vterm-mode
   :hook (vterm-mode . doom-mark-buffer-as-real-h)
   :hook (vterm-mode . hide-mode-line-mode) ; modeline serves no purpose in vterm
-  :init
+  :preface
   ;; HACK Because vterm clusmily forces vterm-module.so's compilation on us when
   ;;      the package is loaded, this is necessary to prevent it when
   ;;      byte-compiling this file (`use-package' blocks eagerly loads packages

modules/ui/nav-flash/config.el

@@ -13,6 +13,7 @@
 ;;
 ;;; Packages
 
+;; DEPRECATED To be replaced with pulsar.el when Emacs 28 support is dropped
 (use-package! nav-flash
   :defer t
   :init