diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-11 18:17:22 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-11 18:17:22 -0500 |
| commit | 29d73772476a047e654632ab38910b37bd63af69 (patch) | |
| tree | 0097f78d0c08bd9b76801d7bedd999fe75156cc3 /modules | |
| parent | ed4e34d0c2bac1d89a6266af4b4cd4a33713ba46 (diff) | |
| download | dotemacs-29d73772476a047e654632ab38910b37bd63af69.tar.gz dotemacs-29d73772476a047e654632ab38910b37bd63af69.zip | |
fix(org-capture): single-source the file+headline cache key
`cj/org-capture--goto-file-headline' stored the marker under `(list (expand-file-name (buffer-file-name)) headline)' while the lookup helper `cj/org-capture--file-headline-cache-key' uses `(list (org-capture-expand-file path) headline)'. They agree for plain absolute paths, but a symlinked target (`expand-file-name' doesn't follow symlinks, `find-file' canonicalizes the buffer name) or a capture path that needs `org-capture-expand-file's special handling would make the put-key and get-key diverge, so the cache would silently never hit. Production now goes through the helper too, so the key is built one way.
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/org-capture-config.el | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/org-capture-config.el b/modules/org-capture-config.el index be5b6b11a..5cd76f0af 100644 --- a/modules/org-capture-config.el +++ b/modules/org-capture-config.el @@ -76,7 +76,7 @@ re-scanning large target files after the first successful lookup." (org-mode)) (org-capture-put-target-region-and-position) (widen) - (let* ((key (list (expand-file-name (buffer-file-name)) headline)) + (let* ((key (cj/org-capture--file-headline-cache-key path headline)) (marker (gethash key cj/org-capture--file-headline-target-cache))) (if (cj/org-capture--headline-marker-valid-p marker headline) (goto-char marker) |