diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-29 04:19:31 -0400 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-29 04:19:31 -0400 |
| commit | 256027866f9ff2cbc3b6155950f21a92b54ea307 (patch) | |
| tree | e63cd2b63bfbd8f893313e5a900710c353e905e9 /tests/test-system-defaults.el | |
| parent | 8c91731f5ba177d9f0f7c078576ddd91c606ed7c (diff) | |
| download | dotemacs-256027866f9ff2cbc3b6155950f21a92b54ea307.tar.gz dotemacs-256027866f9ff2cbc3b6155950f21a92b54ea307.zip | |
refactor: prefix two collision-prone helpers, document naming audit
Two owned helpers carried unprefixed generic names that risk colliding in the single Emacs namespace: car-member (local-repository.el) and unpropertize-kill-ring (system-defaults.el). I renamed them to localrepo--car-member and cj/--unpropertize-kill-ring and updated their callers and tests. Both are non-interactive and contained, so no alias was needed.
docs/design/naming-audit.org records the rest of the scan: the allowlist of deliberate module prefixes, the foreign forward-declarations that aren't owned definitions, and a deferred list (keybound commands, the with-timer macro, the ui-theme defcustoms, the user-constants paths) that each want a focused pass rather than an unattended rename.
Diffstat (limited to 'tests/test-system-defaults.el')
| -rw-r--r-- | tests/test-system-defaults.el | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/test-system-defaults.el b/tests/test-system-defaults.el index f653e1fbb..a641adea1 100644 --- a/tests/test-system-defaults.el +++ b/tests/test-system-defaults.el @@ -8,7 +8,7 @@ ;; writes land, where backups go, and whether the minibuffer GC hooks are ;; installed. Load happens in the shared sandbox (testutil-system-defaults.el). ;; -;; The module's functions (cj/disabled, the GC hook bodies, unpropertize-kill-ring, +;; The module's functions (cj/disabled, the GC hook bodies, cj/--unpropertize-kill-ring, ;; cj/log-comp-warning) are covered by test-system-defaults-functions.el, and the ;; vc-follow-symlinks default by test-system-defaults-vc-follow-symlinks.el. |