-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
add test for deriving Debug on uninhabited enum #55567
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@durka looks like you need to run |
acf0229
to
918c0b9
Compare
Should be fixed. |
918c0b9
to
2279f62
Compare
@bors r+ rollup |
📌 Commit 2279f62 has been approved by |
// when deriving Debug on an empty enum | ||
|
||
#[derive(Debug)] | ||
enum Void {} //~ WARN never used |
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.
Usually I just do #[allow(dead_code)]
, but it's fine this way too.
…ikomatsakis add test for deriving Debug on uninhabited enum Adds a test to close rust-lang#38885.
Rollup of 5 pull requests Successful merges: - #54162 (Hide default impls items) - #55555 (Make `-Z ls` list the actual filename of external dependencies) - #55567 (add test for deriving Debug on uninhabited enum) - #55568 (test that rustdoc doesn't overflow on a big enum) - #55598 (publish-toolstate: ping maintainers when a tool builds again) Failed merges: r? @ghost
Adds a test to close #38885.