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

Nicolas George nicolas.george at normalesup.org
Tue Dec 20 18:38:03 CET 2011


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.

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.

In fact, if you have time, I would be grateful if you could send me a
complete list of the EFOOBAR codes that your build environment supports,
along with its exact name and version. Building such a list for the various
supported systems was one of my immediate projects.

Regards,

-- 
  Nicolas George
-------------- 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/b39b6f90/attachment.asc>


More information about the ffmpeg-devel mailing list