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 DE850475F2 for ; Thu, 14 Sep 2023 13:34:03 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id A12F168C7AE; Thu, 14 Sep 2023 16:34:01 +0300 (EEST) Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2067.outbound.protection.outlook.com [40.92.90.67]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id E486B68C60E for ; Thu, 14 Sep 2023 16:33:54 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=l9J1A8hcNLOntMrD7epxta67Rag00BL5GRzpQ+oMRl+w8DVVlJQ+bwxqesotYxPYZrTSqmWzag6Vr8dExG/WS6XBYtWAygMlyu9XErijoVGkIdx2ltxnMK5q6xat48H878grc7kOS2jdl2CH7TTymTnjRZJUpMl/1VXAl05e2rY6bali8tSu4+TOY1Jv8SWKVHOObOTmbkVYTCD6TeK7VwpA/noIcObE+JbNqEONQr39kIcgHSF6hgtACCokcTPAhtUVPBKC7kuYfZnZ2JMT389+tQUMw80OEo1qFZz/Pj+Iy8fyB8M47TkKcK2Wb/khbRiKnr+OBgz8xbRiP85uSQ== 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=B7pVsQApmqKfoFOTLmCA2bnVH2datm2/IYO834LXNTw=; b=JcSgfs4xyL5dnMPEYHoOClDcchiCyt0IOFDUmVv3O+638/N8ZlMd2QaIi+RnLpFavl+/TvnTD6leoQIBRM7TtnsQgtLT4CONX31AXCBHOLHuhnV4DgOYes7nuGvXJ/MtHtTU5hzCyWJbbHiqpdxpb2NBR6NUAI5fTtGb87t4rJK+q7cmBZ/vgNH5mijuqxCc0rBqN0Xi5zS6IV8fLK5BqmGxnDujL9kKt6US8cdZnhgvex0KIR2TPbt7UM7Af6NqVzoJ9vm6ZywIdvpX8ND1kmAoJbQVelsBV1uQyZF4AVGO0nCO4yTRR/Ygwoo+MBpSyuZNMLmV8f5a3iMFwghwRw== 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=B7pVsQApmqKfoFOTLmCA2bnVH2datm2/IYO834LXNTw=; b=j9O8n78I/FhNeqHxxzNOpBeov59O6e6kd9q1xSHFBZkXbg6wOPIp21ju8naisa3/JcqyhqGJScyL39SPklaCIeNq6ILsE3XpQcuTGCjuWsxJSJtHJo7F3BFSKEhMBFrE4aRbPv4lmDe2U23H7JkxNsLMqGSISFbAxj3xDb8KOU57TZ0jw4MaNnhOUWbPlo1ErJTY0+9Gb4OlVYAPQNbSOFRI+pW1+vRi78reewAi1Apj33wWSbCaN/BAvQoUCZ0bMhFvRBzqBpwzrJ1k/J+kSS/9A+6IYvqSM20sbVTFcYH0dy1aBhVJNsIvBv1aPMdN3KaDB833Vb3KOB8OssyARQ== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by PR3P250MB0150.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:175::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6792.20; Thu, 14 Sep 2023 13:33:53 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::5e01:aea5:d3a8:cafa]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::5e01:aea5:d3a8:cafa%3]) with mapi id 15.20.6768.029; Thu, 14 Sep 2023 13:33:53 +0000 Message-ID: Date: Thu, 14 Sep 2023 15:35:10 +0200 To: ffmpeg-devel@ffmpeg.org References: Content-Language: en-US From: Andreas Rheinhardt In-Reply-To: X-TMN: [dZYQJPoAFqLmAI6MQUEk2wnigMjCw46s] X-ClientProxiedBy: ZR0P278CA0091.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:23::6) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <34ebba89-fb36-45f8-620d-2ebba3194f1c@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8P250MB0744:EE_|PR3P250MB0150:EE_ X-MS-Office365-Filtering-Correlation-Id: cd2016ca-e970-4075-e2ea-08dbb5273cd2 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: /BZCApQj/vQCl7pHbcVdf5YZnn8Z1tmR5Xap8zNiBiflg0SdLqhzMwsVTamGhgynJluVP/l6EgOuPYAA6ejPGCq8gXLFcQEniI3A3Lxu3pf8za9PItoeaaIc+8bUBwNOgeMy3qh58FkuFSTGgGvea/9UcuOfg1w6amPLOM9fiIDvlFMBGWF0SOob0w2fqrltQi0V8QUKUjwzI5gmFTddvaZOYb9pV1Z3sk4CFlgbPPCnhpbSG62qg865jhJxElzxyogZKUkkdqAkmZDHxj4XQ45h+K01kUsIViz4zUSAf0NdQbSFCm+NeaDQCDe9MOTZghIZ1ofDu4wIkAvND1RvNEy9vklJEJIbBSVPw87ukOlUyQXD20rgLJ8GCNqboDuySADp+y1oP2k7LlDJL3MfW494Idk7+5oqbHeoRt1jDQyIbwWXNHNF2VasoqbMWwrbIkOYs3sPUuO0e7D01C7RFFIBZu8amCa6BUyt7aEI89IgArqV/EoGjornePu9/DiLTPrOMasudhPnIX5zBWWiP3l+sYy+Xs8aelqaPAkAW8vKap8hijjTQ4BBW30VDvlu X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RUw1T2h6Z2ZUdk9xY0c5cC9ubUFld0ZJL2lGM1RBYWp2L3dzU2ZSbWswS1dT?= =?utf-8?B?M3RyNnZVcm80dU1iTG52QVpmdllhbUR6c3BVRTVRdDNjeWJJT1hrbTdZNi9F?= =?utf-8?B?TWtEejF4aFVBTFZTQjU5djIxUkJnM3ZBVGFFVE44YWNqZkpqUklJOU9HdXpZ?= =?utf-8?B?a2pHSThZWlQ3L0NHUUIwT0xDajhqUlhqaTdURTNWZjBWOHl6MzFvT0JwOTd3?= =?utf-8?B?RkxuMFYwMlVBSEh4U3d3MDE0aU95SUxQVmVHQjQvTmhLWUhpK21NcVhpaU9t?= =?utf-8?B?RWlXRjUyektyQzNKWjFtQythTThqak5Gdmk5bU5TTnl5aEc1UFJRQmxPd2Qw?= =?utf-8?B?R1g0M3lQdXdMd241MWJ0UWVYcnlHVUxDNkNmcUptTlRsS0sxanFmVkpuVEpZ?= =?utf-8?B?clVHNTQyazVXem9jRms1TFFoQ1BpZmVrMjhLTGlHUFlFbkVrdFZobVpLN1Fo?= =?utf-8?B?cCtWTENYa3JPMzkxSDZIejBvMnhyMUR2QldLTSs3RWtySG9RTjNEN3lIYnRY?= =?utf-8?B?TGQ2K0J1THN5bVV1SmJqNmRVb3RYNi82bldDd3JoV3JZekxvNmVocWVZSDZC?= =?utf-8?B?V0h5YjFVY2dLT2tlc2dmenlTZVFzMlNncncvS1p2eHJDVlhvNTFiaFE5Y0Jj?= =?utf-8?B?SXVZTkJNellTYmRUZTAzL1FFQ1M5dnBlVGwyUStoaGZKVWlnQXhXMkhhV0pM?= =?utf-8?B?S0pHVmNsRE5hQVZqNGtYV2lxVHBzYTRwSVJvSkZkYWxhN214Ui9FUU00RXcx?= =?utf-8?B?MU9tRUpCbHA3LzlKUU9TOHFVRGllQmgwbThVampNKzZmWUtsd21QcCs2ZVg0?= =?utf-8?B?eWpOaWRQbVRHT2ovbjMwUmpGZVNKOGJ4aWZVWmNDa2JIVHNKaHZJenZPTTVl?= =?utf-8?B?NVZTMng0UDJDZGVINTd1Uks4RGtocCtmK3JOU2FOVnY2d002a0lwSUtJOER4?= =?utf-8?B?dWppTnVJdlpSRVN6ZFBtdmg5ZVBFdGNLa2YwU29nTkY3UUtSOFBaRHpjNnB4?= =?utf-8?B?V0QvU3U5TTlWZDFXNTVPS1pWKzZDNUZEeHRRdUpRbjBUbDgxMHpKalZUeU5I?= =?utf-8?B?c00zNnAreVlzSm1heUxZc05TQllSTGRFS1JZMmhUMnRiZjRKbm9wWmxuQVcw?= =?utf-8?B?bUpLMGcvcmo5NENnekRzSWl5RzdVcGpxLzlUcXRIQkl0OE5UNE1xL0FZWFVs?= =?utf-8?B?M2x3SEtzZ0dBUWVOdmhLWW9lT1lyQnUzaHVRbVhXdm5ZK0cvYmQzNnlmQkpR?= =?utf-8?B?RzU4eDlKbkVBYjMxUDJvMTlxL3pabUMrYVk5S0luS1duQXZaSkluMFpZUS9q?= =?utf-8?B?YUNaTkh3OGlONHBuY1JiUnVpazlqYWFMdGd4NElqaS9WejYvbjIzRHVEbUFS?= =?utf-8?B?eUhDWXRsSUtSM1o1TkZxNGw2Um1hSGd2Q3E5STVWelFINEl5RFBmeXdyWnB6?= =?utf-8?B?bVAyYStYNkgrUHhVRWlvZWcvczMvbzc1Rjd2WE1BaS8xQ0E0eVYyRHZ6aG9l?= =?utf-8?B?Q0JTM0dDTjdmSFVmK25wUTJiK2hkZnMySlpXVWcvczlqbjFYUEVPcG5MNGxk?= =?utf-8?B?WmVYSzhodEJVbTQxaHNXV1UwanJoazhUQXdtcFdMQll0VWRtS1Jab2NaeUQv?= =?utf-8?B?MXRnRkcyWDVveEQ3SmY2UEh4T1dhK21Talg3Vk42eEM5Mm5xenhNVkhyVFdr?= =?utf-8?Q?lXBiJ7CYh7AGOuP8Ms9R?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: cd2016ca-e970-4075-e2ea-08dbb5273cd2 X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Sep 2023 13:33:53.4553 (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: PR3P250MB0150 Subject: Re: [FFmpeg-devel] [PATCH] avcodec/lagarith: use VLC for probe code length 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: Paul B Mahol: > + unsigned val, bits; > + > + bits = get_vlc2(gb, lag_tab.table, VLC_BITS, 3); > + if (bits > 31) { > *value = 0; > return AVERROR_INVALIDDATA; Using a signed int and checking for zero is simpler. And the depth of this VLC is 1, not 3. - 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".