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.

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

Reword Immediate ACK Recommendation #2424

Merged
merged 8 commits into from Feb 12, 2019
Merged

Reword Immediate ACK Recommendation #2424

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
3755098
Reword Immediate ACK Recommendation
martinduke Feb 6, 2019
1a642b5
Trailing whitespace is annoying
martinthomson Feb 6, 2019
12ecc93
Address review issues.
martinduke Feb 11, 2019
34b3a26
Update draft-ietf-quic-recovery.md
ianswett Feb 11, 2019
7faad22
Resolves all but one comment
martinduke Feb 11, 2019
9bb2b61
Update draft-ietf-quic-recovery.md
ianswett Feb 11, 2019
55a400d
Simplified the text
martinduke Feb 11, 2019
ddb5cf0
Update draft-ietf-quic-recovery.md
ianswett Feb 11, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion draft-ietf-quic-recovery.md
View file
Open in desktop
Expand Up @@ -138,7 +138,7 @@ Crypto Packets:
Out-of-order Packets:

: Packets that do not increase the largest received packet number for its
packet number space by exactly one. Packets usually arrive out of order
packet number space by exactly one. Packets arrive out of order
when earlier packets are lost or delayed.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd be inclined to drop earlier because it's potentially confusing.

Suggested change
when earlier packets are lost or delayed.
when packets are lost or delayed.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know... I think it might read better with 'earlier' included. I don't think it's confusing.


# Design of the QUIC Transmission Machinery
Expand Down