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.

Sign up for GitHub

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

Jump to bottom

Clarified how to specify missing packets #647

Closed

ianswett wants to merge 1 commit into master from ianswett-clarify-missing

Contributor

ianswett commented Jun 21, 2017

Added a sentence to the ack frame in an effort to close #613


          Clarified how to specify missing packets

1208d8c

ianswett requested a review from martinthomson

June 21, 2017 01:30

martinthomson added -recovery editorial -transport design and removed -recovery editorial labels

martinthomson requested changes

View reviewed changes

Member

martinthomson left a comment

This is a grossly inefficient encoding, but at the point where you have a 255 packet gap, efficiency is probably out the window anyway.

Don't we also have to say that the gap can be 0 also?

mnot removed the design label

ianswett closed this

martinthomson deleted the ianswett-clarify-missing branch

October 12, 2017 23:52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels