Skip to content

Remove stream_options - #2385

Merged
WyattBlue merged 1 commit into
mainfrom
patch
Aug 18, 2026
Merged

Remove stream_options#2385
WyattBlue merged 1 commit into
mainfrom
patch

Conversation

@WyattBlue

Copy link
Copy Markdown
Member

No description provided.

@WyattBlue
WyattBlue force-pushed the patch branch 3 times, most recently from 48a81a9 to 9472d12 Compare August 18, 2026 04:20
@WyattBlue WyattBlue changed the title Move stream_options to InputContainer Remove stream_options Aug 18, 2026
They reached only `avformat_find_stream_info()`, and only for formats
that expose their streams before it runs: anything else raised, so MPEG
and friends could never use them. Output containers rejected them
outright, pointing at `add_stream(options=)`. Type hints were wrong
since they were added, yet zero people complained.

`options` still reaches every stream at probe time,
`stream.codec_context.options` covers one stream and works on every
format, and add_stream(options=) covers writing. Object sizes:

    Container        184 -> 176
    InputContainer   184 -> 176
    OutputContainer  208 -> 200

Also group Container's fields by kind and declare `name` as str.
@WyattBlue
WyattBlue merged commit 9acb1f2 into main Aug 18, 2026
8 checks passed
@WyattBlue
WyattBlue deleted the patch branch August 18, 2026 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant