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

Cleaned up deprecation and unused warnings. #4

Merged
merged 2 commits into from
Apr 9, 2017

Conversation

yorickjh
Copy link
Contributor

@yorickjh yorickjh commented Apr 8, 2017

No description provided.

@yorickjh
Copy link
Contributor Author

yorickjh commented Apr 8, 2017

Not terribly useful, but it's a fix that'd need to be done eventually.

Thanks for bearing with me as I ramp up on practical Rust. Hopefully this saves you a breakage when Futures goes to 2.0.

@dwrensha
Copy link
Member

dwrensha commented Apr 9, 2017

Thanks! As I noted here, I think changing the import path for Future, Stream, and Sink is not necessary. But it doesn't hurt, either, so merging.

@dwrensha dwrensha merged commit a61233d into capnproto:master Apr 9, 2017
@yorickjh
Copy link
Contributor Author

yorickjh commented Apr 9, 2017

Agreed that it's probably not necessary...

Filed rust-lang/futures-rs#444

Seems futures-rs is undergoing very rapid refactoring...

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.

2 participants