[FFmpeg-devel] [PATCH 2/2] encode: copy the context timebase to the packet timebase

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Mon Aug 2 15:55:41 EEST 2021


Andreas Rheinhardt:
> Lynne:
>> (One-line) Patch attached:
>>
>>> --- a/libavcodec/encode.c 
>>> +++ b/libavcodec/encode.c
>>> @@ -44,6 +44,7 @@ int ff_alloc_packet(AVCodecContext *avctx, AVPacket *avpkt, int64_t size)
>>>       av_fast_padded_malloc(&avctx->internal->byte_buffer,
>>>                             &avctx->internal->byte_buffer_size, size);
>>>       avpkt->data = avctx->internal->byte_buffer;
>>> +    avpkt->time_base = avctx->time_base;
>>>       if (!avpkt->data) {
>>>           av_log(avctx, AV_LOG_ERROR, "Failed to allocate packet of size %"PRId64"\n", size);
>>>           return AVERROR(ENOMEM);
>>
>> I'm not sure if this is the correct place to do this.
>>
> 
> Does this not need lots of FATE updates due to 1/2? I am quite surprised.
> 
I am not surprised anymore: We do not run ffprobe on freshly encoded
packets, so it is no wonder FATE doesn't need updates after 1/2.

- Andreas


More information about the ffmpeg-devel mailing list