On Sat, Apr 11, 2015 at 4:58 AM, Carl Eugen Hoyos <ceho...@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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to