Re: [FFmpeg-devel] [PATCH v2 02/27] avcodec/decode: Add new ProgressFrame API

From: Anton Khirnov <anton@khirnov.net>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Subject: Re: [FFmpeg-devel] [PATCH v2 02/27] avcodec/decode: Add new ProgressFrame API
Date: Wed, 10 Apr 2024 09:01:07 +0200
Message-ID: <171273246715.22242.7277960037711067953@lain.khirnov.net> (raw)
In-Reply-To: <GV1P250MB07370D546BF8AB62620CC4088F002@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM>

Quoting Andreas Rheinhardt (2024-04-08 22:13:40)
> diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
> index fd356bd190..6b2c4312e0 100644
> --- a/libavcodec/pthread_frame.c
> +++ b/libavcodec/pthread_frame.c
> @@ -779,6 +779,7 @@ static av_cold int init_thread(PerThreadContext *p, int *threads_to_free,
>      if (!copy->internal)
>          return AVERROR(ENOMEM);
>      copy->internal->thread_ctx = p;
> +    copy->internal->progress_frame_pool = avctx->internal->progress_frame_pool;

I'd still prefer every thread to have its own reference.

Looks good otherwise.

-- 
Anton Khirnov
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".