[PR #242] Added note about invalid block type value in block switch commands #844

Open
opened 2026-01-29 20:51:11 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/google/brotli/pull/242

State: closed
Merged: Yes


Added an explicit note to reject a stream as invalid, if a block type in a block switch command is not within its 0..NBLTYPEx-1 range.
I believe it is of value to mention rejection conditions explicitly in the spec, and with a consistent phrase ("the stream should be rejected as invalid"), to make it easy for implementers to find these conditions through a simple text search.

**Original Pull Request:** https://github.com/google/brotli/pull/242 **State:** closed **Merged:** Yes --- Added an explicit note to reject a stream as invalid, if a block type in a block switch command is not within its 0..NBLTYPEx-1 range. I believe it is of value to mention rejection conditions explicitly in the spec, and with a consistent phrase ("the stream should be rejected as invalid"), to make it easy for implementers to find these conditions through a simple text search.
claunia added the pull-request label 2026-01-29 20:51:11 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/brotli#844