quinn-rs / quinn

Async-friendly QUIC implementation in Rust
Apache License 2.0
3.76k stars 380 forks source link

Remove use of the unwrap crate from quinn tests #1215

Closed Ralith closed 2 years ago

Ralith commented 2 years ago

This crate was a workaround for unwrap panic messages having a crappy source location, but this was fixed upstream in https://github.com/rust-lang/rust/issues/47809. It's now a superfluous dependency, which could be easily dropped.

Rouzip commented 2 years ago

This crate was a workaround for unwrap panic messages having a crappy source location, but this was fixed upstream in rust-lang/rust#47809. It's now a superfluous dependency, which could be easily dropped.

Hello, I think I can try to fix it.

djc commented 2 years ago

@Rouzip would be great, thanks!

Ralith commented 2 years ago

Fixed in #1221.