[FFmpeg-devel] [PATCH v2] libavformat/mxfenc: added support mxf opatom audio muxing

Mark Reid mindmark at gmail.com
Sun Apr 12 02:03:19 CEST 2015


On Sat, Apr 11, 2015 at 4:58 AM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:

> Tomas Härdin <tomas.hardin <at> codemill.se> writes:
>
> > > +        av_log(s, AV_LOG_ERROR, "MXF OPAtom only supports
> > > 16 or 24 bits le audio\n");
> >
> > This should perhaps be reworded as "Only 16- and 24-bit LE
> > is implemented" since the spec itself allows loads more.
> >
> > > +                    return AVERROR(EINVAL);
>
> If this is correct, the return value should be
> AVERROR_PATCHWELCOME.
>
>
will do, I'll send a new patch with these changes


More information about the ffmpeg-devel mailing list