Skip to content

Output whitespace entities at emphasis boundaries#750

Draft
kivikakk wants to merge 4 commits intomainfrom
push-ktkxttsxtyvo
Draft

Output whitespace entities at emphasis boundaries#750
kivikakk wants to merge 4 commits intomainfrom
push-ktkxttsxtyvo

Conversation

@kivikakk
Copy link
Copy Markdown
Owner

Will fix #551; currently just failing test cases.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Feb 19, 2026

Merging this PR will not alter performance

✅ 1 untouched benchmark


Comparing push-ktkxttsxtyvo (9902d72) with main (2d6339c)

Open in CodSpeed

@github-actions
Copy link
Copy Markdown
Contributor

Command Mean [ms] Min [ms] Max [ms] Relative
./bench.sh ./comrak-02c6c19 101.8 ± 2.3 94.8 105.9 1.78 ± 0.06
./bench.sh ./comrak-main 100.6 ± 2.0 96.8 105.2 1.76 ± 0.06
./bench.sh ./pulldown-cmark 57.2 ± 1.5 54.1 61.8 1.00
./bench.sh ./cmark-gfm 87.5 ± 8.6 79.2 109.9 1.53 ± 0.16
./bench.sh ./markdown-it 252.1 ± 4.4 245.1 262.9 4.41 ± 0.14

Run on Thu Feb 19 01:37:27 UTC 2026

We only need to encode the very outer characters so the run is
recognised; some of these will roundtrip differently to CommonMark but
as long as they're re-parsed the same (i.e. become the same HTML), it's
correct.
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.

**Hello ** doesn't roundtrip via CommonMark

1 participant