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

Stefano Sabatini stefano.sabatini-lala at poste.it
Fri Apr 22 10:42:07 CEST 2011


On date Friday 2011-04-22 03:12:42 +0200, Michael Niedermayer encoded:
> On Fri, Apr 22, 2011 at 01:34:34AM +0200, Stefano Sabatini wrote:
> > 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?
> 
> sure

Pushed.
-- 
FFmpeg = Fostering & Frenzy Merciless Philosofic Experimenting Genius


More information about the ffmpeg-devel mailing list