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

Move references to informational references #190

Merged
merged 2 commits into from
Feb 7, 2017
Merged

Conversation

ianswett
Copy link
Contributor

Moves references to informational references, clarifies that SRTT and RTTVar are computed based on an RFC, and adds todos for the congestion control section for anything formerly in the references section.

Moves references to informational references, clarifies that SRTT and RTTVar are computed based on an RFC, and adds todos for the congestion control section for anything formerly in the references section.
@MikeBishop
Copy link
Contributor

BTW, if they're not referenced in the text, why do they need to be in the references?

@MikeBishop MikeBishop added editorial An issue that does not affect the design of the protocol; does not require consensus. -recovery labels Jan 24, 2017
* This should fix your build errors

* Fix the TLP reference as well
@ianswett
Copy link
Contributor Author

All of these were informational for me when implementing QUIC, so I felt they should be included. I believe most of them should be referenced in the text, so I'll try to make sure as many as possible are referenced in an updated version of this PR.

@ianswett
Copy link
Contributor Author

I just reviewed the references that aren't currently in the text, and they're all congestion control related, which is the next section that I plan to properly document, so they should be referenced soon.

@MikeBishop
Copy link
Contributor

Great -- if they're referenced as {{?RFCXXXX}}, you don't need to (and perhaps can't?) have them explicitly in the refs at the top, so remove them from there as you add the inline references.  Or you can reference things that are explicitly in the references as [RFCXXXX] instead.

@ianswett ianswett merged commit c666124 into master Feb 7, 2017
@MikeBishop MikeBishop deleted the ianswett-patch-2 branch February 7, 2017 22:00
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

Successfully merging this pull request may close these issues.

2 participants