[FFmpeg-devel] [PATCH] avio: add a destructor for AVIOContext
h.leppkes at gmail.com
Fri Sep 1 15:25:20 EEST 2017
On Fri, Sep 1, 2017 at 2:10 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> 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
> 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.
More information about the ffmpeg-devel