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

[Michael Niedermayer]

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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110422/09f2b02d/attachment.asc>