[FFmpeg-devel] [PATCH v4 0/5] AMV muxing and adpcm_ima_amv encoding support

[Zane van Iperen]

On 5/11/20 11:48 pm, Zane van Iperen wrote:
> Adds support for encoding to adpcm_ima_amv and muxing to AMV.
> 
> Things to note:
> - AMV is a hard-coded (and broken) subset of AVI. It's not worth sullying the
>    existing AVI muxer with its filth.
> - The muxer is extremely pedantic about its streams, this is to
>    not break certain software players.
> - The sizes of certain tags are deliberately set to 0 as some players break
>    when they're set correctly. Ditto with some header fields.
> - Players can be **very** sensitive to the frame order and sizes.
>    - Frames must be strictly interleaved as V-A, any V-V or A-A will
>      cause crashes.
>    - Variable audio frame sizes cause crashes.
>    - If audio is shorter than video, it's padded with silence.
>    - If video is shorter than audio, the most recent frame is repeated.
> 
> v4:
>    - Always use the coded sample count if its a sensible value
>    - Fix decoder odd sample check
> 
> v3: [2]
>   - Use ffio_fill()
> 
> v2: [1]
>   - Unref allocated audio frame
>   - Check return value of av_packet_ref()
>   - Remove unnecessary initialisations
>   - Add deinit() function
> 
> [1]: https://ffmpeg.org/pipermail/ffmpeg-devel/2020-November/271837.html
> [2]: https://ffmpeg.org/pipermail/ffmpeg-devel/2020-November/271882.html

Ping. Will apply in a few days.