Skip to content
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.

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

Implement defaults for associated types #25796

Merged
merged 4 commits into from
May 28, 2015
Merged

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented May 26, 2015

r? @eddyb

Fixes #19476.

@eddyb
Copy link
Member

eddyb commented May 26, 2015

LGTM. Doesn't this close one or more issues?

@arielb1
Copy link
Contributor Author

arielb1 commented May 26, 2015

Not any that I know.

@eddyb
Copy link
Member

eddyb commented May 26, 2015

@bors r+

@bors
Copy link
Contributor

bors commented May 26, 2015

📌 Commit 65a3245 has been approved by eddyb

@bors
Copy link
Contributor

bors commented May 27, 2015

⌛ Testing commit 65a3245 with merge c8d3aa8...

@bors
Copy link
Contributor

bors commented May 27, 2015

💔 Test failed - auto-mac-64-opt

@nikomatsakis
Copy link
Contributor

@bors r=eddyb

@bors
Copy link
Contributor

bors commented May 27, 2015

📌 Commit 699fc80 has been approved by eddyb

@bors
Copy link
Contributor

bors commented May 27, 2015

⌛ Testing commit 699fc80 with merge 2058f87...

@bors
Copy link
Contributor

bors commented May 27, 2015

⛄ The build was interrupted to prioritize another pull request.

bors added a commit that referenced this pull request May 27, 2015
@bors
Copy link
Contributor

bors commented May 27, 2015

⌛ Testing commit 699fc80 with merge f3819f0...

@bors bors merged commit 699fc80 into rust-lang:master May 28, 2015
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.

Implement defaults for associated types
5 participants