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

[Hendrik Leppkes]

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