Re: [FFmpeg-devel] [PATCH 17/28] ffmpeg: move the mux queue into muxer private data

2022-01-17 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2022-01-13 11:50:48) > > My objections to adding a separately allocated muxing context and to > this MuxStream have not changed. Both incur unnecessary allocations > and indirections and (in case of the latter) loops; I cannot imagine any remotely real situation where

Re: [FFmpeg-devel] [PATCH 17/28] ffmpeg: move the mux queue into muxer private data

2022-01-13 Thread Andreas Rheinhardt
Anton Khirnov: > The muxing queue currently lives in OutputStream, which is a very large > struct storing the state for both encoding and muxing. The muxing queue > is only used by the code in ffmpeg_mux, so it makes sense to restrict it > to that file. > > This makes the first step towards reduci

[FFmpeg-devel] [PATCH 17/28] ffmpeg: move the mux queue into muxer private data

2022-01-11 Thread Anton Khirnov
The muxing queue currently lives in OutputStream, which is a very large struct storing the state for both encoding and muxing. The muxing queue is only used by the code in ffmpeg_mux, so it makes sense to restrict it to that file. This makes the first step towards reducing the scope of OutputStrea