Skip to content

Conversation

@vks
Copy link
Contributor

@vks vks commented Aug 29, 2014

Updates #15294.

@alexcrichton
Copy link
Member

Could these both actually be #[deriving(Hash)]?

@vks
Copy link
Contributor Author

vks commented Aug 29, 2014

They can, I just forgot about that.

@alexcrichton
Copy link
Member

Thanks! Could you squash commits 1,3, and 4 together as well?

@vks
Copy link
Contributor Author

vks commented Aug 29, 2014

Done.

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Aug 29, 2014
bors added a commit that referenced this pull request Sep 2, 2014
@bors bors closed this Sep 3, 2014
@bors bors merged commit 46e6e42 into rust-lang:master Sep 3, 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.

3 participants