[FFmpeg-devel] [PATCH] matroskaenc: replace AVERROR_NOTSUPP with AVERROR(EINVAL)

Stefano Sabatini stefano.sabatini-lala at poste.it
Fri Apr 22 01:34:34 CEST 2011


On date Thursday 2011-04-21 22:07:28 +0200, Michael Niedermayer encoded:
> On Thu, Apr 21, 2011 at 12:01:02PM +0200, Stefano Sabatini wrote:
> > AVERROR_NOTSUPP has been dropped with the major bump. Since the
> > signalled error denotes a configuration problem, AVERROR(EINVAL) looks
> > an adequate replacement.
> 
> if you prefer to undo that drop it can be done

To undrop AVERROR_NOTSUPP? No it was discussed at lenght:
http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/106104/focus=106778

and we came to the conclusion that AVERROR_NOTSUPP should be dropped
in favor of AVERROR(ENOSYS) or AVERROR(EINVAL) or AVERROR_PATCHWELCOME
depending on the case.

Fine to apply this patch?
-- 
FFmpeg = Fascinating and Frightening MultiPurpose Ecletic God


More information about the ffmpeg-devel mailing list