diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-10 15:37:36 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-10 15:37:36 -0500 |
| commit | 618bc7813b9acfcf1dfccc9c6590f6f5aece86cf (patch) | |
| tree | 5719792c87bbc74ae380eb020a39d42b2ac86e48 /modules/cj-window-toggle-lib.el | |
| parent | f59ff9606fd96c6b1b9037ea5befb39b5e5a57b9 (diff) | |
| download | dotemacs-618bc7813b9acfcf1dfccc9c6590f6f5aece86cf.tar.gz dotemacs-618bc7813b9acfcf1dfccc9c6590f6f5aece86cf.zip | |
refactor(external-open): extract external-open-lib for shared helpers
Same shared-helpers split-pattern that ai-vterm/vterm-config use
through cj-window-toggle-lib and that calendar-sync uses through
cj-org-text-lib. Pull the two pure dispatch helpers out of the
external-open feature module into a sibling library so consumers
that only need the dispatch don't have to require the whole feature.
New `modules/external-open-lib.el' carries:
- `cj/external-open-command'
- `cj/external-open-launcher-p'
`modules/external-open.el' stays as the feature module: the
`default-open-extensions' defcustom, the `find-file' advice
(`cj/find-file-auto'), and the interactive commands (`cj/xdg-open',
`cj/open-this-file-with'). It now requires external-open-lib for
the dispatch helpers.
Migrate consumers:
- system-utils.el used to require `external-open' for
`cj/external-open-launcher-p' alone -- now requires
`external-open-lib' directly.
- dirvish-config.el calls `cj/external-open-command' from
`cj/dirvish-open-file-manager-here' -- add an explicit
`(require \='external-open-lib)'.
Test files renamed to match the system-lib naming pattern
(test-<library>-<feature>.el):
- test-external-open-command.el -> test-external-open-lib-command.el
- test-external-open-launcher-p.el -> test-external-open-lib-launcher-p.el
No behavior change.
Diffstat (limited to 'modules/cj-window-toggle-lib.el')
0 files changed, 0 insertions, 0 deletions
