Section 4.2 (comment 4)

[mnot]

Section 4.2: We had trouble parsing this sentence.
Is a word missing? If neither suggestion below is correct, please
clarify what "any frame carrying ... SETTINGS" refers to.

Original:

 A frame size error in a frame that could alter
 the state of the entire connection MUST be treated as a connection
 error ([Section 5.4.1](https://www.rfc-editor.org/authors/rfc9113.html#section-5.4.1)); this includes any frame carrying a field block
 ([Section 4.3](https://www.rfc-editor.org/authors/rfc9113.html#section-4.3)) (that is, HEADERS, PUSH_PROMISE, and CONTINUATION),
 SETTINGS, and any frame with a stream identifier of 0.

Suggestion #1 (... and a SETTINGS frame):
A frame size error in a frame that could alter
the state of the entire connection MUST be treated as a connection
error (Section 5.4.1); this includes any frame carrying a field block
(Section 4.3) (that is, HEADERS, PUSH_PROMISE, and CONTINUATION) and
a SETTINGS frame, and any frame with a stream identifier of 0.

Suggestion #2 (... or a SETTINGS frame):
A frame size error in a frame that could alter
the state of the entire connection MUST be treated as a connection
error (Section 5.4.1); this includes any frame carrying a field block
(Section 4.3) (that is, HEADERS, PUSH_PROMISE, and CONTINUATION) or
a SETTINGS frame, and any frame with a stream identifier of 0.

[Lukasa]

Suggestion 1 is correct.

[martinthomson]

I think that the list structure is supposed to be... includes:

• any frame carrying a field block (...),
• a SETTINGS frame, and
• any frame with a stream identifier of 0.

Which suggests option 3, in my PR.