Re: [FFmpeg-devel] [PATCH v2 5/5] avformat: add amv muxer

2020-11-02 Thread Andreas Rheinhardt
Zane van Iperen: > > > On 3/11/20 10:44 am, Andreas Rheinhardt wrote: >>> +static void amv_write_vlist(AVFormatContext *s, AVCodecParameters *par) >>> +{ >>> +uint8_t buf[FFMAX(AMV_VIDEO_STRH_SIZE, AMV_VIDEO_STRF_SIZE)] = { 0 }; >>> +int64_t tag_list, tag_str; >>> + >>> +av_assert0(pa

Re: [FFmpeg-devel] [PATCH v2 5/5] avformat: add amv muxer

2020-11-02 Thread Zane van Iperen
On 3/11/20 10:44 am, Andreas Rheinhardt wrote: >> +static void amv_write_vlist(AVFormatContext *s, AVCodecParameters *par) >> +{ >> +uint8_t buf[FFMAX(AMV_VIDEO_STRH_SIZE, AMV_VIDEO_STRF_SIZE)] = { 0 }; >> +int64_t tag_list, tag_str; >> + >> +av_assert0(par->codec_id == AV_CODEC_ID_AM

Re: [FFmpeg-devel] [PATCH v2 5/5] avformat: add amv muxer

2020-11-02 Thread Andreas Rheinhardt
Zane van Iperen: > AMV is a hard-coded (and broken) subset of AVI. It's not worth sullying > the existing AVI muxer with its filth. > > Fixes ticket #747. > > Signed-off-by: Zane van Iperen > --- > MAINTAINERS | 1 + > libavformat/Makefile | 1 + > libavformat/allformats.c