James Almer:
> On 4/18/2021 11:06 PM, Andreas Rheinhardt wrote:
>> Mention that avcodec_free_context() always frees it even when
>> encoding. And mention that freeing is of course performed in
>> avcodec_close() when decoding and not necessarily in avcodec_open2().
>>
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com>
>> ---
>> libavcodec/avcodec.h | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
>> index 8a71c04230..9b68aecd31 100644
>> --- a/libavcodec/avcodec.h
>> +++ b/libavcodec/avcodec.h
>> @@ -2011,7 +2011,9 @@ typedef struct AVCodecContext {
>> * [Script Info] and [V4+ Styles] section, plus the [Events]
>> line and
>> * the Format line following. It shouldn't include any Dialogue
>> line.
>> * - encoding: Set/allocated/freed by user (before avcodec_open2())
>> - * - decoding: Set/allocated/freed by libavcodec (by
>> avcodec_open2())
>> + * - decoding: Set/allocated/freed by libavcodec (by avcodec_open2()
>> + * resp. avcodec_close())
>
> What does resp. mean?
>
respectively
>> + * Furthermore, avcodec_free_context() always frees it even when
>> encoding.
>
> What this doxy is missing is the fact the buffer must be allocated with
> av_malloc(), considering we free it with av_free().
>
It is actually currently allowed to use custom allocators when encoding
(which this patch further restricts to the case of encoding and not
using avcodec_free_context()).
> LGTM either way.
>
>> */
>> uint8_t *subtitle_header;
>> int subtitle_header_size;
>>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
