From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 90357402F8 for ; Mon, 23 May 2022 03:26:18 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AA6A868B49C; Mon, 23 May 2022 06:26:17 +0300 (EEST) Received: from out162-62-57-87.mail.qq.com (out162-62-57-87.mail.qq.com [162.62.57.87]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id EE00368B1F7 for ; Mon, 23 May 2022 06:26:10 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foxmail.com; s=s201512; t=1653276363; bh=4tkj2/IkGmMwtGVnWE1JaArO75zqDI+GdHH6CL+dIEk=; h=From:Subject:Date:References:To:In-Reply-To; b=Onjr/vdeqvX4A+A1wKKVt5MVp2Voyqh+92yl3CIVcFP8eBcjgGYSasX2ZP60i0dwn oK4UXDb+Jq/tWYeHuTtDzkCF7bmoxq4TW4sd29wG1/EYl4/GMx3mCebjZOFtH/+n+x suZPzErt9YYOOjXBn9e+HujYWBt6fzX9KwFNnybA= Received: from [192.168.255.10] ([113.108.77.69]) by newxmesmtplogicsvrszc6.qq.com (NewEsmtp) with SMTP id 681AF459; Mon, 23 May 2022 11:26:01 +0800 X-QQ-mid: xmsmtpt1653276361tl3wzbne8 Message-ID: X-QQ-XMAILINFO: Msgy4IdPSbYabN50ozby3LNeMD+1EDvSidANH0/TRm/inDEYUa310SrEmkDQ7B b+x2VN6Ee3OwannfjlOGalsfAcoTcti9BmOWhz2L08DqyeLSwHfCaZvm6jjeC0V13jOjaD3+4G7c FTQuy0iSz9RdVEsf0fJwLbl+UUfz7qdPldglg+SRUylxpZPNKcGBJjho7tBBu2rL083f0ITS/Vfl dsple6WT8epI07maKC3BVON50aczD/bWMHGatIXLLMbat8PxnAKqDL1yZLF+wlHDHfbOUje1j50L KZyf7i3UPxjinxWsdQKpXUcuVv7rperykwlj8ZP9k0+ggBCGCEDe7LVloURnNRXoSFrQ642lXz8d mAfNo/zyzIyUke8fwl8OiAjW/sWY63hfIk15c8guo++1tdKDjpFaJgaDfwcUW9WYeuNLn7l4nCOR AfxIZqGosjEimr0lqOizXWclTwth5WksLk1wv4vsCaVclTVjeP/fhtQ235FHdVqBR8DbYCCyLRhR NNy7W0huP3Cl+M9jpL8jegy7hA7sR8W5s8FUn9cfGj6dNDAjDwHCOAwF9iPfI2Dy1BodYhiO3KG1 P62oto2WBsP17ZnTsTJH+1X5ICjYJGN8CQklbIdTKeu9cydIQ2uZE+KfbAisUDMzR0GmzIRWT9+r tQkQ4zzZcGBj6etRjcFLoI74/pZ0bAFZ5A+DZHHcg+BbF5zgUX7BpnCRtMymngUMaJ2F86YwSbEm YiArqDNRj2o20S1TCtRg0hBk+B0izbBX5SoU2/2PoDngx/2SguhWJJAmX5cGRMa4+cMLOt6u1dAF HVXVHsKbcJZ4vStVLOsfCXclGKy+D+OWinFCL3NVOMGMQngUM9N6pzrucz4IQdTGPowEYwKROqjC SM6LgzY81lN8lP9nUHkZq0pUywkLW/ptZH6s+K4j88mSqNHBbJwJErE7STNyjGEOKu1aUxcacc From: =?utf-8?B?InpoaWxpemhhbyjotbXlv5fnq4spIg==?= Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.60.0.2.21\)) Date: Mon, 23 May 2022 11:26:02 +0800 References: To: FFmpeg development discussions and patches In-Reply-To: X-OQ-MSGID: X-Mailer: Apple Mail (2.3654.60.0.2.21) Subject: Re: [FFmpeg-devel] [PATCH v2 1/3] avcodec/avs3_parser: set has_b_frames properly X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: > On May 13, 2022, at 6:12 PM, Zhao Zhili wrote: > > has_b_frames should be output_reorder_delay field in AVS3 sequence > header and larger than 1. The parser implementation doesn't parse > that field. Decoder can set has_b_frames properly, so use FFMAX > here to avoid resetting has_b_frames from output_reorder_delay to 1. > --- > libavcodec/avs3_parser.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/avs3_parser.c b/libavcodec/avs3_parser.c > index d04d96a03a..a9fd879e9d 100644 > --- a/libavcodec/avs3_parser.c > +++ b/libavcodec/avs3_parser.c > @@ -71,7 +71,7 @@ static void parse_avs3_nal_units(AVCodecParserContext *s, const uint8_t *buf, > if (buf[0] == 0x0 && buf[1] == 0x0 && buf[2] == 0x1) { > if (buf[3] == AVS3_SEQ_START_CODE) { > GetBitContext gb; > - int profile, ratecode; > + int profile, ratecode, low_delay; > > init_get_bits8(&gb, buf + 4, buf_size - 4); > > @@ -114,7 +114,8 @@ static void parse_avs3_nal_units(AVCodecParserContext *s, const uint8_t *buf, > // bitrate_high(12) > skip_bits(&gb, 32); > > - avctx->has_b_frames = !get_bits(&gb, 1); > + low_delay = get_bits(&gb, 1); > + avctx->has_b_frames = FFMAX(avctx->has_b_frames, !low_delay); > > avctx->framerate.num = avctx->time_base.den = ff_avs3_frame_rate_tab[ratecode].num; > avctx->framerate.den = avctx->time_base.num = ff_avs3_frame_rate_tab[ratecode].den; > -- > 2.35.3 > Will apply this patchset tomorrow unless there are objections. _______________________________________________ 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".