[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