On Fri, Apr 19, 2024 at 9:55 PM Jun Zhao <myp...@gmail.com> wrote: > From: Jun Zhao <mypopy...@gmail.com> > > has_b_frames used in decoder for size of the frame reordering > buffer, setting this field from dpb_max_num_reorder_pics. > > Signed-off-by: Jun Zhao <barryjz...@tencent.com> > --- > libavcodec/vvc_parser.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/vvc_parser.c b/libavcodec/vvc_parser.c > index a0e10e1a7c..5373875aae 100644 > --- a/libavcodec/vvc_parser.c > +++ b/libavcodec/vvc_parser.c > @@ -185,8 +185,8 @@ static void set_parser_ctx(AVCodecParserContext *s, > AVCodecContext *avctx, > avctx->color_range = > sps->vui.vui_full_range_flag ? AVCOL_RANGE_JPEG : > AVCOL_RANGE_MPEG; > > - avctx->has_b_frames = (sps->sps_max_sublayers_minus1 + 1) > 2 ? 2 : > - sps->sps_max_sublayers_minus1; > + avctx->has_b_frames = > + > sps->sps_dpb_params.dpb_max_num_reorder_pics[sps->sps_max_sublayers_minus1]; > Should we relocate this to the decoder? Other codecs typically set this parameter in the decoder.
thank you > > if (sps->sps_ptl_dpb_hrd_params_present_flag && > sps->sps_timing_hrd_params_present_flag) { > -- > 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".