[FFmpeg-devel] [PATCH 2/3] lavf/avio: temporarily accept 0 as EOF.

Jan Ekstrom jeebjp at gmail.com
Fri Oct 27 21:56:32 EEST 2017


On Fri, Oct 27, 2017 at 9:46 PM, Nicolas George <george at nsup.org> wrote:
> Print a warning to let applicatios fix their use.
> After a deprecation period, check with a low-level assert.
> Also make the constraint explicit in the doxygen comment.

Difference to the previous version:
> -        av_log(s, AV_LOG_WARNING, "Invalid return value 0 for stream protocol\n");
> +        av_log(NULL, AV_LOG_WARNING, "Invalid return value 0 for stream protocol\n");

Thus LGTM.

Jan


More information about the ffmpeg-devel mailing list