Skip to content

Conversation

@sheremetyev
Copy link
Contributor

@sheremetyev sheremetyev commented Oct 12, 2025

Current implementation supports footnotes with reference in the text and definition in separate paragraph. This PR adds support inline footnote syntax, supported in Obsidian and Pandoc.

@kivikakk
Copy link
Owner

This is amazing — thank you so much! I'll do my usual thing and run fuzzers against it for a few hours before merging.

@kivikakk
Copy link
Owner

Fuzzing terminated; the minimised case can be reproduced with this:

printf '^[\t\r]' | cargo run -- -e footnotes -e inline-footnotes

Fuzzing can be run with cargo fuzz run all_options -j8 (or some other value for -j depending on how many cores you want to try it with!).

@sheremetyev
Copy link
Contributor Author

Thanks for the details @kivikakk! I've pushed a fix for handling whitespace-only footnote, and have run fuzzer for some time - nothing popped up. Please let me know if there are any other issues!

@kivikakk
Copy link
Owner

Thank you so much for your contribution! ❤️

@kivikakk kivikakk merged commit 72b2dd3 into kivikakk:main Oct 14, 2025
22 checks passed
@kivikakk
Copy link
Owner

This is part of Comrak 0.44.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants