Hi Barry and Eliny, LGTM. Thank you for the patch. Is it possible to provide the clip so we can add it to our CI?
ci like https://github.com/ffvvc/FFmpeg/actions On Fri, Apr 19, 2024 at 9:48 PM Jun Zhao <myp...@gmail.com> wrote: > From: Jun Zhao <mypopy...@gmail.com> > > Use dpb_max_num_reorder_pics to control output instead of > dpb_max_dec_pic_buffering, when dpb_max_dec_pic_buffering > is much larger than dpb_max_num_reorder_pics, it may cause > dpb overflow error. > > Signed-off-by: Jun Zhao <barryjz...@tencent.com> > Signed-off-by: elinyhuang <elinyhu...@tencent.com> > --- > libavcodec/vvc/refs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vvc/refs.c b/libavcodec/vvc/refs.c > index a5ee7338d6..6694bc4c51 100644 > --- a/libavcodec/vvc/refs.c > +++ b/libavcodec/vvc/refs.c > @@ -226,7 +226,7 @@ int ff_vvc_output_frame(VVCContext *s, VVCFrameContext > *fc, AVFrame *out, const > > /* wait for more frames before output */ > if (!flush && s->seq_output == s->seq_decode && sps && > - nb_output <= > sps->r->sps_dpb_params.dpb_max_dec_pic_buffering_minus1[sps->r->sps_max_sublayers_minus1] > + 1) > + nb_output <= > sps->r->sps_dpb_params.dpb_max_num_reorder_pics[sps->r->sps_max_sublayers_minus1]) > return 0; > > if (nb_output) { > -- > 2.25.1 > > _______________________________________________ > 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".
