-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error raised when auto reloading the bib file. #286
Comments
Thanks. It's fixed on #281. I plan to merge it in the morning. If you have time to test it, that would be helpful. |
…281) Fix #286 (a recursion bug), but may have other benefits for modularity and performance. 1. bibtex-actions-select-keys -> bibtex-actions-select-refs, which now returns (key . entry) 2. change functions that rely on keys to accept this richer input 3. decouple action and formatting functions from caches (no more looking up data there, which caused the bug)
Merged, though I haven't time to test it thoroughly. PS - the issue should not have anything to do with filenotify etc; it was just a bug that occurred in some situations when loading the library. |
Seems work, thanks. |
PS - how are you find the filenotify support? I've not yet set it up on my system, but was wondering if it should be default in Doom emacs (I'm working on its biblio module). |
Describe the bug
Error raised when auto reloading the bib file.
To Reproduce
Steps to reproduce the behavior:
(bibtex-actions-filenotify-setup '(LaTeX-mode-hook org-mode-hook))
to config.org-cite-insert
and everything works well.bibtex-actions-refresh
and everything works well.Expected behavior
There should not be any error when reloading the bib file.
Emacs version:
GNU Emacs 27.2 (build 1, x86_64-apple-darwin20.3.0, Carbon Version 164 AppKit 2022.3) of 2021-04-02
The text was updated successfully, but these errors were encountered: