Skip to content
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

code spans differ - cmark vs md2html (reprise) #230

Closed
step- opened this issue Jan 25, 2024 · 0 comments
Closed

code spans differ - cmark vs md2html (reprise) #230

step- opened this issue Jan 25, 2024 · 0 comments

Comments

@step-
Copy link
Contributor

step- commented Jan 25, 2024

Different output, involving white space in code spans, between cmark 0.30.3-31-g01e572d and md2html master f46000c.
This test case completes the tests in #223 and broke somewhere in commit range 65957f5 - 70b247c.

Test subject:

`e
f
`

Diff output: cmark (left) - md2html (right):
note "f " vs. "f ".

<p><code>e f </code></p>     |  <p><code>e f  </code></p>
@mity mity closed this as completed in aeddaf5 Jan 25, 2024
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Feb 10, 2024
Changes:

 * Changes mandated by CommonMark specification 0.31:

   - The specification expands set of Unicode characters seen by Markdown
     parser as a punctuation. Namely all Unicode general categories P
     (punctuation) and S (symbols) are now seen as such.

   - The definition of HTML comment has been changed so that `<!-->` and
     `<!--->` are also recognized as HTML comments.

   - HTML tags recognized as HTML block starting condition of type 4 has been
     updated, namely a tag `<source>` has been removed, whereas `<search>`
     added.

   Refer to [CommonMark 0.31.2](https://spec.commonmark.org/0.31.2/) for full
   specification.

Fixes:

 - [#230](mity/md4c#230):
   The fix [#223](mity/md4c#223) in 0.5.1 release
   was incomplete and one corner case remained unfixed. This is now addressed.

 - [#231](mity/md4c#231):
   `md2html --full-html` now emits `<meta  charset="UTF-8">` in the HTML header.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants