Skip to content

Add E0517, E0518 for repr() errors#29716

Merged
bors merged 1 commit into
rust-lang:masterfrom
Manishearth:repr-error
Nov 22, 2015
Merged

Add E0517, E0518 for repr() errors#29716
bors merged 1 commit into
rust-lang:masterfrom
Manishearth:repr-error

Conversation

@Manishearth

Copy link
Copy Markdown
Member

r? @gankro

Comment thread src/librustc/diagnostics.rs Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/constrained set/a constrained set/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh whoah. Curly brace enums without field names.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❓ enums always have curly braces?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Man, how tired was I?

I though this was enum Foo { Foo { Bar, Baz } }

@Manishearth Manishearth force-pushed the repr-error branch 2 times, most recently from 9f25ce9 to c03048f Compare November 11, 2015 18:17
@Manishearth

Copy link
Copy Markdown
Member Author

Updated.

@Gankra

Gankra commented Nov 22, 2015

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Nov 22, 2015

Copy link
Copy Markdown
Collaborator

📌 Commit 92df8a9 has been approved by Gankro

@Manishearth

Copy link
Copy Markdown
Member Author

@bors rollup

bors added a commit that referenced this pull request Nov 22, 2015
@bors

bors commented Nov 22, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 92df8a9 with merge e24fffe...

@bors bors merged commit 92df8a9 into rust-lang:master Nov 22, 2015
@Manishearth Manishearth deleted the repr-error branch December 2, 2016 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants