Skip to content

Replace unnecessary uses of TraitObject with casts #36005

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

Merged
merged 1 commit into from
Aug 27, 2016
Merged

Replace unnecessary uses of TraitObject with casts #36005

merged 1 commit into from
Aug 27, 2016

Conversation

apasel422
Copy link
Contributor

@eddyb
Copy link
Member

eddyb commented Aug 26, 2016

LGTM, nice catch!

@alexcrichton
Copy link
Member

@bors: r+ 47a7cdc85243cad70aa024e6270cc3c83b6b66f2

Nice!

@bors
Copy link
Collaborator

bors commented Aug 26, 2016

☔ The latest upstream changes (presumably #35906) made this pull request unmergeable. Please resolve the merge conflicts.

@apasel422
Copy link
Contributor Author

Rebased.

@eddyb
Copy link
Member

eddyb commented Aug 26, 2016

@bors r=alexcrichton

@bors
Copy link
Collaborator

bors commented Aug 26, 2016

📌 Commit 2b10df7 has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Aug 26, 2016

⌛ Testing commit 2b10df7 with merge a5d320c...

@bors
Copy link
Collaborator

bors commented Aug 26, 2016

💔 Test failed - auto-win-msvc-64-opt-rustbuild

@apasel422
Copy link
Contributor Author

@bors: retry

Manishearth added a commit to Manishearth/rust that referenced this pull request Aug 27, 2016
Replace unnecessary uses of `TraitObject` with casts

r? @alexcrichton
bors added a commit that referenced this pull request Aug 27, 2016
Rollup of 7 pull requests

- Successful merges: #35124, #35877, #35953, #36002, #36004, #36005, #36014
- Failed merges:
@bors bors merged commit 2b10df7 into rust-lang:master Aug 27, 2016
@apasel422 apasel422 deleted the traitobj branch August 27, 2016 13:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants