Skip to content
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

large_enum_variant: Report sizes of variants #5466

Merged
merged 1 commit into from
Apr 15, 2020

Conversation

phansch
Copy link
Member

@phansch phansch commented Apr 15, 2020

This reports the sizes of the largest and second-largest variants.

Closes #5459

changelog: large_enum_variant: Report the sizes of the largest and second-largest variants.

This reports the sizes of the largest and second-largest variants.
@phansch phansch added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 15, 2020
@flip1995 flip1995 added S-waiting-on-bors Status: The marked PR was approved and is only waiting bors and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Apr 15, 2020
@flip1995
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Apr 15, 2020

📌 Commit 69c3e9c has been approved by flip1995

@bors
Copy link
Collaborator

bors commented Apr 15, 2020

🌲 The tree is currently closed for pull requests below priority 1, this pull request will be tested once the tree is reopened

bors added a commit that referenced this pull request Apr 15, 2020
Rollup of 5 pull requests

Successful merges:

 - #5226 (Add lint for explicit deref and deref_mut method calls)
 - #5248 (Add lint on large non scalar const)
 - #5430 (Disallow bit-shifting in integer_arithmetic)
 - #5466 (large_enum_variant: Report sizes of variants)
 - #5468 (Zero single char names)

Failed merges:

r? @ghost

changelog: rollup
@bors bors merged commit 071a590 into rust-lang:master Apr 15, 2020
@phansch phansch deleted the large-enum-variant-output branch May 3, 2020 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: The marked PR was approved and is only waiting bors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"large enum variant" lint should report sizes of variants
3 participants