-
Notifications
You must be signed in to change notification settings - Fork 6
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("No match %d in highlight %S" 2 (2 'org-headline-done prepend)) #24
Comments
It looks like match-data is changed somewhere where it is not supposed to. |
@Whil- By any chance, do you know what kind of org file triggers the error? |
FIle sent offline |
Thanks! Though I cannot reproduce neither with my config nor with clean emacs. I guess, there is some strange interaction between this branch and org-roam. |
Hmm, yeah. Sucks to debug other ppls's problems... :( Sorry for not being more specific here. If it helps it could have to do with |
Thanks! I tried |
Ok, sorry for not being of more help here. Let's assume it's a problem with my setup for now then since it's difficult to reproduce. |
I did a benchmark from your branch @yantar92. No performance improvement. Slower actually. Output of the tests:
First row is with new customization on. Second with it off (showing current performance) and third is with all files already loaded into buffers.
I can note that benchmarking
org-roam--list-all-files
shows that's actually a significant piece of the time spent already there:results:
These numbers are with my personal setup though, Emacs 28.0.50 on Windows 10 with native compilation on. Surely they can differ in another persons config but I suspect the relative performance difference is around the same. (i.e. around 100x faster with this customization)
Sidenote: Got an error with the
org-fold-universal-core
branch.error("No match %d in highlight %S" 2 (2 'org-headline-done prepend))
Can discuss that offline if needed, to not take over this pull-request.Originally posted by @Whil- in alphapapa/org-ql#200 (comment)
The text was updated successfully, but these errors were encountered: