[FFmpeg-devel] [PATCH 2/8] avformat: introduce AVStreamGroup

[Anton Khirnov]

Quoting James Almer (2023-12-11 13:53:40)
> On 12/11/2023 8:03 AM, Anton Khirnov wrote:
> > Quoting James Almer (2023-12-05 23:43:56)
> >> +/**
> >> + * Remove a stream group from its AVFormatContext and free it.
> >> + * The group must be the last stream of the AVFormatContext.
> >> + */
> >> +void ff_remove_stream_group(AVFormatContext *s, AVStreamGroup *stg);
> > 
> > When would this be useful?
> 
> I can't think of a use case right now, at least for the raw iamf demuxer 
> and mp4 implementations. I added it for feature parity with AVStream and 
> because it was four lines long.

Removing streams seems like a disgusting hack with very limited usage,
that should really not exist.

I'd rather not pretend we support removing previously added groups,
unless there is a very strong case for it.

-- 
Anton Khirnov