diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-24 16:23:34 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-24 16:23:34 -0500 |
| commit | cb08bf8754b09222081a571dc4134d5568d61012 (patch) | |
| tree | ad068dc2062d2d17773309c932c38afaa63e7fa3 /modules/music-config.el | |
| parent | c24dbe25d25bee514393b54bc3cbd5e7d638fd9b (diff) | |
| download | dotemacs-cb08bf8754b09222081a571dc4134d5568d61012.tar.gz dotemacs-cb08bf8754b09222081a571dc4134d5568d61012.zip | |
refactor: clear six byte-compile warnings across four modules
- org-agenda-frame declared two of its own functions with declare-function.
- Same-file forward refs resolve at end of compile, so those were redundant.
- Worse, the declared empty arglist overrode safe-redo's real (&optional frame).
- That silently disabled arg-count checking on it; removed both declarations.
- music-config and video-audio-recording referenced three vars before their defvars.
- Added forward declarations so each compiles as a dynamic binding.
- dashboard declared el special, which made the section macro's own binding shadow it.
- Removed that declaration; el is the macro's lexical binding and resolves cleanly.
- Declared wttrin, the last undefined-function reference in the launcher table.
Diffstat (limited to 'modules/music-config.el')
| -rw-r--r-- | modules/music-config.el | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/modules/music-config.el b/modules/music-config.el index a7ec62c8..233bae72 100644 --- a/modules/music-config.el +++ b/modules/music-config.el @@ -374,6 +374,11 @@ point until it ends; the snap lands when the search exits." (defvar-local cj/music--renumber-timer nil "Pending idle timer for the playlist row renumber, or nil.") +;; Forward declaration: the real `defvar-local' is a few defuns below, next to +;; the highlight helper that owns it. Declared special here so the setq in this +;; function compiles as a dynamic binding, not a free-variable warning. +(defvar cj/music--current-number-overlay) + (defun cj/music--renumber-rows (&optional buffer) "Number every playlist row in BUFFER (default: current buffer) via overlays. Each non-blank line gets an \"NNN \" before-string so the cursor stays @@ -789,6 +794,10 @@ ENTRIES. Nil when neither applies (the caller falls back to a timestamp)." (plist-get (cdr (assoc (emms-track-name tr) entries)) :name))))) +;; Forward declaration: the real `defvar' lives with the radio config block far +;; below. Declared special here so this reference compiles clean. +(defvar cj/music-radio-save-dir) + (defun cj/music--save-directory (tracks) "Directory a saved playlist targets. An all-stream queue is a radio playlist and saves into |
