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

Jan Ekstrom jeebjp at gmail.com
Sun Oct 29 19:46:04 EET 2017


On Fri, Oct 27, 2017 at 9:56 PM, Jan Ekstrom <jeebjp at gmail.com> wrote:
> 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

The patch set has been applied.


Jan


More information about the ffmpeg-devel mailing list