quinn icon indicating copy to clipboard operation
quinn copied to clipboard

Fix debug assert with reordered ACKs

Open Ralith opened this issue 1 year ago • 3 comments

Fixes https://github.com/quinn-rs/quinn/issues/1889.

On careful examination, I think @tthebst's instinct was correct: the assert was fundamentally wrong, and our logic works as intended without it. I've hence removed the assert and attempted to better document why this is okay, but please double-check my thinking.

Ralith avatar Jun 07 '24 22:06 Ralith