Skip to content

Derived Hash impl is duck-typed #21160

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.

Sign up for GitHub

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

Jump to bottom
Closed
apasel422 opened this issue Jan 14, 2015 · 3 comments · Fixed by #21161
Closed

Derived Hash impl is duck-typed #21160

apasel422 opened this issue Jan 14, 2015 · 3 comments · Fixed by #21161
Labels
P-medium Medium priority
Milestone
1.0 beta

Comments

@apasel422
Copy link
Contributor

The following compiles successfully with rustc 1.0.0-nightly (170c4399e 2015-01-14 00:41:55 +0000):

struct Bar;
impl Bar {
    fn hash<T>(&self, _: T) {}
}
#[derive(Hash)]
struct Foo(Bar);

The derived code should probably use UFCS.
@apasel422
Copy link
Contributor Author

I'm working on a fix for this.

japaric pushed a commit to japaric/rust that referenced this issue Jan 14, 2015
use UFCS in #[deriving(Hash)]
59e9cfa 
expansion now uses `::std::hash::Hash::hash(&*__self_0_0, __arg_0)` instead of
`(*__self_0_0).hash(__arg_0)`

closes rust-lang#21160
@japaric japaric mentioned this issue Jan 14, 2015
Merged
@japaric
Copy link
Member

japaric commented Jan 14, 2015

fix in #21161

@apasel422 sorry, didn't see your message! I should've left a comment here

@apasel422
Copy link
Contributor Author

@japaric That's OK! Glad it's fixed. We should make sure the other deriving implementations don't suffer from the same problem.

@japaric japaric mentioned this issue Jan 15, 2015
Closed
@brson brson added P-medium Medium priority and removed I-nominated labels Jan 15, 2015
@brson brson added this to the 1.0 beta milestone Jan 15, 2015
alexcrichton added a commit to alexcrichton/rust that referenced this issue Jan 15, 2015
@alexcrichton
rollup merge of rust-lang#21161: japaric/ufcs-hash
c3c47f5 
expansion now uses `::std::hash::Hash::hash(&*__self_0_0, __arg_0)` instead of
`(*__self_0_0).hash(__arg_0)`

closes rust-lang#21160

r? @alexcrichton
@apasel422 apasel422 mentioned this issue Nov 18, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P-medium Medium priority
Projects
None yet
Milestone
1.0 beta
Development

Successfully merging a pull request may close this issue.

use UFCS in #[derive(Hash)] japaric/rust
4 participants
@alexcrichton @brson @japaric @apasel422