Re: [quicwg/base-drafts] No octets (#1921)

[ianswett <[e-mail]>]

ianswett approved this pull request.

Thanks Martin, this looks great.

> @@ -443,8 +442,8 @@ sent as a probe into the network prior to establishing any packet loss, prior
 unacknowledged packets SHOULD NOT be marked as lost.
 
 A packet sent on an RTO timer MUST NOT be blocked by the sender's congestion
-controller. A sender MUST however count these bytes as additional bytes in
-flight, since this packet adds network load without establishing packet loss.
+controller. A sender MUST however count these bytes as being in flight, since

```suggestion
controller. A sender MUST however count these packets as being in flight, since
```

> @@ -618,9 +617,9 @@ sent_packets:
 : An association of packet numbers to information about them, including a number
   field indicating the packet number, a time field indicating the time a packet
   was sent, a boolean indicating whether the packet is ack-only, a boolean
-  indicating whether it counts towards bytes in flight, and a bytes
-  field indicating the packet's size.  sent_packets is ordered by packet number,
-  and packets remain in sent_packets until acknowledged or lost.  A sent_packets
+  indicating whether it counts towards bytes in flight, and a size field that
+  indicates the packet's size.  sent_packets is ordered by packet number, and

```suggestion
  indicates the packet's size in bytes.  sent_packets is ordered by packet number, and
```

> @@ -965,8 +964,8 @@ losses or increases in the ECN-CE counter.
 ## Tail Loss Probe
 
 A TLP packet MUST NOT be blocked by the sender's congestion controller. The
-sender MUST however count these bytes as additional bytes-in-flight, since a TLP
-adds network load without establishing packet loss.
+sender MUST however count TLP packets against bytes-in-flight, since a TLP adds

Good point, how about we remove the - here, because it's spelled "bytes in flight" everywhere else.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/pull/1921#pullrequestreview-168816001