Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

reference full path DefaultHasher #37579

Merged
merged 1 commit into from
Nov 6, 2016
Merged

reference full path DefaultHasher #37579

merged 1 commit into from
Nov 6, 2016

Conversation

liigo
Copy link
Contributor

@liigo liigo commented Nov 4, 2016

Since DefaultHasher does not in scope, it's not obvious where to find it.

@rust-highfive
Copy link
Collaborator

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Nov 4, 2016

📌 Commit a5f6aa1 has been approved by alexcrichton

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Nov 4, 2016
@alexcrichton
Rollup merge of rust-lang#37579 - liigo:defaulthasher, r=alexcrichton
db6f52c 
reference full path DefaultHasher

Since `DefaultHasher` does not in scope, it's not obvious where to find it.
@alexcrichton alexcrichton mentioned this pull request Nov 4, 2016
Merged
bors added a commit that referenced this pull request Nov 5, 2016
@bors
Auto merge of #37597 - alexcrichton:rollup, r=alexcrichton
16d5330 
Rollup of 24 pull requests

- Successful merges: #37255, #37317, #37408, #37410, #37422, #37427, #37470, #37501, #37537, #37556, #37557, #37564, #37565, #37566, #37569, #37574, #37577, #37579, #37583, #37585, #37586, #37587, #37589, #37596
- Failed merges: #37521, #37547
sophiajt pushed a commit to sophiajt/rust that referenced this pull request Nov 5, 2016
Rollup merge of rust-lang#37579 - liigo:defaulthasher, r=alexcrichton
c6bd7fe 
reference full path DefaultHasher

Since `DefaultHasher` does not in scope, it's not obvious where to find it.
@sophiajt sophiajt mentioned this pull request Nov 5, 2016
Closed
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Nov 5, 2016
@alexcrichton
Rollup merge of rust-lang#37579 - liigo:defaulthasher, r=alexcrichton
50def8e 
reference full path DefaultHasher

Since `DefaultHasher` does not in scope, it's not obvious where to find it.
bors added a commit that referenced this pull request Nov 6, 2016
@bors
Auto merge of #37597 - alexcrichton:rollup, r=alexcrichton
3fc8304 
Rollup of 24 pull requests

- Successful merges: #37255, #37317, #37408, #37410, #37422, #37427, #37470, #37501, #37537, #37556, #37557, #37564, #37565, #37566, #37569, #37574, #37577, #37579, #37583, #37585, #37586, #37587, #37589, #37596
- Failed merges: #37521, #37547
@bors bors merged commit a5f6aa1 into rust-lang:master Nov 6, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@aturon aturon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@liigo @rust-highfive @alexcrichton @bors @aturon