quinn-rs / quinn

Async-friendly QUIC implementation in Rust
Apache License 2.0
3.85k stars 394 forks source link

Replace futures-util with optional futures-io dep #1263

Closed Ralith closed 2 years ago

Ralith commented 2 years ago

A trivial amount of additional code lets us save another dep and be a bit more accessible to new users (see e.g. #1262).

Ralith commented 2 years ago

This also improves consistency with tokio primitives, e.g. mpsc::Receiver::recv.

djc commented 2 years ago

Should we use futures-io as the optional dep rather than futures-util? Otherwise this looks great!

Ralith commented 2 years ago

Good catch, I didn't know that crate existed! Fixed.