aboutsummaryrefslogtreecommitdiff
path: root/modules/custom-buffer-file.el
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-05-28 02:45:31 -0500
committerCraig Jennings <c@cjennings.net>2026-05-28 02:45:31 -0500
commit2c11d1a45c9d5b33fe2142a10b1c3d6f26b306c8 (patch)
tree74562b4dcfef9df69bb90ef312f3969ef7aeb3da /modules/custom-buffer-file.el
parentf3e9405c6607ae6f2738ec4c973ce2776a519635 (diff)
downloaddotemacs-2c11d1a45c9d5b33fe2142a10b1c3d6f26b306c8.tar.gz
dotemacs-2c11d1a45c9d5b33fe2142a10b1c3d6f26b306c8.zip
fix(signal): require signel before reading its private variables
cj/signel--ensure-started in modules/signal-config.el was reading signel--process-name in the first branch of its cond before the use-package autoload of signel had fired. The forward-declared (defvar signel--process-name) at L137 silences the byte-compile warning but doesn't actually bind the variable. Its value comes from signel.el's defconst, which doesn't run until signel is loaded. The first call to cj/signel-connect (C-; M SPC) after Emacs launch produced "Symbol's value as variable is void: signel--process-name" instead of starting the daemon. Surfaced tonight during the manual verify walk of the initiate-message workflow. I added (require 'signel) at the top of cj/signel--ensure-started so signel loads before any of its variables get read. The require is idempotent, so callers that hit the function after signel is already loaded pay nothing. The new ERT test test-signal-config-ensure-started-requires-signel-first asserts ordering: require must be the first call inside the function, not just called somewhere. A future refactor that moves the require below the cond would fail this test instead of passing silently.
Diffstat (limited to 'modules/custom-buffer-file.el')
0 files changed, 0 insertions, 0 deletions