diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-13 06:32:08 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-13 06:32:08 -0500 |
| commit | eaef768f97945577200252fa9dec3e2b8a7e7dc6 (patch) | |
| tree | e5460840c06bfcf07fbfaebe61a9dadc8409ead1 /modules/test-runner.el | |
| parent | e012334e168f4aba265c89a35a13ee578da6fa52 (diff) | |
| download | dotemacs-eaef768f97945577200252fa9dec3e2b8a7e7dc6.tar.gz dotemacs-eaef768f97945577200252fa9dec3e2b8a7e7dc6.zip | |
fix(calibredb-epub): point cj/nov--file-path at nov-file-name
The fallback chain was checking `nov-epub-filename` and `nov-epub-file`, but neither symbol exists in nov.el — the real var is `nov-file-name`, set by `nov-mode` from the visited file. Both `boundp` arms always returned nil, so the fallbacks were dead code. The bug was dormant rather than active: `buffer-file-name` always holds the EPUB path for normal nov buffers and covered the first arm of the `or`.
I replaced both wrong-named arms with a single live arm on `nov-file-name`, and added a Boundary test that exercises it.
Diffstat (limited to 'modules/test-runner.el')
0 files changed, 0 insertions, 0 deletions