added #[must_use] for impl Future structs

rustasync / runtime

Empowering everyone to build asynchronous software

https://docs.rs/runtime

Apache License 2.0

862 stars 28 forks source link

Closed ibaryshnikov closed 5 years ago

ibaryshnikov commented 5 years ago

Closes #30

Did I cover all the required cases?

ibaryshnikov commented 5 years ago

Thank you for your help! I covered struct which impls Stream. I guess, there's nothing special about TcpStream, right? https://github.com/rustasync/runtime/blob/master/runtime-raw/src/tcp.rs#L10

yoshuawuyts commented 5 years ago

This is excellent. https://github.com/rust-lang/rust/issues/60797 was filed yesterday. Perhaps we should follow suit? Probably should do that in a follow-up PR (: