Rename min_guaranteed_mtu to min_mtu for clarity

quinn-rs / quinn

Async-friendly QUIC implementation in Rust

Apache License 2.0

3.85k stars 394 forks source link

Rename min_guaranteed_mtu to min_mtu for clarity #1551

Closed Ralith closed 1 year ago

Ralith commented 1 year ago

Depends on #1529.

aochagavia commented 1 year ago

We should probably rename usages of min_guaranteed_mtu in other places too. See the function parameters of MtuDiscovery::{new, disabled, with_state} and BlackHoleDetector::new, and the min_guaranteed_mtu field of BlackHoleDetector.

Ralith commented 1 year ago

Good catch, thanks; fixed.

aochagavia commented 1 year ago

@Ralith I have created #1552 to save you the hassle of rebasing 😉