[FFmpeg-devel] [PATCH] avio: deprecate url_feof

Reimar Döffinger Reimar.Doeffinger
Tue Mar 8 17:54:01 CET 2011


On 7 Mar 2011, at 23:46, "Ronald S. Bultje" <rsbultje at gmail.com> wrote:
> On Mon, Mar 7, 2011 at 5:43 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> On Mon, Mar 07, 2011 at 05:34:10PM -0500, Ronald S. Bultje wrote:
>>> Hi,
>>> 
>>> On Mon, Mar 7, 2011 at 5:31 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>>>> On Mon, Mar 07, 2011 at 05:29:14PM -0500, Ronald S. Bultje wrote:
>>>>> On Mon, Mar 7, 2011 at 5:22 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>>>>>> On Mon, Mar 07, 2011 at 09:50:25PM +0100, Anton Khirnov wrote:
>>>>>>> AVIOContext.eof_reached should be used directly instead.
>>>>>> 
>>>>>> If you are at eof, and some other process (wget?) appends to the file
>>>>>> url_feof() could return 0 again. with direct access this is harder
>>>>> 
>>>>> url_feof() reutrns s->eof_reached??
>>>> 
>>>> you can put something else in there like
>>>> if(protocol.check_eof()) return protocol.check_eof();
>>> 
>>> Right, but that wasn't there for the past 5 years, so it's likely OK.
>> 
>> the API hasnt changed in 5 years and so its ok to make it worse?
>> nice reasoning
> 
> Ok, so do you want to make it a static av_always_inline avio_eof()
> function then?

That isn't much better, improving its behaviour would still require an application recompile that way.



More information about the ffmpeg-devel mailing list