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
Renumber frames #1945
Renumber frames #1945
Conversation
For rationale, see https://docs.google.com/spreadsheets/d/1G73qp-GBhXGyKEsZBudzWJY85FSa5txT4MxtVC5ajiE/edit#gid=0 Note that this isn't 100% mechanical. There are some other minor changes: Frames with multiple codepoints are not referred to in the plural in their first mention (because we talk about multiple frame types), and ahead of the figure that shows the fields. I have made the text announcing the figure, and the text announcing the fields consistent. There was a grammatical error (the very last change here).
c87a35a
to
15c59e1
Compare
@@ -2698,24 +2698,24 @@ frames are explained in more detail in {{frame-formats}}. | |||
| Type Value | Frame Type Name | Definition | | |||
|:------------|:---------------------|:-------------------------------| | |||
| 0x00 | PADDING | {{frame-padding}} | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Considering that CONNECTION_CLOSE has a frame type field, where 0x0 means that the error is not frame related, wouldn't it make sense to not use 0x0 at all for frames.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
0 was used for padding intentionally. calloc() and write-in-place implementations are easier this way.
For rationale, see this spreadsheet
Note that this isn't 100% mechanical. There are some other minor
changes:
Frames with multiple codepoints are not referred to in the plural in
their first mention (because we talk about multiple frame types), and
ahead of the figure that shows the fields.
I have made the text announcing the figure, and the text announcing the
fields consistent.
There was a grammatical error (the very last change here).