[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