Re: [FFmpeg-devel] [PATCH v3 4/6] avformat: add argo_asf muxer

2020-08-05 Thread Zane van Iperen
On Wed, 5 Aug 2020 23:09:45 +0200 "Andreas Rheinhardt" wrote: > > + > > +static int argo_asf_write_trailer(AVFormatContext *s) > > +{ > > +int16_t ret; > > Using an int16_t here is wrong: It may mask errors and apart from that > it may lead to warnings from static analyzers. > Good catch,

Re: [FFmpeg-devel] [PATCH v3 4/6] avformat: add argo_asf muxer

2020-08-05 Thread Andreas Rheinhardt
Zane van Iperen: > Signed-off-by: Zane van Iperen > --- > Changelog| 1 + > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/argo_asf.c | 127 ++- > libavformat/version.h| 2 +- > 5 files changed, 130

[FFmpeg-devel] [PATCH v3 4/6] avformat: add argo_asf muxer

2020-08-04 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- Changelog| 1 + libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/argo_asf.c | 127 ++- libavformat/version.h| 2 +- 5 files changed, 130 insertions(+), 2 deletions(-) d