Skip to content

Commit

Permalink
Swap TSS and Stream Cancellation codepoints
Browse files Browse the repository at this point in the history
This is somewhat superficial, but we're changing it anyways so bear with me.  HEADER_ACK and STREAM_CANCEL are almost identical, so code processing them should be next to each other.  By swapping TSS and Stream Cancel codepoints, the processing code might be more readable.
  • Loading branch information
afrind committed Jun 25, 2018
1 parent eedc1d3 commit 5a38c97
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions draft-ietf-quic-qpack.md
Expand Up @@ -524,7 +524,7 @@ server's header blocks and table updates.

### Table State Synchronize

The Table State Synchronize instruction begins with the '01' two-bit pattern.
The Table State Synchronize instruction begins with the '00' two-bit pattern.
The instruction specifies the total number of dynamic table inserts and
duplications since the last Table State Synchronize or Header Acknowledgement
that increased the Largest Known Received dynamic table entry. This is encoded
Expand All @@ -535,7 +535,7 @@ entries might cause a stream to become blocked, as described in
~~~~~~~~~~ drawing
0 1 2 3 4 5 6 7
+---+---+---+---+---+---+---+---+
| 0 | 1 | Insert Count (6+) |
| 0 | 0 | Insert Count (6+) |
+---+---+-----------------------+
~~~~~~~~~~
{:#fig-size-sync title="Table State Synchronize"}
Expand Down Expand Up @@ -592,14 +592,14 @@ table on that stream are no longer outstanding.
An encoder cannot infer from this instruction that any updates to the dynamic
table have been received.

The instruction begins with the '00' two-bit pattern. The instruction includes
The instruction begins with the '01' two-bit pattern. The instruction includes
the stream ID of the affected stream - a request or push stream - encoded as a
6-bit prefix integer.

~~~~~~~~~~ drawing
0 1 2 3 4 5 6 7
+---+---+---+---+---+---+---+---+
| 0 | 0 | Stream ID (6+) |
| 0 | 1 | Stream ID (6+) |
+---+---+-----------------------+
~~~~~~~~~~
{:#fig-stream-cancel title="Stream Cancellation"}
Expand Down

0 comments on commit 5a38c97

Please sign in to comment.