feat: angle brackets if link paren is unbalanced#87
Conversation
e538fa4 to
2638216
Compare
Byron
left a comment
There was a problem hiding this comment.
That's great work!
Particularly the switch to the TextMergeStream has a stunning effect. It's surprising the text-event splitting done otherwise changes the output so much, so I must wonder if this hints at a defect in this crate. From the looks of it the kind of processing the stream does, this crate could do too, there is no private API involved I think.
|
And I forgot, why is one commit marked s And bonus points for a specific unit test so this change isn't 'only' tested by the quite broad stupicat fixture based test. Thanks again. |
Increases passing spec tests from 473 to 477.
This increases the amount of passing tests to 578. It doesn't strictly improve the quality of implementation, but since separate text events aren't supposed to be semantic, these failures seem spurrious.
2638216 to
b3d7df2
Compare
|
Good catch 👍 The As for event-splitting, try running this from the pulldown-cmark repository: $ cargo run --example events
lucky*star
lucky\*starThese two paragraphs mean the same thing, but the text events are different. The spec.txt test suite contains a lot of this stuff, because the whole point of the thing is to demonstrate corner cases in CommonMark. To make sure the event stream was identical, we'd need to avoid spurious backslash escapes, even if the library user has edited the event stream to insert their own text nodes with asterisks in them. |
Byron
left a comment
There was a problem hiding this comment.
Thanks a lot!
And I (think I) see, the event handling with TeextMergeStream doesn't have to be 'embedded' into this crate.
Now I wonder if this should be documented somewhere, along with explanation when one would want to use the one over the other. I'd be most grateful if this is something you could do, maybe even on the crate-level docs.
|
And here is the new release - it turns out there were other breaking changes before it: https://github.com/Byron/pulldown-cmark-to-cmark/releases/tag/v18.0.0 |
No description provided.