[FFmpeg-devel] [PATCH] avcodec: parse options from AVCodec.bsfs

Nicolas George george at nsup.org
Thu Jul 19 22:31:16 EEST 2018


James Almer (2018-07-19):
> This is more in line with the similar error above. Aborting gracefully
> by printing a message and returning an error value instead of a hard
> abort() with no explanation as to why "ret == AVERROR(ENOMEM)" was expected.

Explaining to whom? If it is a consistency bug within lavc, notifying
the user is worthless, the correct thing to do is to prevent the
inconsistency from getting committed, hence assert and FATE test.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180719/3fe03978/attachment.sig>


More information about the ffmpeg-devel mailing list