On Fri, Sep 1, 2017 at 2:10 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote:
> Hi,
>
> On Fri, Sep 1, 2017 at 12:53 AM, James Almer <jamr...@gmail.com> wrote:
>
>> On 8/27/2017 1:31 PM, James Almer wrote:
>> > From: Anton Khirnov <an...@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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to