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 AAD7142EFC for ; Fri, 13 May 2022 09:43:58 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 8217A68B444; Fri, 13 May 2022 12:43:55 +0300 (EEST) Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05olkn2083.outbound.protection.outlook.com [40.92.89.83]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id AF1DD68B2A0 for ; Fri, 13 May 2022 12:43:48 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jHrV/XFynM4qeUZWHTJdX/NoFfl01rlk8sADYh4SVaPO1WNn0HfFiLxYlUBW4tW2HgXbT0V+ffkBiMhnZ/gUAVPLCAdDSeNf/qXvRr6EJsU0P7sHf+KlEBU46HPaKddabZM+bcmpCMj0qvBvJREcaf14wJJ1jOklCixbIfa0qCQF6uSxdv7nNH/lwauoQX6sDWDACjb1ynukyWPLV56HOFswdl33hc7WPt4p/jpsQpC5AI8axF/r5rm6rmfvXbsMx6MpkTEGMUY2vriGJ4hIuGriy6ljPFQ8ZYm0NueKmjTf0jm/fC43FwU7Up1RF15bGAQ70saX/o18BdNBc5aScw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=My849LRfvNrjkFVnC+8UYkkDzcM580ESjwBqhcwICl8=; b=VVBIYWVAU67rHYa3KumKsrL30Jb2r0w9opcwK39L0dlkPY8wha3miy+1RXeT9wBj8xFk+9j54qhLJJMX5LMx115ddDlgwodXY3hC4TnCVrF0aeLdCyae2QrXJ+3gzZoakreFBZKtSiLa9Zkz8v+HGao6WR3yl9ViyH8IPOhHZwv12BzO+Iju5+6hJYHUQXd+b30jHQAmWtfVRkk0Av1e2c1iuzRAGCJxCPdUQNd+QkYNmLUtaj/wH12095tGhrKZqK/wDCK7FVFS5AdrFWeBu2bHO07ZUc/D3+0cmuvaUw9LSaxXTKN/jIJjemfBIAvab9ovmlIwu1rh/zU2r4ea9A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=My849LRfvNrjkFVnC+8UYkkDzcM580ESjwBqhcwICl8=; b=L2WRroua+rqVms4ae3PtOvexk7cnBehhBMjTOdbySstk/c6xems0IL6pNXjfjr4zqoxJ3YaeoPWx4EYSwzSdSj0OLYXyXi8lFS2GkQtKivYvw7g+zg7kMD7p55qePcjS0NhpiGgDxSF0151CdokgF+bvIxx0m8imWmCkGg5TJPk/1kC6OuzR8FQc8yNoDDT3BOatI49UtlCcOEzuQfPjuze4faa7wJeg1xUk1vWtBkR0CnbrIdhEgln9wKJ7DI8ekvfNdhiqxVTflJGN8eiIVWsDKUKJC0Dyg7IGo1Uxr/RFqzoUu5olsRWBYBayk5VS0PiJi21D/uv7piGLTWnr5A== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by AM6PR01MB5176.eurprd01.prod.exchangelabs.com (2603:10a6:20b:9c::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5227.23; Fri, 13 May 2022 09:43:46 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c%9]) with mapi id 15.20.5227.023; Fri, 13 May 2022 09:43:46 +0000 Message-ID: Date: Fri, 13 May 2022 11:43:44 +0200 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: From: Andreas Rheinhardt In-Reply-To: X-TMN: [yUSUGOENZpnszZ+jCthwNeIuKjlMv8KB] X-ClientProxiedBy: ZRAP278CA0013.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:10::23) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 5d6b5c13-eb01-4247-8701-08da34c5135e X-MS-TrafficTypeDiagnostic: AM6PR01MB5176:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 2X30AXga2gszDyAaxgCd986+4nQz56LJIkVfhXIx6yeNZsGq+W15IOInSiLRifjZ+rzZN+HKJ5lOzhyyvl5VX+VpQ94EmQxU++eTqhfiPY9eOZZWcdrx199zxIdDyf2uVRivQfIo8H8Suembvif0Ek5IIC/8C8rwjiCsBjs4YfGCYNl3qj08s2j0fDt3gZpqe9lB0q4rLt7D01k9EwElBQWtZu0XZgp2FM0RkaA7DCmgQ4a+S/wsxvWlZM9X8hJu7InidJd6qzcnh55FSTtJ57wBcGmMaRLjUQnNhFhMuevR4G2EYCOolBDOQV27kHR76sfCN+H0usCZJx+w8bNp5RbARPXAsYHEcGs0tHgJWkEhhIdQCL/XumRX8kn13kJWkErgI5O9BI8mBU0YhfkRJlp+6tXj+F4kZbA367ZOxJPJYUY+NzAHBVdfNWqDviufEx4hghhUyYiA9KOMBWvLSPpEstN9giAWuqDs9zEcd1J15tz2+zr9nySkw6YlT5IzPKalsPUBEaEIbnZATFoA83z6RpJPqV6UiqB6qpMgD2NjlEgOvdzZaZ7TWzCgWF5eO38cJFen3j3n1gwmIwIUpQ== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?STM1QUZZZCtVWG1RYzNBbDY1TnA2NTR0eU9CTnVqNmMvaGZkRkVNTU9IRUM5?= =?utf-8?B?RnVhYVFiT0RXYlZwUWRIWTlyNDJPM2tpVW9MY1h6THdpMDJHYjB2UTArZmhu?= =?utf-8?B?UE1STWl5RlR5Yk0rVTZxWHdwd2drdWtETkUrc1JuZmthS21oVEJwVDlPNjBw?= =?utf-8?B?aHMyL1ZWNThVYjFVa1ZyaS9zSXRlNU5oL1ZMSXB0bjI0Z0pSaUdzaG01Z1BQ?= =?utf-8?B?cEdBSHlTOWE4SFErUS9icEYxeGtLZ3JhMDZ6VFRtcFlESkR5S1FtbE1JQUNU?= =?utf-8?B?TURlK1pEVGlIbXVZT2xDbzZ4ZENGL3VnT2lmOXgrM3p5Sy9mZC9SRUl2OEhH?= =?utf-8?B?MUVST0xrdnFwK1pjdWdIMVNidVVBOGhMMXNDcithUUhDeHNpQUM5cEQrY1ZG?= =?utf-8?B?TFZYcG4rY0lpVFR5WWplSEU5U2Y4N0pGNG9WaXhzZlV3YUhScXFGSlN3VlpG?= =?utf-8?B?clZZUHlWVnVoV0ZjWW9ZeVJ5M21yV20wbHo5bTQrcXhrallvc2FnUDlWaHN4?= =?utf-8?B?RkhDdGhHM0NZMWZWVEhPN3ZzeHMxUGY2QW8zUDBiMW5EVkQ1d0JLR1E0N2J5?= =?utf-8?B?MmVzaU4xeVJVeHhlU1QrZ3FVdStKcFlGNGQ1ajIwSDV3TFc0Z3JZdVQ4Y21v?= =?utf-8?B?M0pWenBjeW9vZU51enlxSzVZa1AvZ0E4ZitqRnJodm8xSzNJRTk4MWhFaElD?= =?utf-8?B?eWVBYTVReVlXQkNxQXoxbzcyVEh3eG14a2FxTnJsUk1MWUtEQXdNblNwUkR0?= =?utf-8?B?YlRESnRWcFVYZUN1Y3Q1TTNmVFBkd0RTOENOSjJnSmtacFdab2JRbW1GOFZa?= =?utf-8?B?ZG82UWc5Z2lZL0xpUTFFeUo1UnRjeDZucnFqYk5KTVRVUW1xWFBXOWVtb3Jx?= =?utf-8?B?eFZGTTF5dzVKMEZtbGloUTU4aGMyQzlsaElPM3BXZk9tQjNYZFZHbjNQbU90?= =?utf-8?B?aFBJMCsrUHFhQW5hQnh2UHNEbGRTNzVxWmNpZHN1UVVJYzRtNGxSMXh1OWtq?= =?utf-8?B?MDNIN1dqbEw3OW9TcE1TWGZxOElhWUZYRTVhelVQN1V6Tld1cWkxT0ZKVFRa?= =?utf-8?B?dnNhMWxkcm1EdktYRzREZnROVmRnSFBHQUVXUjIrUG1QbkVMSnFZdzkxclFn?= =?utf-8?B?U1ZmL2FXNk91K1k5bk4va3c2K0dqTkdoMXQ4U2FJbDMyNEdYQVhJeHBZWm51?= =?utf-8?B?UUFzTmpRQTVLcVVPMDhzRmlybGhjMzVQV0tLaDN5TUhRZDl0YkdjcHpkY1pQ?= =?utf-8?B?bGV1b2h5VG93VVN1cUN3V3puR0NWRVZ6dHVKVmhkSVI2SElUQXZWTmc0aFV6?= =?utf-8?B?eDhEd05oRVgwVVVITXRYbjlsa1ZOMHJad3NCV0FIQ3hNZ0F2MlZmSlRaTm9C?= =?utf-8?B?ZitHaHkzZVJvcEg0TE9RVm1XWExsWW85UmJYVXN1QXRVWTZOUlQ4dFNNWElh?= =?utf-8?B?eHRNVWIxek9PUFI3WUdVZERnZFNBN3hoTXlqcUpnRWM0TzNnY05lcTVScGY1?= =?utf-8?B?a1R2aFZoVktxWTZrSlpiNURnUUlyVzMzcDhROTRFbDU5YTEyc09KUlFtTDhS?= =?utf-8?B?SGlzbEovd0p1SkVVVG93enhnTWhlVUY3cjdva1Y4MSsvUTdMOG54eCtSQ3p1?= =?utf-8?B?cXRNTU9pTDRmYy8xL1lhUHk3QzhUT2FiMkhuV2xIOTRTYjJRZHFFZVBQenVk?= =?utf-8?B?VW5JajlpWW8zVWU3MWhNLzRQdHYxUEJsbzZhcWlHTkNTbVNtNWJtM0tkYVFZ?= =?utf-8?B?RlRjVFdjeUdha3MrdkMzNEdVczJNSFZCUVp3dXlhaUh4WTdPcjBNaEhVME96?= =?utf-8?B?TUtIZ3VTUEt6RExNNkUyQnV4VDlDVUI4UFFmbXZNbmkyL1o5Uks5UnVab3V5?= =?utf-8?B?MlpIR3g1MG5CNEVpbGk5WmxPMHVZcm1ISlUzZTFsZEZlODJJVHV6bDlHK2dJ?= =?utf-8?B?UVE5S3NncFlaN25qMkY3enhrR20rejV0b0JlQlIwbWpUVjhBdmtXK2V4ZG96?= =?utf-8?B?UWZBT3dOK3JnPT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5d6b5c13-eb01-4247-8701-08da34c5135e X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 May 2022 09:43:46.7010 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR01MB5176 Subject: Re: [FFmpeg-devel] [PATCH 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: Zhao Zhili: > 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 | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/avs3_parser.c b/libavcodec/avs3_parser.c > index d04d96a03a..0d2e940d1e 100644 > --- a/libavcodec/avs3_parser.c > +++ b/libavcodec/avs3_parser.c > @@ -114,7 +114,7 @@ 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); > + avctx->has_b_frames = FFMAX(avctx->has_b_frames, !get_bits(&gb, 1)); > > 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; FFMAX can evaluate its arguments more than once which is not intended here. - Andreas _______________________________________________ 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".