Hi, On Wed, Nov 13, 2019 at 2:00 AM <quinkbl...@foxmail.com> wrote:
> From: Zhao Zhili <zhiliz...@tencent.com> > > vp7 decoder doesn't set update_thread_context field > --- > libavcodec/vp8.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/vp8.c b/libavcodec/vp8.c > index eb51d1f3c9..b4deb3ed67 100644 > --- a/libavcodec/vp8.c > +++ b/libavcodec/vp8.c > @@ -2715,7 +2715,8 @@ int vp78_decode_frame(AVCodecContext *avctx, void > *data, int *got_frame, > > s->next_framep[VP56_FRAME_CURRENT] = curframe; > > - ff_thread_finish_setup(avctx); > + if (avctx->codec->update_thread_context) > + ff_thread_finish_setup(avctx); > OK, I guess. What's the actual problem solved by this patch (e.g. some commandline invocation with particular settings)? Does it cause some assertion with frame-multithreaded VP7 decoding? Maybe add that to the commit message so we know how to test this change. Ronald _______________________________________________ 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".
