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 | fc94e5b7101f6677950c414c2822b451351ccc56 (patch) | |
| tree | 0097f78d0c08bd9b76801d7bedd999fe75156cc3 | |
| parent | 1aa8d0f60ceec62d37c34d5cb31c46434a647ff6 (diff) | |
| download | dotemacs-fc94e5b7101f6677950c414c2822b451351ccc56.tar.gz dotemacs-fc94e5b7101f6677950c414c2822b451351ccc56.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.
| -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 be5b6b11..5cd76f0a 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) |