[FFmpeg-devel] [PATCH] avio: add a destructor for AVIOContext

[James Almer]

On 9/1/2017 9:25 AM, Hendrik Leppkes wrote:
> On Fri, Sep 1, 2017 at 2:10 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>> Hi,
>>
>> On Fri, Sep 1, 2017 at 12:53 AM, James Almer <jamrial at gmail.com> wrote:
>>
>>> On 8/27/2017 1:31 PM, James Almer wrote:
>>>> From: Anton Khirnov <anton at khirnov.net>
>>>>
>>>> Before this commit, AVIOContext is to be freed with a plain av_free(),
>>>> which prevents us from adding any deeper structure to it.
>>
>>
>> The docs for avio_alloc_context() state:
>>
>> 441  * Allocate and initialize an AVIOContext for buffered I/O. It must be
>> later
>> 442  * freed with av_free().
>>
>> Should that be updated?
>>
> 
> It should. While av_free remains "OK" for now (deprecation period
> etc), documentation should of course mention the appropriate function.
> 
> - Hendrik

Updated. Thanks for noticing.