-
Notifications
You must be signed in to change notification settings - Fork 13.4k
rustdoc crashes parsing strange doc-comment on 1.36 #62571
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Comments
by process of elimination I determined this was the offending doc-comment:
|
Excuse me? What kind of bug is this? ^^ Thanks for finding. I managed to reduce it to this: /// __|_ _|__||_|
pub struct A;
|
This is an issue in the pulldown-cmark parser and I think it has already been fixed in pulldown-cmark/pulldown-cmark@238d827, pulldown-cmark/pulldown-cmark@b931bb4 and pulldown-cmark/pulldown-cmark@a4e41df (see pulldown-cmark/pulldown-cmark#306) @raphlinus are you planning to release a version 0.5.3 soon so rustdoc can use that to fix this issue? |
@Centril Are there any plans of introducing a new tag, a la S-waiting-for-upstream? This could be useful for LLVM issues and this one in particular? What do you think? Can you bring in that idea to the next Rust meeting? |
We can release an 0.5.3 soon. The other thing I probably want to get in is pulldown-cmark/pulldown-cmark#356 (just the main emphasis, not the table stuff), but we don't have a patch for that yet. I'll see what I can do. |
Added to T-release agenda. |
@raphlinus Please ping us once the release is done. |
We discussed this in today's release team meeting, and we decided not to add the label. There isn't really a difference in the way we would triage |
We just released 0.5.3 and it contains this fix among others. |
Thanks @raphlinus! Sending the update PR right away. |
…earth Update pulldown-cmark version Fixes #62571. Fixes #62770. Fixes #62552. cc @rust-lang/rustdoc @Centril @pietroalbini
After upgrading to 1.36 rustdoc is crashing like this when run on my crate:
It doesn't produce the line that caused it to crash and the crate is quite large.
It seems to correspond with perhaps this issue in pulldown-cmark:
pulldown-cmark/pulldown-cmark#306
The text was updated successfully, but these errors were encountered: