[FFmpeg-devel] [PATCH] Do not use ENOTSUP

Michael Niedermayer michaelni at gmx.at
Tue Dec 20 19:03:23 CET 2011


On Tue, Dec 20, 2011 at 06:38:03PM +0100, Nicolas George wrote:
> Thanks for the report.
> 
> Le decadi 30 frimaire, an CCXX, Rafaël Carré a écrit :
> > -        case CELT_UNIMPLEMENTED:    return AVERROR(ENOTSUP);
> > +        case CELT_UNIMPLEMENTED:    return AVERROR_PATCHWELCOME;
> 
> PATCHWELCOME means "Not yet implemented in FFmpeg, patches welcome", but
> CELT_UNIMPLEMENTED means that the problem is in libcelt. ENOSYS would
> probably be a good replacement.
> 
> >      if (flags || stream_index > 0)
> > -        return AVERROR(ENOTSUP);
> > +        return AVERROR_PATCHWELCOME;
> 
> Here too, this is not something that can be fixed. EINVAL would probably
> mean mostly the same thing.

non zero flags should be quite fixable
stream_index>0 not


> 
> I can fix it that way in my working tree, or you can send your patch again,
> but I would like you tu confirm that ENOSYS would work for you before
> pushing anything.

ENOSYS is ok, its used all over the ffmpeg code

[...]


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- 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/20111220/867a9804/attachment.asc>


More information about the ffmpeg-devel mailing list