-
Notifications
You must be signed in to change notification settings - Fork 677
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
(CC refactor 9) recovery: move congestion into module #1780
base: cc-refactor-8-cc-struct
Are you sure you want to change the base?
Conversation
on_packet_sent, | ||
on_packets_acked, | ||
congestion_event, | ||
collapse_cwnd, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why reset() and collape_cwnd() is removed? (I understand it's not used, but no plan to implement persistent congestion in the future?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll probably want to do it at some point... but that's been the case for years, so for now removing them simplifies the refactoring a bit. We can add them back when needed.
AFAICT these were never actually used (as in, read) anywhere.
505df84
to
fb68839
Compare
9b27bd4
to
0152913
Compare
No description provided.