Skip to content

Conversation

@huonw
Copy link
Contributor

@huonw huonw commented Nov 16, 2014

Useful for #[deriving].

Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: Could you add spaces between the commas?

@eddyb
Copy link
Member

eddyb commented Nov 16, 2014

We should really have a lint for these. Reminds me of the hypothetical #[deriving(Data)], actually.

@ghost
Copy link

ghost commented Nov 16, 2014

These should probably also implement Hash but that's not possible until #16530 is fixed.

@alexcrichton
Copy link
Member

r=me with some spaces added

@huonw huonw force-pushed the more-marker-impls branch from 86823a8 to f975243 Compare November 16, 2014 22:26
bors added a commit that referenced this pull request Nov 17, 2014
@bors bors closed this Nov 17, 2014
@bors bors merged commit f975243 into rust-lang:master Nov 17, 2014
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