-
-
Notifications
You must be signed in to change notification settings - Fork 564
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
Delimiter type of bulleted list #201
Comments
Looks like a bug. This line should probably be removed, or changed to initialize with |
Agreed. I suspect this goes back to a time when we didn't have
CMARK_NO_DELIM. We should initialize with CMARK_NO_DELIM.
+++ Nick Wellnhofer [May 29 17 18:14 ]:
… Looks like a bug. [1]This line should probably be removed, or changed
to initialize with CMARK_NO_DELIM.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, [2]view it on GitHub, or [3]mute the
thread.
References
1. https://github.com/jgm/cmark/blob/6313fff5f9482bdc0489dd19bf9c4ff6ed817e71/src/blocks.c#L418
2. #201 (comment)
3. https://github.com/notifications/unsubscribe-auth/AAAL5NWv19dULwLzWXcj12rurr3oyTRmks5r-22DgaJpZM4NphRW
|
kapyshin
added a commit
to kapyshin/paka.cmark
that referenced
this issue
May 30, 2017
Now that commonmark/cmark#201 is fixed, cmark correctly returns CMARK_NO_DELIM for bulleted lists.
phillmv
added a commit
to eli-schwartz/cmark
that referenced
this issue
Jan 31, 2023
[Autolink] Store positioning info for url_match
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
While testing
cmark_node_get_list_delim
I noticed that it returnsCMARK_PERIOD_DELIM
for bulleted list. This was a bit unexpected (CMARK_NO_DELIM
seems natural). Should it be this way?JavaScript implementation gives
null
in such case:The text was updated successfully, but these errors were encountered: