[FFmpeg-devel] [PATCH 08/23] fftools/ffmpeg: use sync queues for enforcing audio frame size

[Michael Niedermayer]

On Sat, Mar 25, 2023 at 08:15:14PM +0100, Anton Khirnov wrote:
> The code currently uses lavfi for this, which creates a sort of
> configuration dependency loop - the encoder should be ideally
> initialized with information from the first audio frame, but to get this
> frame one needs to first open the encoder to know the frame size. This
> necessitates an awkward workaround, which causes audio handling to be
> different from video.
> 
> With this change, audio encoder initialization is congruent with video.
> ---
>  fftools/ffmpeg.c          | 58 ++++++++-------------------------------
>  fftools/ffmpeg_filter.c   |  8 ------
>  fftools/ffmpeg_mux_init.c | 19 +++++++++----
>  3 files changed, 25 insertions(+), 60 deletions(-)

this results in the following to be apparently stuck

ffmpeg -y -i https://samples.ffmpeg.org/V-codecs/geov.avi -t 1 file.avi


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20230325/94e20543/attachment.sig>