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

Tidy up "sending version negotiation packets" #4721

Merged
merged 1 commit into from
Jan 11, 2021

Conversation

kaduk
Copy link
Contributor

@kaduk kaduk commented Jan 8, 2021

There was a sentence in this section that seems to be specifying
how to handle a version negotiation packet, not how to send one,
and as such is out of place.

There was a sentence in this section that seems to be specifying
how to handle a version negotiation packet, not how to send one,
and as such is out of place.
Copy link
Member

@martinthomson martinthomson left a comment

Choose a reason for hiding this comment

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

I can see how this duplicates text from the next section and so is safe to remove. Thanks for separating and providing independent justification. I missed the justification before.

@martinthomson martinthomson added -transport editorial An issue that does not affect the design of the protocol; does not require consensus. labels Jan 11, 2021
@martinthomson martinthomson merged commit c2956a2 into quicwg:master Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants