[FFmpeg-devel] [PATCH v2 5/7] avformat: add apm muxer

Zane van Iperen zane at zanevaniperen.com
Thu Jun 11 05:05:37 EEST 2020


On Thu, 11 Jun 2020 03:25:26 +0200
"Andreas Rheinhardt" <andreas.rheinhardt at gmail.com> wrote:

> > +        av_log(s, AV_LOG_ERROR,
> > +               "Filesize %"PRId64" invalid for APM, output file
> > will be broken\n",
> > +               file_size);
> > +        return AVERROR(EINVAL);  
> 
> EINVAL seems wrong here (which invalid argument has been provided?).
> Maybe ERANGE?
> 

Okay, I'll use ERANGE. Is that the only thing? I'm prepared to send a
v3, but don't want to jump the gun.

Zane



More information about the ffmpeg-devel mailing list