-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
rustdoc: don't ICE on TyKind::Typeof
#103006
Conversation
r? @jsha (rust-highfive has picked a reviewer for you, use r? to override) |
no test? 😞 |
I've added a test, is it better now, sweetheart? 🥺 @matthiaskrgr |
<3 😂 |
@@ -0,0 +1,4 @@ | |||
struct Struct { | |||
y: (typeof("hey"),), | |||
//~^ `typeof` is a reserved keyword but unimplemented |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For future reference, I think it's conventional to do //~^ ERROR
just to make sure we're asserting the kind of diagnostic here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ooops, I forgor 💀
…ler-errors rustdoc: don't ICE on `TyKind::Typeof` Fixes rust-lang#102986 I'm not sure why rustdoc started seeing `TyKind::Typeof` all of a sudden (the code being editted was last touched 3 months ago), probably something to do with error recovery? idk.
…iaskrgr Rollup of 7 pull requests Successful merges: - rust-lang#103000 (Add suggestion to the "missing native library" error) - rust-lang#103006 (rustdoc: don't ICE on `TyKind::Typeof`) - rust-lang#103008 (replace ReErased with fresh region vars in opaque types) - rust-lang#103011 (Improve rustdoc `unsafe-fn` GUI test) - rust-lang#103013 (Add new bootstrap entrypoints to triagebot) - rust-lang#103016 (Ensure enum cast moves) - rust-lang#103021 (Add links to relevant pages to find constraint information) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Fixes #102986
I'm not sure why rustdoc started seeing
TyKind::Typeof
all of a sudden (the code being editted was last touched 3 months ago), probably something to do with error recovery? idk.