-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Improve long explanation for E0546 #81835
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
Improve long explanation for E0546 #81835
Conversation
r? @varkor (rust-highfive has picked a reviewer for you, use r? to override) |
@GuillaumeGomez I didn't add the reference to #61137 when I pushed, so I don't know if you're going to receive a notification. |
|
||
``` | ||
#![feature(staged_api)] | ||
#![stable(since = "1.0.0", feature = "test")] | ||
|
||
#[unstable(feature = "unstable_fn", issue = "none")] // ok! | ||
#[unstable(feature = "unstable_fn", issue = "none")] |
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.
Please keep the // ok!
.
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.
Ok, I'm going to add it also here then. Isn't it? :)
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.
Arf indeed! Sorry, didn't pay enough attention...
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.
Don't worry, I think now I have everything to do it properly in the net PRs. Thanks!
Thanks! @bors: r+ rollup squash |
📌 Commit ac6c09a has been approved by |
…laumeGomez Rollup of 5 pull requests Successful merges: - rust-lang#81526 (btree: use Option's unwrap_unchecked()) - rust-lang#81742 (Add a note about the correctness and the effect on unsafe code to the `ExactSizeIterator` docs) - rust-lang#81830 (Add long error explanation for E0542) - rust-lang#81835 (Improve long explanation for E0546) - rust-lang#81843 (Add regression test for rust-lang#29821) Failed merges: - rust-lang#81836 (Add long explanation for E0547) r? `@ghost` `@rustbot` modify labels: rollup
Helps with #61137