Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

get_delay() undefined in loss detection pseudocode #142

Closed
martinduke opened this issue Jan 13, 2017 · 2 comments
Closed

get_delay() undefined in loss detection pseudocode #142

martinduke opened this issue Jan 13, 2017 · 2 comments
Assignees
Labels
-recovery editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@martinduke
Copy link
Contributor

In draft-ietf-quic-recovery.md

` if (rto_count = 0):
alarm_duration = max(200ms, smoothed_rtt + 4 x rttvar);
else:
alarm_duration = loss_detection_alarm.get_delay() << 1;
rto_count++;

The return value of get_delay() is not defined anywhere in the document.

@martinthomson martinthomson added -recovery editorial An issue that does not affect the design of the protocol; does not require consensus. labels Jan 13, 2017
@martinthomson
Copy link
Member

@ianswett, correct this if it isn't purely editorial.

@ianswett
Copy link
Contributor

It's editorial, as well as not defined. I'll fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-recovery editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

No branches or pull requests

3 participants