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.

Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to