From patchwork Sat Sep 3 20:30:11 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37636 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:139a:b0:8f:1db5:eae2 with SMTP id w26csp1746363pzh; Sat, 3 Sep 2022 13:30:35 -0700 (PDT) X-Google-Smtp-Source: AA6agR5XTSfeSmTEZBZawQToyHXSdskh6LFVPohnNuMmlelhk9foPi78qf0S13abSTypIGvikOAx X-Received: by 2002:a17:907:1ca9:b0:741:4f9a:5984 with SMTP id nb41-20020a1709071ca900b007414f9a5984mr23810943ejc.86.1662237035421; Sat, 03 Sep 2022 13:30:35 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id qb23-20020a1709077e9700b0073d8a8fedbfsi3198169ejc.386.2022.09.03.13.30.34; Sat, 03 Sep 2022 13:30:35 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b=er+3CIxF; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 8373768B559; Sat, 3 Sep 2022 23:30:31 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-oln040092073045.outbound.protection.outlook.com [40.92.73.45]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 081DC68B7DF for ; Sat, 3 Sep 2022 23:30:23 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JRETZj4RmUFjs7ghlJFF9T64CN9c4mvxmREZxLXO24X9ZuwV4US41Mf9nJYghAjleghKVbUm73dp5cao8bFFwV2pv6pSkXYxypXEIh9sKvm3AwJ/wqDt99pgjJbVkABYTI3CwDCePwoBxJQQrJqXZl69e7bkN4CfCvGXwypHhSiczu54RnbTY+AQCaHiARWvR+eyWbyMVVce3/WSJM+wln1Dru7dZ3iUr4nV6fZCbr54An3WPKbtXPa8sr3HNlF5bHCBDB5Z4nZFCn1+UPCEN955Pt/TOoGA1W8HSYVuJDgmTWbpup49G45lOMpCdT+AAvYCezU7t+F58t7LUQ/bCw== 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=xDzNI4PHfhM80FCqmKbd79QJqt51fDtqiCd8iKvLdwo=; b=eiQjJicWo7Uak3b6DTJNPdoJfyNPBXQBclA2bR8zt9cW+bp60FNoFXhcyc6UBhh3HrhqutZYatsNA97o0T78FlnfpIj29AUMNJjKIJl2lldCyZ6bWcdY0/PmA3gSvnM92njxhsEDGt4tXUyQy5AOCKrFYIiqmd+lQscXrdKOpfHxaDzyspmREcP3OmKbJcnZurDB12ttmEpg6GNmfOAInSvu0+jpwIVr50rbvJe7CQcu2V4VAPmp3qHiytiYzYk3wUlF5azKXA4RaQoOcC4Fhff/0R+1bySDlk0eD8zG8vILwPN6a7JYREf14aaYbKebnl+625q9WG9hBANTDGdBSw== 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=xDzNI4PHfhM80FCqmKbd79QJqt51fDtqiCd8iKvLdwo=; b=er+3CIxFi16+ZO5TQCVt8HmLNWyLh26DyPe0I5VqD3rRoSqlPc91o9vmYoNpg3W79hCw3u77YBXEhlh/llUeVk8Oe8KjlWZo4e51tRm/ANxgURA+98dnfJ6vr5zN9HgJjDZYqVQjQw5HOkpaASCSRBQjL3amE1b7XaKCuOFFva3ejjZ+IR0+t5qwY8F453fke5UhwZvHSndkejYIbS/fUtreZV6GdM9mQP8n3zeZIIJLiEbO7VOT8RxTr6jf2vn8m9v4KFTe7pdAL+PQ5LZaoC8seq1ZqMTH9MWUqCxd7aNO3e3cRcc7LXZHVfU1YL17Q3A9RgZdvOuGwqCuS3jzFQ== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by DB4P250MB0928.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:383::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5588.14; Sat, 3 Sep 2022 20:30:21 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47%2]) with mapi id 15.20.5588.010; Sat, 3 Sep 2022 20:30:21 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Sat, 3 Sep 2022 22:30:11 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 X-TMN: [BNhz3juziUyEpp1OUJzAPxg6lXADWYHLa6kBVTDc2o0=] X-ClientProxiedBy: ZR0P278CA0042.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:1d::11) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <20220903203015.1960864-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: c9140b79-3184-4f49-b8d8-08da8deb1f17 X-MS-Exchange-SLBlob-MailProps: feAVlmA1hHVJajBbP41IGVvcEX4IAvp8OjIx2V4JeIlZoCifK7fLEoShJosie0G8bY/O9ucnM4RdgIDt/4SW7tHC58D8rn1/RJMO1gHxnHkEVJTTs9dN+EZ4JaNvTSYd2jFy+8Wltw07rTkOEW7+z4Pxj2QBgtTUMPOqvelng53Gy6fsPcEZmJnzSD9r38+1vjKSnuNPnpb8gAOK1e/K7YBnQzpoRRP38J+/YxlX+lR5cvtCDLrEs2fhR6B2PtKOEJMJXuEYvANHJml9rPEl4TwVkNNGosbW+Qul13zgFIJZwhAEGg+eGz3AXTbajkwMqrg9TVNbpRnJsNu2HF5EuUKAkB5ofg9q8D05G1P6Y5PEjPYKm6WEiMu/FyjalxHGB00dsxpJ1zz6KfMb5P6V3vu1Uk+tw1QHEMOXuXBsFm3Sc2yNjneaPvVsbduv8BgLYrlinvd0UvFUkNW+F7KZDoCKKN99qhx834Ja/ed2+sTpX7xxLKqolw05nIji3ULT+rErLmxwmj4N0RTgbAFSyqLBps4Nvp7Vj6XJcNHw2gWE5oeYZhKIVfd+6HdCoOBsLt8rwf/NMhQmJCuf9yUty2X08ZQt4W0O2Wc/flH2/Z8a/Max+jAblKHaouTJq9VSJ9UZekPcXrg3fQ1UUXEQ/2roOLJLN8dkDKh/dIA9T7jZ1YJThj9WJw0c1finDDEzV2kUFF8yUtMSSUW/qph4tg== X-MS-TrafficTypeDiagnostic: DB4P250MB0928:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 2HpiSYwNocno/8GMzukp87vRkbpUyDNIoFzvl8cAAG21iCRUbXsMCvLiUImahtaL19wd3jxqfsOX3aSK2gkdebzgama2YuUlnbJcT1n+dQzcfewkABHmz6b20xqU165E9PJp9U7voj7ShcppxEtYw4FJnuzXYoAlzoeq3hnX6k9600s/S6OSSYBMexPsGKdNuXtt4qYFkUPS4RLMOiftnRHksnlRFXsiaR5t5x7SX1CJoU+Bo54krdZci38J5HRvdjUsx5oUnrSuiC58IQBG+ksRbBIeRZ99dmW0h4lnvr+wz03v52KS9v+2oXpjTNfykA5ZnscIKi7XLyD7xOq10ejo4QDtqEXYPSWNEtrgmEnnkTnegoDmUkoTNDbJ1mvD/kcnd9ADYstS4aXVnmINSDvU0eV06uCN17PunIYuvB61MWJY0MgUp8EwPFP9DxQQ1/JpQ2nzlHIQS04mcRG5nJ4/AXoZjBLES8I34PvILX7hcPXAobJdUeJzXarT1duPAox6PG9eHqDygvlaMXiSzcM1HqPOUrHMVoC5XDTP8Q8KUJPG/1D5ZzKHLRHMGNSMy28INViYWcA6awtn7WAu2h5EqWCpXG3Aih1StL2mlUEPD77dALI7ST6ln1OaqEjW1mwPc1XMpFG/2f0u/+kKpA== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: MxPy2oPXj78Q9lUCkpj26sJzusVYyjJWpU5U0xLw9Xr/SGjNZO86LK/BOcz/6obO/t3ElMCj+ockqHxw49gIU1ltp0rRaYSKvEeqI+dpp10geErLpLg77YNZuBDWy+4HfAwbUMrV8I0xFNGHiYfi3FJtqSEZjMXGC1WCgW5acGL64+JLlhaLzXfoXGLnZo3xBJLbc3ynd2lnKh8lPEmS/89fhRTc0wwuXVEmzd0YdhEhWzje7WupkbTI0YpSA/xwHTgaUT0HsqBGij3WjKQK2XU5LJdc3fm1uITnz3nR8kKiJAS1SFMPomllO62VJb+09XbJI8IXjZlvehr/n3V0FEuW8vkYqeb5csHzWwg9q2hOfQCSRJACIuxOKTiDDSabQoUQywvGsSg9YpoCS3uhaerw1FUWCZtAhNHFzDDrt4vCYCN4Aqt8biuGFlme0mcj9YqX/VfiGL1LrRUTv5xFlxOZzHI0oLfg0nhkGhaZ0gKFtfFXX3xHIVF7znx6J06NpV9pZwX4J6Zn71jT1zhLWHKCtBQVUcYr0u0D1InJ5Zmqqz+7oPgekPtud7vzWqIeWwLFAu2TXeXLiPYKHgt9PNr1/yfJ0qH/mujdGB3nQUXxNsF5G0mZYDyds3hxeKKJNUrRuHWqWz6nAlxKvbNcD5H6z3IWmnDQd+lnl1m7mF5wZgEZDTuq2ZvTuu3FjsiAdLdex4+Rwz8Bef/YJxAAfeEg8kV1GNxMCsIkWGNROw82PFthaVO9q/38us3l4AkvkejbUSn0cvuOiaTPJNYO56+c6v5Tb8fbZMDGIHeHP4fEKf2s+GXHQs8MQ5utdYXrUyJeYKDJPcQvw0wa8/v8Tsm/R6SCrPj82bYtTMdhO8NRgABoicrB2VqW7WMTmLWkrZwQLkYRzc5VnqdfOrWLJCgp/X1y1JCm70PTjXjZis1FEaXqSxnMJVzALFop4agTAM9tCaO6OJ3/XIZc8P2KMDz+zGqFFdmeLfC+Yb1O25EBTk2iSW2868TMZgDu1k6g6UfAkzTduUphFs2Mk7/uG6jCRkAUIF5YzLR+RHRczja7r3HfGlTBfN2XBVWZfsHTwwNL0y0r1wBCOjYiRpL9BfvM2SYSltBcY44P6nJWUvL5K+bC+WzS2ILXZ5Czcn97KUPMvKoWHuaMqtv/+0lh1r/1XAjKazDjyHzcyZiy/JaVFc7ZVjLrTspm2V2/IsGkT2knGoAP0O/fT93+tx+Wxs8gBxh2w+usKrCKJ0yFZdmpgaGM//7chYLCygM1i1l0hVvupGP+1S+5MqAC9sHpkYanOAmT68eyjhalvoaW6KydBmMwxunEzz7tyBN7OhfA X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: c9140b79-3184-4f49-b8d8-08da8deb1f17 X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2022 20:30:21.2480 (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: DB4P250MB0928 Subject: [FFmpeg-devel] [PATCH 1/5] avcodec/cfhd, cfhddata: Simplify check for escape 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 Cc: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: FsUUqT7n+oxH cfhd.c checked for level being equal to a certain codebook- dependent constant and to run being two. The first check is actually redundant, as all codebooks contain only one (real) entry with run == 2 (as is usual with VLCs, this one real entry has several corresponding entries in the table). But given that no entry has a run of zero (except incomplete entries which just signal that one needs to do another round of parsing), one can actually use that as sentinel. This patch does so. Signed-off-by: Andreas Rheinhardt --- One could now deduplicate reading the RL VLCs; or one could branch based upon whether (codebook == 0) || (codebook == 1) or not as this is the actual check in dequant_and_decompand(). libavcodec/cfhd.c | 4 ++-- libavcodec/cfhddata.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index 7afbbac59e..90b3d0a850 100644 --- a/libavcodec/cfhd.c +++ b/libavcodec/cfhd.c @@ -819,7 +819,7 @@ static int cfhd_decode(AVCodecContext *avctx, AVFrame *pic, VLC_BITS, 3, 1); /* escape */ - if (level == 64 && run == 2) + if (!run) break; count += run; @@ -850,7 +850,7 @@ static int cfhd_decode(AVCodecContext *avctx, AVFrame *pic, VLC_BITS, 3, 1); /* escape */ - if (level == 255 && run == 2) + if (!run) break; count += run; diff --git a/libavcodec/cfhddata.c b/libavcodec/cfhddata.c index 67bd8e66db..212dccadb9 100644 --- a/libavcodec/cfhddata.c +++ b/libavcodec/cfhddata.c @@ -72,7 +72,7 @@ static const uint16_t table_9_vlc_run[NB_VLC_TABLE_9] = { 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, - 1, 2, + 1, 0, }; static const uint8_t table_9_vlc_level[NB_VLC_TABLE_9] = { @@ -226,7 +226,7 @@ static const uint16_t table_18_vlc_run[NB_VLC_TABLE_18] = { 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 2, + 1, 1, 1, 1, 1, 1, 1, 0, }; static const uint8_t table_18_vlc_level[NB_VLC_TABLE_18] = { From patchwork Sat Sep 3 20:35:56 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37637 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:139a:b0:8f:1db5:eae2 with SMTP id w26csp1748035pzh; Sat, 3 Sep 2022 13:36:17 -0700 (PDT) X-Google-Smtp-Source: AA6agR7CONpGrWlzfyvpEbFDFe3fiHE7EVxCETUNu2c+UKD5Vu+tJBd2cMC6B9mvKND6WGd19eDy X-Received: by 2002:a17:907:b0c:b0:73d:dc7f:4b8 with SMTP id h12-20020a1709070b0c00b0073ddc7f04b8mr30288275ejl.183.1662237376962; Sat, 03 Sep 2022 13:36:16 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id w22-20020a170906385600b0073d72dbfdf8si3779020ejc.220.2022.09.03.13.36.16; Sat, 03 Sep 2022 13:36:16 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b="CtiIa/dZ"; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 21B8C68BA3B; Sat, 3 Sep 2022 23:36:13 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-oln040092065072.outbound.protection.outlook.com [40.92.65.72]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id E282B68B5F7 for ; Sat, 3 Sep 2022 23:36:05 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ihZPLE6SfaDU/VAh5IwpwNhxXgmxATlCXXlsDL/2l/zkxLeR5/ngIT+73zuy3zSPM+w9vR6GqWuTgecAcUX70foVK7qxEECUJ6Obvw333u4cScSIRWGvGA4UEBUSyH+n5E80u4+cR+LHt53QUNkuwGE/SAfa7QzKRGQANoEEFn9e2UfAwHF7PGdq+BtkfSpD1vHD/xWbwnXbWbFwJImD56xrPT1/A0hXhPlMMvCVHmYMiVsH8lhDp9zW8flwD0bHswM0LrdRLprq0W+DT9dxSfLfKm/tcOTyR69AjEWRxftQVqcmehXdmaZYWJ+6hExpfByfxfIMQShGl6AufkWdFA== 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=as+io2efNsvdzxNOaBGgOfFi8tA91hvYXg6ZIAHExEQ=; b=nSTIDtXaiRuT8sfUY23QAVWk7pXQX6HIb2vJcgA2XMm6CvN3yu7T0q35A6ULIYwKz2W5Ye36DBbWcUBixxgdRmh/2ToazqoTELtlxt2EaFzOeV+VpZCfQ8dLT+d4yuOme99W/3s9G73OmOZElbSZpT6uOwkqjrKQ+0fEs2nFq07MqA8fo9MRFcYqM6kvuEnNqRIkCnpin8/Xmgrb8ECK3KykaZ9T5dV+YWBajAgwFSjzeHVdIJENraYC0OXDceKSdGgc9cvb1tPUrSO7RWRVPgflnArk8ls0vIzn6cLoJyQn/0R9EKHSfwg0dBKLhPm/TpTrYexeAvfD7EC7Wqx4Fg== 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=as+io2efNsvdzxNOaBGgOfFi8tA91hvYXg6ZIAHExEQ=; b=CtiIa/dZxz+niRYr+JuxVuwQswaR1BkBOY2ZEAjXOhTtgg8wGaVDWRJa8onEtldOFQ/wWnQoK7sVCmyxL1K0zbDomgd1nqPq/MYHNmkrhLXUp5FwSe+G506W+lO9mVMMh4Px/D63dnY5g2um1OZTc94vwrJ0DibAaZE4XV/oMpEyv8rZpavU9IkFyW/ESemhC0sxHxpkbPQ1BJRTupYHTmwzuUPxC2g4QoVjW6ld7oQ82S1qndnlgMDesk/MkuBcDg4qu6FLpz22gZf/IJVRGjs9tlkZxCd/KGv4ciMDLii7ebg8NjdzLdtXDV98gItL9xvdysWq09RWmgrGfguT2Q== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by PR3P250MB0241.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:17c::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5588.12; Sat, 3 Sep 2022 20:36:03 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47%2]) with mapi id 15.20.5588.010; Sat, 3 Sep 2022 20:36:03 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Sat, 3 Sep 2022 22:35:56 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: X-TMN: [VSjcDl+F9DgAC5bcuD8EPUddkTpCqojCSrhMvGDZCEw=] X-ClientProxiedBy: AM6P193CA0086.EURP193.PROD.OUTLOOK.COM (2603:10a6:209:88::27) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <20220903203559.1961353-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 5d076d4b-0763-404e-ff05-08da8debeb7d X-MS-Exchange-SLBlob-MailProps: feAVlmA1hHVJajBbP41IGVvcEX4IAvp85sPUG3IRiSiBCm2g11fXaVZL2fMGB5a8jgGEw6jNj6KyccmB9HZMiilMveiCy5t3TCWEZSjhHLRWPNPYdRUno4n+Iz9khMpyPcp1Vi/O5xinSV4YG2wtSnIae5X666dhS6yUktXOfsRds5DaAQzohe2f59RPSfH2uel+NfYjaYddOCqO0KdOBMGp22nQ9Jq2aoIOa0G8TLrVM1gQbOYUkEUS/OELlaJPRx4qNPVKzg6Z8qPl1DkDkV8hd98Z0MjZpJCbh5jjWS4oAlSfDcq9HCf44UWX5PowqFyPUxUVlbIKCvlntEu2M8VvtKkBeSkf32bCLr5e7pg3Bc8KFXj2v7R7qqILtCYn/7gTZWoP+enVp7PWR9GTTqjxl63c+SZBDtEdCxuZxXO5mY6WpNxzgtWR8dWKdTuHTht5/kvBGEvSJqDC0TIaQtvTzivdsZ8tthNsvcs7mtXCYgkYP991oCM9r5ymfKn4+r9MEC4vt0s9rqnCjF0AbK0tCd7sOk9cElFCG5AFw+w8yVnCYOLceOfBVy7afHisNJed8qUGsRz6gVPZDFOqqa/3B8XnjaItg+SzyyVrP3Y9n+EvdFC7B2zcb24HqGUKR8ExaFkI+Y53ieidLZRLHiZsA+UfuaE4YgZpyLeWAvA2u0tLcI7sSx35BeC/Jeykz/Owdu6Q4367LOqxZWZVVg== X-MS-TrafficTypeDiagnostic: PR3P250MB0241:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: P7DAbj9PW4xAGYDV13o5Jaw3K/cGsPMR4sufkzTboYMaF//pT4uzT2ApDj+PAv8yvb3r5YvZm2V8F8MW+NmGTpjSWePxVzA/xoOh9R94pZzh4lb08RH+XwTV8ODXJSzbnwBeaceqIZL6NwZzjwFrlIMwnSJbGB8XiSJybdT5E9rXPG1M/5Sctlwm9tjAIAYNSRR1UYeaAZ6GrJhWlOP1vgPicQ79ModWysgL5XjjoAMjebm4NBL1ynxdpIbCM+qjn71gt6HCh0h5Log+nFBAb4M/zxSFqVfYIjV4QabXyWm0lYj0BbSziRF2HvGK3vf3PpBZLx5SfaxNXJQ5zrRzQtALMJIhRR1p+3jzrjaibD1nKTHF+9O8rur1Yos3Wkn5BCYUJ1Y2V4UthW3x3TbJ1314IqhLf4vakVH71Wzlf+eYp5mJqZ15hrUqe61hkd47PA124aXDbkkM9Polj7EyQUlJ5SlkL8WXO+BPUU7AFrEoh2YrjPbK7JJ7W+gC43sBiGOnXS3QgrXh5MCshuW6i6A+f78BzgSnj4OjRCLybijLIAOjVg6jW6qnNc3nzFjbhA4J7uh0ZoupTToearhujf46fOyilkpKTF9WrkMO9bTzAL2M1mviMSAK3j+7k/nLAThXJU7WN2Tf+GkXNGbb5A== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: Aa8NGsvigUSnVQbBQcdNpcGrbtcnt30up9T5ZSMiRkNLeKwAeMmI9shjbdnC0/vEJeoh8Nq6t5IGC7IPw9X5gN2t+UbYbDgV61Eu3HR1CO9ddRrboxRju5MCDh1sWARPwEoVZGinjCO/FNmuj0wKhnhhU2dU4KMy8GOpo4k15HJJ5Iflm/Kz015VWyVoVfecB1xTG5lFZmrtOF9tsBQT5KjoBCrYhRq9Uhj6D3OI/dhZYdibWi8BFkx7KXg9R7yi7wQplbR8l3k0TTaREAJPUKvqwK/PUtm2SiMObRIHweVSBX+siGeGeOhYp4k8IWawpTNWMNSqOtZJiIE65CYIz3p+w4yNTeP2RiQSzlaivn30DWVT4D4O12nY64TkXvl28MTKcHhupk9BWnZRZsvO6bPisRIE2HABnRHtUH55bm6MAAiIRUYqk31zukeOgPBd3Bm7pjYTscEMvB6Smx7goz+Kv15hw76MdqTugh/88e1yc0bsjUxHUJWJSyORqk2gJlUBNTD3YGWNobtXvYGMRPZjBrZSzNKxVAATylYrtlpnDU8GFIhUgPQZp16AYRbVFnHTrv7x/4aUkYZGblWN40E3QPZPeWpIwl6VQ8u6vuw+z6wQnG37JkOr+DpF1ms7empp74yPfZPJyYZR/vqbqQkMmwkeght+ELVpmj1pLjknZ5QE51U4HtMx6AhPjdhcPrNijLwz9VNJPo8PU2v9L8Euv/8zCBp1l2k16mfd9LjEavOs9Ibd9ZV0VeZmFAGfB9mxVlpaFKtn5jZVxUBbf5sYrFvs4H2Kkd3QeOOVEikI5jrr08V85MzuEUTUoNVDOiZtO45/uObLdpSs7Ixv8hURac4VfrZoCScl+5wbV/aknWScelpXCK5N9/9CKNFcKs6NlDQyvJzA/TtiIfdR3jTgAvFWMCCSZnJ0hNXHddDGJnEfYQI0K6yHDZncCf8CfXeDnta33HugaSJdfu8wVqP6pQ8mod0ZJj4ZUPYF9sOvUXI9oF7sz0H+BcIbNMO4J02U3bMBLrj9/JQmCMly8Qr9NaR/7NrBnSfluuLqr3XfzcU+cjWzt2JgcKK9ssf5sl2gdJElKWRUnSCpHo8mLfxpO1PYY64ThwbgRWqnJTLS/UrVhVpo7nQ1C5lSNexhRTlPeB+EbqK7WMn01HxyZgzGvPwFgo/UjRcKxzBrdIXMabGN8q1fc5d7VSIcqIDmY0MDvhi9W+FX3q+SvE1iHDd5TI0KKDKazLikFkycLLWdd2uoHGuBrjs7zZUebIMQ/v1vIwSB092fPRf3dT4U1VQki/7bswnuDnUi3ZspnJuIJWOvkolIAlQkjMp3ohXN X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5d076d4b-0763-404e-ff05-08da8debeb7d X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2022 20:36:03.6111 (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: PR3P250MB0241 Subject: [FFmpeg-devel] [PATCH 2/5] avcodec/cfhddata: Avoid code tables 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 Cc: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: LJUMBbB/gg28 cfhddata.c initializes a RL VLC table via code tables and corresponding tables for length, run and level. code and length tables are used to initialize a VLC, no symbol table is used. Afterwards the symbols of said VLC are just the indices of the corresponding entries in the code and length table that were used for initialization; they can therefore be used to get the matching level and run entry and they are not used for anything else. Therefore one can just permute these tables without changing the resulting RL VLC tables. This commit does just this. It permutes these tables so that the code tables are ordered from left to right in the resulting tree and then switches to ff_init_vlc_from_lengths(), which allows to remove the codes table altogether. Given that these tables are constructed on the stack, this also reduces stack usage, potentially fixing part of #9399. (The size of the tables on the stack decreases from 4752 to 2640.) Signed-off-by: Andreas Rheinhardt --- libavcodec/cfhddata.c | 372 +++++++++++++----------------------------- 1 file changed, 111 insertions(+), 261 deletions(-) diff --git a/libavcodec/cfhddata.c b/libavcodec/cfhddata.c index 212dccadb9..7c4b1454f3 100644 --- a/libavcodec/cfhddata.c +++ b/libavcodec/cfhddata.c @@ -27,248 +27,106 @@ #define NB_VLC_TABLE_9 (71 + 3) #define NB_VLC_TABLE_18 (263 + 1) -static const uint32_t table_9_vlc_bits[NB_VLC_TABLE_9] = { - 0, 0x2, 0xc, 0x1a, - 0x1d, 0x1e, 0x39, 0x3e, - 0x37, 0x7e, 0x6c, 0xe2, - 0xfe, 0xdb, 0xe0, 0x1c3, - 0x1c6, 0x1ff, 0x1fe, 0x1b5, - 0x369, 0x385, 0x71d, 0x6d0, - 0x708, 0x71f, 0xe3d, 0xe39, - 0xe13, 0xe12, 0x1c71, 0x1b45, - 0x1b47, 0x3689, 0x38f2, 0x38e1, - 0x38e0, 0x38f1, 0x3688, 0x6d1b, - 0x71e0, 0x6d19, 0x71e7, 0xe3cd, - 0xda35, 0xda30, 0xe3c3, 0x1b469, - 0x1b462, 0x1c798, 0x1b463, 0x1c799, - 0x38f08, 0x38f09, 0x38f0a, 0x6d1a0, - 0x6d1a3, 0x6d1a1, 0xda345, 0xda344, - 0xe3c2d, 0xe3c2f, 0xe3c2e, 0x38f0b2, - 0x71e160, 0x71e162, 0x71e166, 0x71e161, - 0xe3c2ce, 0xe3c2c6, 0xe3c2c7, 0x1C7859E, - 0x38F0B3F, 0x38F0B3E, +typedef struct CFHD_RL_ELEM { + uint16_t run; + uint8_t level; + uint8_t len; +} CFHD_RL_ELEM; + +static const CFHD_RL_ELEM table_9_vlc[NB_VLC_TABLE_9] = { + { 1, 0, 1 }, { 1, 1, 2 }, { 1, 2, 4 }, { 1, 3, 5 }, + { 1, 6, 7 }, { 1, 15, 11 }, { 1, 26, 14 }, { 1, 27, 14 }, + { 1, 23, 13 }, { 1, 36, 16 }, { 1, 40, 17 }, { 1, 41, 17 }, + { 1, 32, 15 }, { 1, 48, 19 }, { 1, 49, 19 }, { 1, 51, 20 }, + { 1, 52, 20 }, { 1, 50, 19 }, { 1, 42, 17 }, { 1, 37, 16 }, + { 1, 33, 15 }, { 1, 24, 13 }, { 1, 13, 10 }, { 1, 10, 9 }, + { 1, 8, 8 }, { 1, 5, 6 }, { 80, 0, 8 }, { 1, 16, 11 }, + { 1, 19, 12 }, { 1, 20, 12 }, { 1, 14, 10 }, { 120, 0, 9 }, + { 320, 0, 8 }, { 1, 11, 9 }, { 1, 28, 14 }, { 1, 29, 14 }, + { 1, 25, 13 }, { 1, 21, 12 }, { 1, 17, 11 }, { 1, 34, 15 }, + { 1, 45, 18 }, { 1, 46, 18 }, { 1, 47, 18 }, { 1, 57, 23 }, + { 1, 58, 23 }, { 1, 59, 23 }, { 1, 62, 24 }, { 1, 63, 24 }, + { 1, 56, 22 }, { 1, 60, 23 }, { 1, 61, 24 }, { 1, 64, 25 }, + { 0, 64, 26 }, { 1, 64, 26 }, { 1, 53, 20 }, { 1, 54, 20 }, + { 1, 55, 20 }, { 1, 38, 16 }, { 1, 30, 14 }, { 1, 31, 14 }, + { 1, 43, 17 }, { 1, 44, 17 }, { 1, 39, 16 }, { 1, 35, 15 }, + { 1, 22, 12 }, { 1, 18, 11 }, { 32, 0, 6 }, { 12, 0, 5 }, + { 1, 4, 5 }, { 160, 0, 6 }, { 1, 7, 7 }, { 1, 9, 8 }, + { 100, 0, 9 }, { 1, 12, 9 }, }; -static const uint8_t table_9_vlc_len[NB_VLC_TABLE_9] = { - 1, 2, 4, 5, 5, 5, 6, 6, - 6, 7, 7, 8, 8, 8, 8, 9, - 9, 9, 9, 9, 10, 10, 11, 11, - 11, 11, 12, 12, 12, 12, 13, 13, - 13, 14, 14, 14, 14, 14, 14, 15, - 15, 15, 15, 16, 16, 16, 16, 17, - 17, 17, 17, 17, 18, 18, 18, 19, - 19, 19, 20, 20, 20, 20, 20, 22, - 23, 23, 23, 23, 24, 24, 24, 25, - 26, 26, -}; - -static const uint16_t table_9_vlc_run[NB_VLC_TABLE_9] = { - 1, 1, 1, 1, 12, 1, 32, 160, - 1, 1, 1, 320, 1, 1, 80, 120, - 1, 1, 100, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 0, -}; - -static const uint8_t table_9_vlc_level[NB_VLC_TABLE_9] = { - 0, 1, 2, 3, 0, 4, 0, 0, - 5, 7, 6, 0, 9, 8, 0, 0, - 11, 12, 0, 10, 13, 14, 17, 15, - 16, 18, 22, 21, 20, 19, 25, 23, - 24, 27, 31, 29, 28, 30, 26, 33, - 34, 32, 35, 39, 37, 36, 38, 42, - 40, 43, 41, 44, 45, 46, 47, 48, - 50, 49, 52, 51, 53, 55, 54, 56, - 57, 59, 60, 58, 61, 62, 63, 64, - 64, 64, -}; - -static const uint32_t table_18_vlc_bits[NB_VLC_TABLE_18] = { - 0, 0x2, 0x7, 0x19, - 0x30, 0x36, 0x6f, 0x63, - 0x69, 0x6b, 0xd1, 0xd4, - 0xdc, 0x189, 0x18a, 0x1a0, - 0x1ab, 0x377, 0x310, 0x316, - 0x343, 0x354, 0x375, 0x623, - 0x684, 0x685, 0x6ab, 0x6ec, - 0xddb, 0xc5c, 0xc5e, 0xc44, - 0xd55, 0xdd1, 0xdd3, 0x1bb5, - 0x188b, 0x18bb, 0x18bf, 0x1aa8, - 0x1ba0, 0x1ba5, 0x1ba4, 0x3115, - 0x3175, 0x317d, 0x3553, 0x3768, - 0x6e87, 0x6ed3, 0x62e8, 0x62f8, - 0x6228, 0x6aa4, 0x6e85, 0xc453, - 0xc5d3, 0xc5f3, 0xdda4, 0xdd08, - 0xdd0c, 0x1bb4b, 0x1bb4a, 0x18ba5, - 0x18be5, 0x1aa95, 0x1aa97, 0x188a4, - 0x1ba13, 0x31748, 0x317c8, 0x35528, - 0x3552c, 0x37424, 0x37434, 0x37436, - 0x62294, 0x62e92, 0x62f92, 0x6aa52, - 0x6aa5a, 0x6e86a, 0x6e86e, 0x6e84a, - 0xc452a, 0xc5d27, 0xc5f26, 0xd54a6, - 0xd54b6, 0xdd096, 0xdd0d6, 0xdd0de, - 0x188a56, 0x18ba4d, 0x18be4e, 0x18be4f, - 0x1aa96e, 0x1ba12e, 0x1ba12f, 0x1ba1af, - 0x1ba1bf, 0x37435d, 0x37437d, 0x317498, - 0x35529c, 0x35529d, 0x3552de, 0x3552df, - 0x62e933, 0x62295d, 0x6aa53d, 0x6aa53f, - 0x6aa53e, 0x6e86b9, 0x6e86f8, 0xd54a79, - 0xc5d265, 0xc452b8, 0xdd0d71, 0xd54a78, - 0xdd0d70, 0xdd0df2, 0xdd0df3, 0x188a5f6, - 0x188a5f5, 0x188a5f4, 0x188a5f3, 0x188a5f2, - 0x188a5f1, 0x188a5f0, 0x188a5ef, 0x188a5ee, - 0x188a5ed, 0x188a5aa, 0x188a5e3, 0x188a5df, - 0x188a589, 0x188a5dd, 0x188a578, 0x188a5e0, - 0x188a588, 0x188a5d6, 0x188a5db, 0x188a5e1, - 0x188a587, 0x188a59a, 0x188a5c4, 0x188a5ec, - 0x188a586, 0x188a573, 0x188a59c, 0x188a5c8, - 0x188a5fb, 0x188a5a1, 0x188a5eb, 0x188a5a8, - 0x188a584, 0x188a5d2, 0x188a599, 0x188a598, - 0x188a583, 0x18ba4c9, 0x188a5d0, 0x188a594, - 0x188a582, 0x188a5cb, 0x188a5d8, 0x188a5e7, - 0x188a581, 0x188a5ea, 0x188a5a9, 0x188a5a6, - 0x188a580, 0x188a5a0, 0x188a59d, 0x188a5c3, - 0x188a57f, 0x188a5c0, 0x188a5de, 0x188a5d4, - 0x188a57e, 0x188a5c2, 0x188a592, 0x188a5cd, - 0x188a57d, 0x188a5a3, 0x188a5e8, 0x188a5a2, - 0x188a57c, 0x188a58e, 0x188a5b3, 0x188a5b2, - 0x188a5b1, 0x188a5b0, 0x188a5af, 0x188a5ae, - 0x188a5ad, 0x188a5ac, 0x188a5ab, 0x188a5da, - 0x188a5e4, 0x188a5e5, 0x188a5d9, 0x188a5b5, - 0x188a5bc, 0x188a5bd, 0x188a5e9, 0x188a5cc, - 0x188a585, 0x188a5d3, 0x188a5e2, 0x188a595, - 0x188a596, 0x188a5b8, 0x188a590, 0x188a5c9, - 0x188a5a4, 0x188a5e6, 0x188a5a5, 0x188a5ce, - 0x188a5bf, 0x188a572, 0x188a59b, 0x188a5be, - 0x188a5c7, 0x188a5ca, 0x188a5d5, 0x188a57b, - 0x188a58d, 0x188a58c, 0x188a58b, 0x188a58a, - 0x18ba4c8, 0x188a5c5, 0x188a5fa, 0x188a5bb, - 0x188a5c1, 0x188a5cf, 0x188a5b9, 0x188a5b6, - 0x188a597, 0x188a5fe, 0x188a5d7, 0x188a5ba, - 0x188a591, 0x188a5c6, 0x188a5dc, 0x188a57a, - 0x188a59f, 0x188a5f9, 0x188a5b4, 0x188a5a7, - 0x188a58f, 0x188a5fd, 0x188a5b7, 0x188a593, - 0x188a59e, 0x188a5f8, 0x188a5ff, 0x188a5fc, - 0x188a579, 0x188a5f7, 0x3114ba2, 0x3114ba3, -}; - -static const uint8_t table_18_vlc_len[NB_VLC_TABLE_18] = { - 1, 2, 3, 5, 6, 6, 7, 7, - 7, 7, 8, 8, 8, 9, 9, 9, - 9, 10, 10, 10, 10, 10, 10, 11, - 11, 11, 11, 11, 12, 12, 12, 12, - 12, 12, 12, 13, 13, 13, 13, 13, - 13, 13, 13, 14, 14, 14, 14, 14, - 15, 15, 15, 15, 15, 15, 15, 16, - 16, 16, 16, 16, 16, 17, 17, 17, - 17, 17, 17, 17, 17, 18, 18, 18, - 18, 18, 18, 18, 19, 19, 19, 19, - 19, 19, 19, 19, 20, 20, 20, 20, - 20, 20, 20, 20, 21, 21, 21, 21, - 21, 21, 21, 21, 21, 22, 22, 22, - 22, 22, 22, 22, 23, 23, 23, 23, - 23, 23, 23, 24, 24, 24, 24, 24, - 24, 24, 24, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 25, 25, - 25, 25, 25, 25, 25, 25, 26, 26, -}; - -static const uint16_t table_18_vlc_run[NB_VLC_TABLE_18] = { - 1, 1, 1, 1, 1, 1, 1, 1, - 12, 1, 20, 1, 1, 1, 32, 1, - 1, 1, 1, 1, 60, 1, 1, 1, - 1, 100, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 180, 1, - 1, 320, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 0, -}; - -static const uint8_t table_18_vlc_level[NB_VLC_TABLE_18] = { - 0, 1, 2, 3, 4, 5, 8, 6, - 0, 7, 0, 9, 10, 11, 0, 12, - 13, 18, 14, 15, 0, 16, 17, 19, - 20, 0, 21, 22, 29, 24, 25, 23, - 26, 27, 28, 35, 30, 31, 0, 32, - 33, 0, 34, 36, 37, 38, 39, 40, - 46, 47, 42, 43, 41, 44, 45, 48, - 49, 50, 53, 51, 52, 61, 60, 55, - 56, 57, 58, 54, 59, 62, 63, 64, - 65, 66, 67, 68, 69, 70, 71, 72, - 73, 75, 76, 74, 77, 78, 79, 80, - 81, 82, 83, 84, 85, 86, 87, 88, - 89, 90, 91, 92, 93, 99, 100, 94, - 95, 96, 97, 98, 102, 101, 103, 105, - 104, 106, 107, 111, 109, 108, 113, 110, - 112, 114, 115, 225, 189, 188, 203, 202, - 197, 207, 169, 223, 159, 235, 152, 192, - 179, 201, 172, 149, 178, 120, 219, 150, - 127, 211, 125, 158, 247, 238, 163, 228, - 183, 217, 168, 122, 128, 249, 187, 186, - 136, 181, 255, 230, 135, 233, 222, 145, - 134, 167, 248, 209, 243, 216, 164, 140, - 157, 239, 191, 251, 156, 139, 242, 133, - 162, 213, 165, 212, 227, 198, 236, 234, - 117, 215, 124, 123, 254, 253, 148, 218, - 146, 147, 224, 143, 184, 185, 166, 132, - 129, 250, 151, 119, 193, 176, 245, 229, - 206, 144, 208, 137, 241, 237, 190, 240, - 131, 232, 252, 171, 205, 204, 118, 214, - 180, 126, 182, 175, 141, 138, 177, 153, - 194, 160, 121, 174, 246, 130, 200, 170, - 221, 196, 142, 210, 199, 155, 154, 244, - 220, 195, 161, 231, 173, 226, 116, 255, +static const CFHD_RL_ELEM table_18_vlc[NB_VLC_TABLE_18] = { + { 1, 0, 1 }, { 1, 1, 2 }, { 1, 4, 6 }, { 1, 14, 10 }, + { 1, 23, 12 }, { 1, 41, 15 }, { 1, 54, 17 }, { 1, 69, 19 }, + { 1, 77, 20 }, { 1, 85, 21 }, { 1, 108, 24 }, { 1, 237, 25 }, + { 1, 238, 25 }, { 1, 101, 23 }, { 1, 172, 25 }, { 1, 173, 25 }, + { 1, 170, 25 }, { 1, 171, 25 }, { 1, 227, 25 }, { 1, 162, 25 }, + { 1, 156, 25 }, { 1, 157, 25 }, { 1, 243, 25 }, { 1, 134, 25 }, + { 1, 135, 25 }, { 1, 136, 25 }, { 1, 128, 25 }, { 1, 129, 25 }, + { 1, 247, 25 }, { 1, 127, 25 }, { 1, 178, 25 }, { 1, 179, 25 }, + { 1, 214, 25 }, { 1, 118, 25 }, { 1, 204, 25 }, { 1, 205, 25 }, + { 1, 198, 25 }, { 1, 199, 25 }, { 1, 245, 25 }, { 1, 246, 25 }, + { 1, 242, 25 }, { 1, 244, 25 }, { 1, 230, 25 }, { 1, 119, 25 }, + { 1, 193, 25 }, { 1, 194, 25 }, { 1, 186, 25 }, { 1, 187, 25 }, + { 1, 211, 25 }, { 1, 190, 25 }, { 1, 163, 25 }, { 1, 164, 25 }, + { 1, 220, 25 }, { 1, 221, 25 }, { 1, 216, 25 }, { 1, 217, 25 }, + { 1, 212, 25 }, { 1, 213, 25 }, { 1, 206, 25 }, { 1, 208, 25 }, + { 1, 209, 25 }, { 1, 210, 25 }, { 1, 122, 25 }, { 1, 248, 25 }, + { 1, 235, 25 }, { 1, 148, 25 }, { 1, 253, 25 }, { 1, 254, 25 }, + { 1, 123, 25 }, { 1, 124, 25 }, { 1, 215, 25 }, { 1, 117, 25 }, + { 1, 234, 25 }, { 1, 236, 25 }, { 1, 142, 25 }, { 1, 143, 25 }, + { 1, 153, 25 }, { 1, 154, 25 }, { 1, 176, 25 }, { 1, 177, 25 }, + { 1, 174, 25 }, { 1, 175, 25 }, { 1, 184, 25 }, { 1, 185, 25 }, + { 1, 240, 25 }, { 1, 241, 25 }, { 1, 239, 25 }, { 1, 141, 25 }, + { 1, 139, 25 }, { 1, 140, 25 }, { 1, 125, 25 }, { 1, 126, 25 }, + { 1, 130, 25 }, { 1, 131, 25 }, { 1, 228, 25 }, { 1, 229, 25 }, + { 1, 232, 25 }, { 1, 233, 25 }, { 1, 132, 25 }, { 1, 133, 25 }, + { 1, 137, 25 }, { 1, 138, 25 }, { 1, 255, 25 }, { 1, 116, 26 }, + { 0, 255, 26 }, { 1, 249, 25 }, { 1, 250, 25 }, { 1, 251, 25 }, + { 1, 252, 25 }, { 1, 120, 25 }, { 1, 121, 25 }, { 1, 222, 25 }, + { 1, 224, 25 }, { 1, 218, 25 }, { 1, 219, 25 }, { 1, 200, 25 }, + { 1, 201, 25 }, { 1, 191, 25 }, { 1, 192, 25 }, { 1, 149, 25 }, + { 1, 150, 25 }, { 1, 151, 25 }, { 1, 152, 25 }, { 1, 146, 25 }, + { 1, 147, 25 }, { 1, 144, 25 }, { 1, 145, 25 }, { 1, 165, 25 }, + { 1, 166, 25 }, { 1, 167, 25 }, { 1, 168, 25 }, { 1, 158, 25 }, + { 1, 159, 25 }, { 1, 223, 25 }, { 1, 169, 25 }, { 1, 207, 25 }, + { 1, 197, 25 }, { 1, 202, 25 }, { 1, 203, 25 }, { 1, 188, 25 }, + { 1, 189, 25 }, { 1, 225, 25 }, { 1, 226, 25 }, { 1, 195, 25 }, + { 1, 196, 25 }, { 1, 182, 25 }, { 1, 183, 25 }, { 1, 231, 25 }, + { 1, 155, 25 }, { 1, 160, 25 }, { 1, 161, 25 }, { 1, 48, 16 }, + { 1, 36, 14 }, { 1, 30, 13 }, { 1, 19, 11 }, { 1, 11, 9 }, + { 32, 0, 9 }, { 1, 15, 10 }, { 1, 24, 12 }, { 1, 42, 15 }, + { 1, 62, 18 }, { 1, 70, 19 }, { 1, 94, 22 }, { 1, 180, 25 }, + { 1, 181, 25 }, { 1, 109, 24 }, { 1, 102, 23 }, { 1, 86, 21 }, + { 1, 78, 20 }, { 1, 55, 17 }, { 1, 49, 16 }, { 1, 37, 14 }, + { 1, 31, 13 }, { 1, 25, 12 }, { 1, 43, 15 }, { 1, 63, 18 }, + { 1, 71, 19 }, { 1, 79, 20 }, { 1, 87, 21 }, { 1, 88, 21 }, + { 1, 56, 17 }, { 1, 50, 16 }, { 1, 38, 14 }, { 180, 0, 13 }, + { 1, 6, 7 }, { 1, 3, 5 }, { 1, 12, 9 }, { 1, 20, 11 }, + { 100, 0, 11 }, { 60, 0, 10 }, { 20, 0, 8 }, { 12, 0, 7 }, + { 1, 9, 8 }, { 1, 16, 10 }, { 1, 32, 13 }, { 1, 44, 15 }, + { 1, 64, 18 }, { 1, 72, 19 }, { 1, 80, 20 }, { 1, 95, 22 }, + { 1, 96, 22 }, { 1, 110, 24 }, { 1, 111, 24 }, { 1, 103, 23 }, + { 1, 104, 23 }, { 1, 105, 23 }, { 1, 57, 17 }, { 1, 65, 18 }, + { 1, 73, 19 }, { 1, 81, 20 }, { 1, 89, 21 }, { 1, 97, 22 }, + { 1, 98, 22 }, { 1, 58, 17 }, { 1, 39, 14 }, { 1, 26, 12 }, + { 1, 21, 11 }, { 1, 13, 9 }, { 1, 7, 7 }, { 1, 5, 6 }, + { 1, 10, 8 }, { 1, 33, 13 }, { 1, 51, 16 }, { 1, 66, 18 }, + { 1, 74, 19 }, { 1, 82, 20 }, { 1, 90, 21 }, { 1, 91, 21 }, + { 1, 59, 17 }, { 1, 45, 15 }, { 1, 52, 16 }, { 1, 67, 18 }, + { 1, 75, 19 }, { 1, 83, 20 }, { 1, 112, 24 }, { 1, 113, 24 }, + { 1, 106, 23 }, { 1, 99, 22 }, { 1, 92, 21 }, { 1, 68, 18 }, + { 1, 76, 19 }, { 1, 84, 20 }, { 1, 107, 23 }, { 1, 114, 24 }, + { 1, 115, 24 }, { 1, 100, 22 }, { 1, 93, 21 }, { 1, 46, 15 }, + { 1, 27, 12 }, { 1, 34, 13 }, { 320, 0, 13 }, { 1, 28, 12 }, + { 1, 17, 10 }, { 1, 22, 11 }, { 1, 40, 14 }, { 1, 53, 16 }, + { 1, 60, 17 }, { 1, 61, 17 }, { 1, 47, 15 }, { 1, 35, 13 }, + { 1, 29, 12 }, { 1, 18, 10 }, { 1, 8, 7 }, { 1, 2, 3 }, }; av_cold int ff_cfhd_init_vlcs(CFHDContext *s) { int i, j, ret = 0; - uint32_t new_cfhd_vlc_bits[NB_VLC_TABLE_18 * 2]; uint8_t new_cfhd_vlc_len[NB_VLC_TABLE_18 * 2]; uint16_t new_cfhd_vlc_run[NB_VLC_TABLE_18 * 2]; int16_t new_cfhd_vlc_level[NB_VLC_TABLE_18 * 2]; @@ -277,26 +135,22 @@ av_cold int ff_cfhd_init_vlcs(CFHDContext *s) /* Table 9 */ for (i = 0, j = 0; i < NB_VLC_TABLE_9; i++, j++) { - new_cfhd_vlc_bits[j] = table_9_vlc_bits[i]; - new_cfhd_vlc_len[j] = table_9_vlc_len[i]; - new_cfhd_vlc_run[j] = table_9_vlc_run[i]; - new_cfhd_vlc_level[j] = table_9_vlc_level[i]; + new_cfhd_vlc_len[j] = table_9_vlc[i].len; + new_cfhd_vlc_run[j] = table_9_vlc[i].run; + new_cfhd_vlc_level[j] = table_9_vlc[i].level; /* Don't include the zero level nor escape bits */ - if (table_9_vlc_level[i] && - new_cfhd_vlc_bits[j] != table_9_vlc_bits[NB_VLC_TABLE_9-1]) { - new_cfhd_vlc_bits[j] <<= 1; + if (table_9_vlc[i].level && table_9_vlc[i].run) { new_cfhd_vlc_len[j]++; j++; - new_cfhd_vlc_bits[j] = (table_9_vlc_bits[i] << 1) | 1; - new_cfhd_vlc_len[j] = table_9_vlc_len[i] + 1; - new_cfhd_vlc_run[j] = table_9_vlc_run[i]; - new_cfhd_vlc_level[j] = -table_9_vlc_level[i]; + new_cfhd_vlc_len[j] = table_9_vlc[i].len + 1; + new_cfhd_vlc_run[j] = table_9_vlc[i].run; + new_cfhd_vlc_level[j] = -table_9_vlc[i].level; } } - ret = init_vlc(&s->vlc_9, VLC_BITS, j, new_cfhd_vlc_len, - 1, 1, new_cfhd_vlc_bits, 4, 4, 0); + ret = ff_init_vlc_from_lengths(&s->vlc_9, VLC_BITS, j, new_cfhd_vlc_len, + 1, NULL, 0, 0, 0, 0, s->avctx); if (ret < 0) return ret; for (i = 0; i < s->vlc_9.table_size; i++) { @@ -318,26 +172,22 @@ av_cold int ff_cfhd_init_vlcs(CFHDContext *s) /* Table 18 */ for (i = 0, j = 0; i < NB_VLC_TABLE_18; i++, j++) { - new_cfhd_vlc_bits[j] = table_18_vlc_bits[i]; - new_cfhd_vlc_len[j] = table_18_vlc_len[i]; - new_cfhd_vlc_run[j] = table_18_vlc_run[i]; - new_cfhd_vlc_level[j] = table_18_vlc_level[i]; + new_cfhd_vlc_len[j] = table_18_vlc[i].len; + new_cfhd_vlc_run[j] = table_18_vlc[i].run; + new_cfhd_vlc_level[j] = table_18_vlc[i].level; /* Don't include the zero level nor escape bits */ - if (table_18_vlc_level[i] && - new_cfhd_vlc_bits[j] != table_18_vlc_bits[NB_VLC_TABLE_18-1]) { - new_cfhd_vlc_bits[j] <<= 1; + if (table_18_vlc[i].level && table_18_vlc[i].run) { new_cfhd_vlc_len[j]++; j++; - new_cfhd_vlc_bits[j] = (table_18_vlc_bits[i] << 1) | 1; - new_cfhd_vlc_len[j] = table_18_vlc_len[i] + 1; - new_cfhd_vlc_run[j] = table_18_vlc_run[i]; - new_cfhd_vlc_level[j] = -table_18_vlc_level[i]; + new_cfhd_vlc_len[j] = table_18_vlc[i].len + 1; + new_cfhd_vlc_run[j] = table_18_vlc[i].run; + new_cfhd_vlc_level[j] = -table_18_vlc[i].level; } } - ret = init_vlc(&s->vlc_18, VLC_BITS, j, new_cfhd_vlc_len, - 1, 1, new_cfhd_vlc_bits, 4, 4, 0); + ret = ff_init_vlc_from_lengths(&s->vlc_18, VLC_BITS, j, new_cfhd_vlc_len, + 1, NULL, 0, 0, 0, 0, s->avctx); if (ret < 0) return ret; av_assert0(s->vlc_18.table_size == 4572); From patchwork Sat Sep 3 20:35:57 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37638 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:139a:b0:8f:1db5:eae2 with SMTP id w26csp1748081pzh; Sat, 3 Sep 2022 13:36:27 -0700 (PDT) X-Google-Smtp-Source: AA6agR7HijDnd2gH6J0IX/s+hNBEL5jvMsA9oWgrCZnm4thTNwpvTxbzM5Vp+JF8XvmD+M6wVbMN X-Received: by 2002:a17:907:9812:b0:742:2865:eca8 with SMTP id ji18-20020a170907981200b007422865eca8mr16137972ejc.537.1662237387298; Sat, 03 Sep 2022 13:36:27 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id sb28-20020a1709076d9c00b007413d6c7ebcsi3333948ejc.362.2022.09.03.13.36.26; Sat, 03 Sep 2022 13:36:27 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b=Xy3e9hVG; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 3DDDC68B411; Sat, 3 Sep 2022 23:36:18 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-oln040092065072.outbound.protection.outlook.com [40.92.65.72]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 2794368009F for ; Sat, 3 Sep 2022 23:36:11 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cVKU7bLq8Zv1hg1lNCZcrUGnx7eHkuBxWMbg3WH2lVuCKYGIboPEOVLLX8lzP+BZvALlNZtKhD3KdWUCIxDJeMMf+Q0Qo0sO5OgFVjK94/K8MSwc3e+qynBcQxqOWV1z/f1GTDUvu3BaQ8qq0nW3/Aq8iFReTgQlpZ6uOh9uAEEvekjxfQsA887W/QDBZAFpk6uFcWmonJJ8j90mb4mKlausiH7NWqCuPRmBLFSwmmJ1hkut6EFgTCFNAasUQ4rA8cPys6SuWeDRun9ZVX2Y+4qMFmOspJ/S8CrOYIJhMwtKHGltASwBMd7SHhc5FKuSHKMEVx5gZ8ZdL1K0/ORxOg== 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=pgp4RhThCY2pddbtqvYJj3fIQxOXO/iWrIt878ZbZtE=; b=kkcrNGOurycTWK9HlQsa8/xQKwE6uA2jZhgsT7svzIclnePMriSW7rUbQD7l7G1uiJjZdp/JpWAXgOgQS4JExGeH4SdwXbDCEiEeQ+tJRqabbwH2ThlJEPh81p/9+TGBxXqpq5un7YPloMcCRS9K6xllnb2gi4hDC6CASJ66Rng9MxUhrkaCCvFuzVSy9A/Q2crMZDZHgNsFRn+D15SZP2bmJvacasWdwgjpDF8tndayzuVF+PtKKoZN50vfsz+AoLcGh2GEgwFckAmC0QhENmxbHlPIj0EhkUrz5dcvI3xAAJp77TrbPduXym54Wwpm5QlX+V2toVMAkEXIlwEFDw== 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=pgp4RhThCY2pddbtqvYJj3fIQxOXO/iWrIt878ZbZtE=; b=Xy3e9hVGIoRwpu3GepJ2cHJWnVVNWVODuZVSuNiMBmzXpNpIvsDw2FiyJJC2QnAltzHm7WcNusl69A5Kxqo/aHY10vojQmh3y7kje0rAzfBxNZ87Vo+nSsKPTY+BDDr5aaVpCZi/th8UDHnFv4RgLI2iFy+xC/20lFJW18iYFyPGa5xwtLFYf01xDOLQuh6qVP60Bk9v+ae2A6NxdpVLavEnNLA5vFy3chv9FILRlFkX7ix3J/uBD3PdYYx177bZUf2FLbQCMdL/JwkfkfTzsOxEEPKleYrAEQXaLT1ye/PT+JFKjP84A2KJR7gx+3z0UdFW5aJwrAPya37u7ZAJ0w== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by PR3P250MB0241.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:17c::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5588.12; Sat, 3 Sep 2022 20:36:05 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47%2]) with mapi id 15.20.5588.010; Sat, 3 Sep 2022 20:36:05 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Sat, 3 Sep 2022 22:35:57 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: X-TMN: [3Eyp6KSZfiMJOeJSSmS1vZoX3lbJmDJ3CgrIU2rssHI=] X-ClientProxiedBy: AM6P193CA0086.EURP193.PROD.OUTLOOK.COM (2603:10a6:209:88::27) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <20220903203559.1961353-2-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 8f8fb4bb-c9d6-4143-e8e2-08da8debec43 X-MS-Exchange-SLBlob-MailProps: AZnQBsB9XmpvZmz4TmgkoDzJrsVMX7C8czoNbO2n/w68gWYTXszAZbMgPObdHJDFXAL6m20sa9XBAFH7ovRN9PpcZI3LyZqHJy/lyjzF2EEgP5DFgeUTHPxsc+Cx33rM+qsrLGWH43aQCiX+7VfDurzxqy0dXOYSTKTf4k+A8hmg+LKpqlx0FUW00mYIq3dM4/ZuHhonm5G+j4m/6EmvrIYhkha3IydQu0xyET4gs8s+baciG8YpWpR2nTAdAa9XzWcUGPRdzzBAyFwm6vYhJRjnY1RJYB2ti1m66QHtIZeMd0SuRxtvTuLY5UthFPJoT+ReyGLt/rdv9W3m6lUEN8yeIj5QnKo/2EcmcVe8AYE3AmoMsehaSnYJqT72bWar67wJ7HHDdTeAr4yJzL9yoNj1mXygitvgVbseSGgn+9XFDT7ot3fL3caJe5HqFbczzIZBovER2K3T+bFXTSdrmI6t+Yam5wU9zZUu470b3pGd4yr3wXvbOadePumJA1XvLlkVY52Eq1vjpcYam6xLFR1Crl49BxnVIwfUUpD7HPTMpyGYdzWG+V4v96/WhXkd8uggAF4iGsjNdkWhHB8KQZPI2Tz4ItKcKtRowsVJ593zq792oeyFOJ9l95woIR6pRetpnBuJAlp5ARz7Pw578BKgjaQPYndw2xt9jxyZL+LngZdY/5o6jBSitLFeGV+R6S6Lf6u7hDCzchWvJBJb9V6beCp8PejxGJTGA2sk9k+jDOktOfwpQO5V18KI7yik2JSuoZdzf08= X-MS-TrafficTypeDiagnostic: PR3P250MB0241:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: uJ9jbYR2p62Q9PB3YdHgJhRJiW0+mfEoVB9IF/3Ehdb/f6wFI8xXM2hbaZBGRPAGSYYbqQBB6maOMCBjrQlQib/pgN4jb9mCscxLIm30V6GFRn1r9eOuDx9eDhdeD9WEqmVJ7peajsIu0obUH+U3f4vZ7+uQ+Fb9FVLqKsKgB8Wj9brRpwYTcjjE4eQLW94/gfHxnTn+jtE1QuNzHdRQiR60aodCY3CebZUlymUhFD7xIwE0qRr7zjE2dseC5olQpqav5PyAIuOE3OB6MHpxQINdMCjZNRLYLcFvKSXwlHs9IrXw3Fexp4AGJJDB8PfjqYbC+mrVMQ7AU7Oyfp+QUWksGfLEHW1Q3ODYCzsK32cluI556KaWjsesWuEqRZ4+GJ1Vvz1D4Npu1Ai2ulzzqbBwRoTdk3c3GJpxnpaYJ7jldq5i32QVareiqQ3FYU9n7ITugQ7zk7AyZRnuypS7qzgB9tRiT7Ur87wbeCVJK9Kc/5lwPvDjGR+MM4euPa9WXZxSuhtnDCa406RO4Dbci5JUoUPzrxet+BykT6KML+e9HTISRpPNLfpqfHVlPJExgcFhNDtskn1fzsUdALMqLzy1Ro7PtlAUx6n3yMee2pfCDvCqoUhvY3QemH/T013ShLIY4RXbDBq2za2md6VlRw== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 1amVfmTILk1Jg/dmSByMyZXNYDjZo8zVKsuG6WmnKa3UwvSlBKnGIF6iuv5edHeuEEGjMR35+qKFwHuuEpyoEhAU66yNAguRhG8wPhBa4H+Uf+3XIDfpYtXib/ZLONi3urDdyRKw3XPkIYZO75EZFpJI0Y72F8+PU4sOOlk68D2UWtenh+cowYEkmxkvyPTMB5XZQ4pJzVNU+ro1frlJiUI30h/utPnMg5GCT/nCFYj55PeodyHHbOXGSqc5txAiXx/XQsTjEksH+TkLbllXUdq79GpW99gmUXlwx5/7H8qdWY2aEapurLSOdABgvQdiRyL+6zMZ8TKaoe54nPt+TbXt+/KpRvxNm7Q+6b78PzPHHRWjwPGBFiVDu6B4KV1wgUc3jqN6rke9j0zafLjClEXfdqCTwhTTNje+F4pzH0fGyYlgHuK5l01IsQSwq44FHa/7U1AGRp1q5RkdUfJ6CE0jsGw90TbHSkCUO4DbTa5X8BMSm6mgRt2hma6vIOzmIkSKxOKU6d448+NIVD6isPqgypX1TlgLnhJwocMXiMT/MDGFfjS6l5hbo4/7Wd8SoyTVGT+ejl23gLZDTXqMLL2dt4EL3+NDKApcEBn70+iLgsPgJN1W1Wo56qtF/AbhqnBVxDAiUeq/mZQpjB3cBGUQJDN0pok0Q9N5fv4DinCrdbNfUH5dClX5s9R/o3mCLBIQK64ZDn5Ae4DHJUIfA1okY5TtUlI0TpDUd1jeBSYHk3gOp1uJtgJhRLJJabC0hNHwSt3QL7j7z5YDyBxzcWSS049OfYrfP+jC0i1cUZhiT1KnURu+6T8JMLeUMsIqXQzQpc1/ag5Ep0wAs2tPXzCEercXI4jMZKU3CyXy4+cBAxw+QJiaaKl06YkVplF5CUZDKxmK3KUEk/1cvcosf2qFN4mZpY4oRKBjFjdbVf1ciFanrbT0M2kcI4YSQhVccF9xkcYYO2WRtSh8ucbSY+OMTXIUCr77M3QPm3ytSJ0+L4zTfJpv+z2f9B2w1ZEerWmc63Dbdrg0kxZKJWIzwy3U7PcPN8E+RhTu7sznfSzFFetkGQoSGAyJ/aY+3xcriCI7SPJRcLUgQNoQuBOaod6QDx7FKBlbyoAdFOzUjZ9mppEiqqJ2zML6TryO4MZQSA6pxyv8saQL/625gX7DKkoW6+a5/+qd+WpcZecIqfNdjPDaCIAXN9LzVKNnZoPTUHMNbtFICtGI+L6/jqjRf+VPAMvQ/BUGN3jwWR7Rk18OG3K/wWbGzute8YWJRmxt+pd9vlTxfhEJ+2IW0G8ruBrAqo32Rt80/1AzR6jcBrMNpY83fIxtkez29/MxNcrG X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8f8fb4bb-c9d6-4143-e8e2-08da8debec43 X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2022 20:36:04.9393 (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: PR3P250MB0241 Subject: [FFmpeg-devel] [PATCH 3/5] avcodec/cfhddata: Avoid code duplication when creating codebooks 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 Cc: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: Tfmlx3/TAGvK Signed-off-by: Andreas Rheinhardt --- libavcodec/cfhddata.c | 94 ++++++++++++++++++------------------------- 1 file changed, 39 insertions(+), 55 deletions(-) diff --git a/libavcodec/cfhddata.c b/libavcodec/cfhddata.c index 7c4b1454f3..017eb9375c 100644 --- a/libavcodec/cfhddata.c +++ b/libavcodec/cfhddata.c @@ -124,38 +124,42 @@ static const CFHD_RL_ELEM table_18_vlc[NB_VLC_TABLE_18] = { { 1, 29, 12 }, { 1, 18, 10 }, { 1, 8, 7 }, { 1, 2, 3 }, }; -av_cold int ff_cfhd_init_vlcs(CFHDContext *s) +static av_cold int cfhd_init_vlc(CFHD_RL_VLC_ELEM out[], unsigned out_size, + const CFHD_RL_ELEM table_vlc[], unsigned table_size, + VLC *vlc, void *logctx) { - int i, j, ret = 0; uint8_t new_cfhd_vlc_len[NB_VLC_TABLE_18 * 2]; uint16_t new_cfhd_vlc_run[NB_VLC_TABLE_18 * 2]; int16_t new_cfhd_vlc_level[NB_VLC_TABLE_18 * 2]; + unsigned j; + int ret; /** Similar to dv.c, generate signed VLC tables **/ - /* Table 9 */ - for (i = 0, j = 0; i < NB_VLC_TABLE_9; i++, j++) { - new_cfhd_vlc_len[j] = table_9_vlc[i].len; - new_cfhd_vlc_run[j] = table_9_vlc[i].run; - new_cfhd_vlc_level[j] = table_9_vlc[i].level; + for (unsigned i = j = 0; i < table_size; i++, j++) { + new_cfhd_vlc_len[j] = table_vlc[i].len; + new_cfhd_vlc_run[j] = table_vlc[i].run; + new_cfhd_vlc_level[j] = table_vlc[i].level; /* Don't include the zero level nor escape bits */ - if (table_9_vlc[i].level && table_9_vlc[i].run) { + if (table_vlc[i].level && table_vlc[i].run) { new_cfhd_vlc_len[j]++; j++; - new_cfhd_vlc_len[j] = table_9_vlc[i].len + 1; - new_cfhd_vlc_run[j] = table_9_vlc[i].run; - new_cfhd_vlc_level[j] = -table_9_vlc[i].level; + new_cfhd_vlc_len[j] = table_vlc[i].len + 1; + new_cfhd_vlc_run[j] = table_vlc[i].run; + new_cfhd_vlc_level[j] = -table_vlc[i].level; } } - ret = ff_init_vlc_from_lengths(&s->vlc_9, VLC_BITS, j, new_cfhd_vlc_len, - 1, NULL, 0, 0, 0, 0, s->avctx); + ret = ff_init_vlc_from_lengths(vlc, VLC_BITS, j, new_cfhd_vlc_len, + 1, NULL, 0, 0, 0, 0, logctx); if (ret < 0) return ret; - for (i = 0; i < s->vlc_9.table_size; i++) { - int code = s->vlc_9.table[i].sym; - int len = s->vlc_9.table[i].len; + av_assert0(vlc->table_size == out_size); + + for (unsigned i = 0; i < out_size; i++) { + int code = vlc->table[i].sym; + int len = vlc->table[i].len; int level, run; if (len < 0) { // more bits needed @@ -165,49 +169,29 @@ av_cold int ff_cfhd_init_vlcs(CFHDContext *s) run = new_cfhd_vlc_run[code]; level = new_cfhd_vlc_level[code]; } - s->table_9_rl_vlc[i].len = len; - s->table_9_rl_vlc[i].level = level; - s->table_9_rl_vlc[i].run = run; + out[i].len = len; + out[i].level = level; + out[i].run = run; } - /* Table 18 */ - for (i = 0, j = 0; i < NB_VLC_TABLE_18; i++, j++) { - new_cfhd_vlc_len[j] = table_18_vlc[i].len; - new_cfhd_vlc_run[j] = table_18_vlc[i].run; - new_cfhd_vlc_level[j] = table_18_vlc[i].level; + return 0; +} - /* Don't include the zero level nor escape bits */ - if (table_18_vlc[i].level && table_18_vlc[i].run) { - new_cfhd_vlc_len[j]++; - j++; - new_cfhd_vlc_len[j] = table_18_vlc[i].len + 1; - new_cfhd_vlc_run[j] = table_18_vlc[i].run; - new_cfhd_vlc_level[j] = -table_18_vlc[i].level; - } - } +av_cold int ff_cfhd_init_vlcs(CFHDContext *s) +{ + int ret; - ret = ff_init_vlc_from_lengths(&s->vlc_18, VLC_BITS, j, new_cfhd_vlc_len, - 1, NULL, 0, 0, 0, 0, s->avctx); + /* Table 9 */ + ret = cfhd_init_vlc(s->table_9_rl_vlc, FF_ARRAY_ELEMS(s->table_9_rl_vlc), + table_9_vlc, FF_ARRAY_ELEMS(table_9_vlc), + &s->vlc_9, s->avctx); if (ret < 0) return ret; - av_assert0(s->vlc_18.table_size == 4572); - - for (i = 0; i < s->vlc_18.table_size; i++) { - int code = s->vlc_18.table[i].sym; - int len = s->vlc_18.table[i].len; - int level, run; - - if (len < 0) { // more bits needed - run = 0; - level = code; - } else { - run = new_cfhd_vlc_run[code]; - level = new_cfhd_vlc_level[code]; - } - s->table_18_rl_vlc[i].len = len; - s->table_18_rl_vlc[i].level = level; - s->table_18_rl_vlc[i].run = run; - } - - return ret; + /* Table 18 */ + ret = cfhd_init_vlc(s->table_18_rl_vlc, FF_ARRAY_ELEMS(s->table_18_rl_vlc), + table_18_vlc, FF_ARRAY_ELEMS(table_18_vlc), + &s->vlc_18, s->avctx); + if (ret < 0) + return ret; + return 0; } From patchwork Sat Sep 3 20:35:58 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37639 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:139a:b0:8f:1db5:eae2 with SMTP id w26csp1748116pzh; Sat, 3 Sep 2022 13:36:36 -0700 (PDT) X-Google-Smtp-Source: AA6agR6H1aaJCzdrqRCIWQIw4JMdtp8V36/xmvbXdZfuDgbJvQAmPjgCpkDOAG9fVCTcliM5jZkO X-Received: by 2002:a17:907:a068:b0:741:3cb1:9ffe with SMTP id ia8-20020a170907a06800b007413cb19ffemr25460427ejc.293.1662237396036; Sat, 03 Sep 2022 13:36:36 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id l14-20020a1709062a8e00b0073d888f8c4fsi3979361eje.236.2022.09.03.13.36.35; Sat, 03 Sep 2022 13:36:36 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b=Mp2CPk55; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5C18268B779; Sat, 3 Sep 2022 23:36:23 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-oln040092065072.outbound.protection.outlook.com [40.92.65.72]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 543B068AAD7 for ; Sat, 3 Sep 2022 23:36:16 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=QNq26KXdElFQm0s9D2IQM3fRLTKG5/VgLbWLmexOHI3lB4QkGdDX76m4j0D9zT95fyr9/53LSYh/gCcLjkkZF5ZxT8SerQd1w6o2GrUM6b7x7LL/C1+zPMTPQnYxvAR2PU1AcvfeQ+Eq/NhEJjUyI45pmu1qdPzBmY4G2vEccIeAvIMpnaqxK791t5De0nGcVNWXOkRbmO/waotWRvl0ASyHfOXePWF1oQ/S5/yQdQsFUiqoDvnXC5HqaNaWljSnalPmGI1tA1+qkmdtFrNUGJHfksLvXBqsIFApfOc1i/cslZ0Y4R9iCFoHAmCEUq4Louk5DyNbPFojn2xplykq6g== 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=BNsl8pKduoxM9sNjepQ3w7z6ekHTfI80aDf37ye940Y=; b=a32Itbpcmk7X9Xh4DaoJ3p4YNE9WveNQPo3CvE8alDuSy44lYgdLZw2wjNrd7lr6ITAJQR7nm+OTuA9Ng9/JdWCKM9wPRJpf+phdWNjvwwLdZztCuQhBbFwmc0JRmgyZ9VPnCG5EYHwt4FdqTlVjc/EKExipXgNcE1cO+ltfYM0wJqFMoq9spYUI4+Rnx/tKpcs6+S/Zs9if+C2fiqElZin1H8Qh1pwIIW0b6niWUIEMLEOvY3VPQUeXfOy8lp5xnI3pgTVUUIU1+lUq53lGJGaNTeVhFkq9fi9RxU/Oxv+udSAwSU6ZwqmL6dyhgTY+Kb/Xr5KdTKgRNUopWGS06w== 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=BNsl8pKduoxM9sNjepQ3w7z6ekHTfI80aDf37ye940Y=; b=Mp2CPk55750NoDHYOn/53VwY4Ar8HswjG1HVovK/vUxNtNZAPjr5m8wpqT8ASo4PlN3Pa414F8NMG16DEzhlVMxMjl5OKxU02YmA/4CKRNLqccdcwY4L+t2q0EIg5WIFe6QRUmEeboCiM0pgkQzXs2nZ72f51ULOTSy9Pm/SG7D9otUgRx5mpO1wKZzULhu1Qubt80T9CCeb8ixdED2cx5HOD+jlr9UzRtoR3k32PlDJzbhueBVJ/lfbu105ZwQTcFTvekc8XZ/jF3Qgs75UM40o3UnELVllORXSSwEif185xEpZmEy46H0sAfMi0Ow1WcW0tnOtnEgw3hHiVBKrNA== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by PR3P250MB0241.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:17c::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5588.12; Sat, 3 Sep 2022 20:36:05 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47%2]) with mapi id 15.20.5588.010; Sat, 3 Sep 2022 20:36:05 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Sat, 3 Sep 2022 22:35:58 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: X-TMN: [Cty6esNtoOS7jKRBj/KruqODfoIuxsxjyoT/dext9lw=] X-ClientProxiedBy: AM6P193CA0086.EURP193.PROD.OUTLOOK.COM (2603:10a6:209:88::27) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <20220903203559.1961353-3-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a4199803-cd8e-46b4-7614-08da8debecdc X-MS-Exchange-SLBlob-MailProps: AZnQBsB9XmpvZmz4TmgkoDzJrsVMX7C8AGcxckHTBrCRl76o3+JNv354bj1sd5y2EJJDUOGyZ+t8TloRhBn1XTrV/3g+g9e2n1k+79XlpopkLNCoxD2kH+n0ZarxJtZSFci5pH6nS/sZwp+OX8MFHQO5KX7m9BCbo5N8vDNk+iM3ySBh3CkfGv4vpRWAjg9rm8kQyGsDp0qD5tHEpM+5cDdOOxr0NdX7cwhgr1Od92VUNkmapRrRAur81WDeffLqFKxplKj7Sg23Y58SMUSVJdlf+7t3l6i0IDbbvHSMvlDYdAzE5/6IHu8iI7a9DR01WVoz/x58BhN2sVkGfBEArPE67Lk0HQT1+UhHEmbVeSWQ8X/ZeQOahFqnzRUE2ZxsVapl3/P8P3BQMGvMGbpX3u1S6BwAPe+U+P8r7RMo5Gz5t5YN9Ut/ZwatfJ0Hq8AlrRW4O9SUxKHhF3tdTyb19Eo5EPBH8qOCiq5NuDcgQNMLd+IR0e9gZaypIs1xcvU7uALIT5pjX0O3P19wrGZI0zmOdy+8WVhQaydQBWffdV1EVz/1zXTJLhMg/vLNCcD1G8WKYzBkZKZ92j0nj+jyAF+IjmFqVZnUDmoVQeVWmfAYCQoxi2D8mdBAEiYatw2t8EwGTiGWawmyanBQIzUnLSkyXqMoPYwvcPbTnvKvq0SsrwPXHwBnvbJe0x7qY9Z10jWwbiVDOejpEeppAUuqGnJjADI+0lvbr3aFOMlnIMLx8KxXc8JK91BjeREUpYsNHVDoZT2kLlA= X-MS-TrafficTypeDiagnostic: PR3P250MB0241:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: DLXkgzOkRxkIzRPxDQdpOEXaDkvpD8R7cb6rpsgM6nDG5cP6vol4GEBQdwy22y70IQEVX6/Xjk0xibkJlrGm2O1XcQ9lKS61Wc3InCcckM3wGwFwpbaT28BLj6oEGhwhl4qpWsTWOiMuo1wVqb2KhAZ19K2fK3u/lP6XUrdqhbskdGNvtj2WeO1arLgYD4kfraZckAqaVpzeUDIyYzY+jP+P4M1uZJ6gATkSvVuGV1CdyiocxprbavfeZ/GJ24NWmdn2XLgfi0dpp10l9eyZrxQ+SbWGlWUuTxS1dS5lkz1EV19U1TRnMzQWhgdg1KJJsFO/J8EBEKz6+5K51THmXd87379eXhYqwRlLIECskBx1R8jEPCI0hj/LiTS3xmMBpEUhXdVgRH9nkcjUE9WxsCWvg6phh8f7bK59DOihCSequ2maL96+nHJIfNbrfrKPIMFISRGlfCLHzUpdS3xzSzf8EDoZc34DPCvOPikfiWDYSsht1zKKp/PokScf1/tHJDcfqyHh6dJC/15wkqJtJN7dIMC9tN4ttnLTq5BH+3w5F9SfJnUBkxd3MY8mCKejem/I7yrnQDA52P2mfGH/POT8/bHgxHIha+7g7bqCKSUdciFb15u+jMGty1edusd9qflLmwVtp5E/vkJdGyVwag== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: ddOYmpsfBC+Uncasy0skmASLczkr7L5GVyqOk6UUnMZd/u0kx+QUFjCVWszD6g/0B9My0iI9eCPP9gbQUgKxyN3chSaBfVVwj/VG34jzaEub1k2El4GLuUuaeH/8zO3ROVmq7LsvZR2lz5DhkVFRo7tpkEc9z7GUeeOvgDZa3L1nAB5rWGBHiLuDHVaZfqJvvjTiAJ/EBFZ+FdBCnTYCjEnqin7qvNVA0bQXyr3SKREk65zI+s76TccOky8uWJgTHZI/1oV5SZK1Va02/MbYLvp0qX2J8unOpVE5r77i/wdWvMx/Dhre01KFr11ffSZvJ+ddobkB438lLPKNBIfTIQEXcXx2yPDiNlb8BWJAWB7nMqAlHyRZx+YaKsFMZk549IZZeeq0VHe0D4T8Gzphsjly7oaeC01qBjs01JbIa5EYk0jLTA0bY4xSsAHyCX8zpz3PJNCIZ/csPL36kRfoj+aKOYhmWQhKSn+QvX52ICFYXzRVj091nX5/Ke2iFtvQxjsJVoDvaruXx7ORg5wJnLGB1JQWIC/6gZmij1mK6QJw50XzwQTtbBNvsKDI4tsWPTQvsctkJ9NwlFiFI67Jd+4MbS5aAPeJOEaSyzvLqt19X9OrrKeEJN+5VmnZtu+pPR4193dXVKkWPQ0GZrOr++XWGxrgErSuO+YK1OQDXhBcU+FJwMLh56tPdCX24Hi5KHgWRCqyNc9NeqMp7ok+GqOXcqLooB0RQfg0EICi/cDXTKXvDN+7hUHZKL8FF6W1RIYSJecIjUqOhIPIVw3UyH35nkfanDAmR+qpoyDhrrQuJRIFe97FLr6Y1R9pEmLp6VuybFp1iFOAi5NUzaVAdAGxlJAxqchAGiJbguh+zSSMr6Hy6+WI9aIPD5axVZMk8RSrNulTAXJqR0Y5+Z8jP2AXfx/OUdajvL46VsowoVMn336V816LV9lpZSd2Eyg8S7/etOoyUE6N0P+hfvW0PFF+gi/i9MGTSJiyqAogcCq+XmF1vPzmicBdtVA1WNechZDzS2EQJGdeYOD6uXG6oxe0lciaFX0dfQHMedcV5NT0KLb/rrtRKkpwcxT/mM6t4lkPngPCRgAmpQsgDqBIJMWoUhqBOYrg6j6jqmQWfNODkuuDZsIDpyWONi/xMkY02aYL8wrDTnWm641dXuTLu41Feq8QkrDblRLK7UIxza/lIoG09u97G7V+idueMJhNPwcKJihTjLLm7eom7gjNxauCYYBZJBDO+RRqDeLlAvz1AE0pppdOqm+56XmhAyUpb0n56P6IETpukw7eYi0YpQ1oSlzTAOUIDT1HQStpNO4P6gj2V++jbu8tAx53FMbP X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: a4199803-cd8e-46b4-7614-08da8debecdc X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2022 20:36:05.8922 (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: PR3P250MB0241 Subject: [FFmpeg-devel] [PATCH 4/5] avcodec/cfhd, cfhddata: Free VLC as soon as it is not needed 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 Cc: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: epSyBKE4esBv The VLC is only used to initialize RL VLC. Signed-off-by: Andreas Rheinhardt --- libavcodec/cfhd.c | 3 --- libavcodec/cfhd.h | 4 ---- libavcodec/cfhddata.c | 17 ++++++++++------- 3 files changed, 10 insertions(+), 14 deletions(-) diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index 90b3d0a850..c23eb069c6 100644 --- a/libavcodec/cfhd.c +++ b/libavcodec/cfhd.c @@ -1404,9 +1404,6 @@ static av_cold int cfhd_close(AVCodecContext *avctx) free_buffers(s); - ff_free_vlc(&s->vlc_9); - ff_free_vlc(&s->vlc_18); - return 0; } diff --git a/libavcodec/cfhd.h b/libavcodec/cfhd.h index 4c066da5fc..9b09c91262 100644 --- a/libavcodec/cfhd.h +++ b/libavcodec/cfhd.h @@ -26,7 +26,6 @@ #include "avcodec.h" #include "bytestream.h" #include "get_bits.h" -#include "vlc.h" #include "cfhddsp.h" enum CFHDParam { @@ -141,10 +140,7 @@ typedef struct CFHDContext { AVCodecContext *avctx; CFHD_RL_VLC_ELEM table_9_rl_vlc[2088]; - VLC vlc_9; - CFHD_RL_VLC_ELEM table_18_rl_vlc[4572]; - VLC vlc_18; int lut[2][256]; diff --git a/libavcodec/cfhddata.c b/libavcodec/cfhddata.c index 017eb9375c..efe932dc3b 100644 --- a/libavcodec/cfhddata.c +++ b/libavcodec/cfhddata.c @@ -23,6 +23,7 @@ #include "libavutil/attributes.h" #include "cfhd.h" +#include "vlc.h" #define NB_VLC_TABLE_9 (71 + 3) #define NB_VLC_TABLE_18 (263 + 1) @@ -126,11 +127,12 @@ static const CFHD_RL_ELEM table_18_vlc[NB_VLC_TABLE_18] = { static av_cold int cfhd_init_vlc(CFHD_RL_VLC_ELEM out[], unsigned out_size, const CFHD_RL_ELEM table_vlc[], unsigned table_size, - VLC *vlc, void *logctx) + void *logctx) { uint8_t new_cfhd_vlc_len[NB_VLC_TABLE_18 * 2]; uint16_t new_cfhd_vlc_run[NB_VLC_TABLE_18 * 2]; int16_t new_cfhd_vlc_level[NB_VLC_TABLE_18 * 2]; + VLC vlc; unsigned j; int ret; @@ -151,15 +153,15 @@ static av_cold int cfhd_init_vlc(CFHD_RL_VLC_ELEM out[], unsigned out_size, } } - ret = ff_init_vlc_from_lengths(vlc, VLC_BITS, j, new_cfhd_vlc_len, + ret = ff_init_vlc_from_lengths(&vlc, VLC_BITS, j, new_cfhd_vlc_len, 1, NULL, 0, 0, 0, 0, logctx); if (ret < 0) return ret; - av_assert0(vlc->table_size == out_size); + av_assert0(vlc.table_size == out_size); for (unsigned i = 0; i < out_size; i++) { - int code = vlc->table[i].sym; - int len = vlc->table[i].len; + int code = vlc.table[i].sym; + int len = vlc.table[i].len; int level, run; if (len < 0) { // more bits needed @@ -173,6 +175,7 @@ static av_cold int cfhd_init_vlc(CFHD_RL_VLC_ELEM out[], unsigned out_size, out[i].level = level; out[i].run = run; } + ff_free_vlc(&vlc); return 0; } @@ -184,13 +187,13 @@ av_cold int ff_cfhd_init_vlcs(CFHDContext *s) /* Table 9 */ ret = cfhd_init_vlc(s->table_9_rl_vlc, FF_ARRAY_ELEMS(s->table_9_rl_vlc), table_9_vlc, FF_ARRAY_ELEMS(table_9_vlc), - &s->vlc_9, s->avctx); + s->avctx); if (ret < 0) return ret; /* Table 18 */ ret = cfhd_init_vlc(s->table_18_rl_vlc, FF_ARRAY_ELEMS(s->table_18_rl_vlc), table_18_vlc, FF_ARRAY_ELEMS(table_18_vlc), - &s->vlc_18, s->avctx); + s->avctx); if (ret < 0) return ret; return 0; From patchwork Sat Sep 3 20:35:59 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 37640 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:139a:b0:8f:1db5:eae2 with SMTP id w26csp1748161pzh; Sat, 3 Sep 2022 13:36:45 -0700 (PDT) X-Google-Smtp-Source: AA6agR5Hp1m/Qs6vBF07MWG4Mjm3HBl0r9ogj3jq0AWwo7xhf5TWyFFjL2QEq0ZAacHEkSgmfuxI X-Received: by 2002:aa7:c488:0:b0:448:d11:4830 with SMTP id m8-20020aa7c488000000b004480d114830mr31717111edq.97.1662237405066; Sat, 03 Sep 2022 13:36:45 -0700 (PDT) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id y21-20020a170906471500b0073db945fcc0si4264675ejq.214.2022.09.03.13.36.44; Sat, 03 Sep 2022 13:36:45 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@outlook.com header.s=selector1 header.b=KzocTOoZ; arc=fail (body hash mismatch); spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=outlook.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 6A15068BA39; Sat, 3 Sep 2022 23:36:28 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-oln040092065072.outbound.protection.outlook.com [40.92.65.72]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 80AF968B71E for ; Sat, 3 Sep 2022 23:36:21 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=a//iHs8Tjq4Q5yJo1pNkxl90f9zE3kSkjqQCoMqdUfoRJ/ItaIfw3rYuZcXMjAYyw7KLr1tbG1HE43dFUERTRmUB/KmtIWPXwU46r6ow/mkta4xrrvL45olyRO60nUGG+M2IEFaYxwXKWR5nsSNQjavBpfz0xxOteTINXaQdY9KUxtQceBPiLYlMIQM73WApEEhMe2kNXDU3YRlIKSAPXGGZWZr7ohHlfn2WvlbnSbve3m3pd0OdioTVaQNEwR3a8/24RzGuMz+vXmffqKQmhbAi+0x9Mo5imvpg0wAeloAviwUyVS6zc8SsHEcAMtoy8kErzPx5nhtmstz8vwPi4g== 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=/mSDdMNarvSfC3EYm90gD1H8mI8I4yLiUSWz/pDQFfU=; b=Vwei8InwA96ObHs1p/v4Ifg8VXVVc2kpSgvBwN1ycx5HRlQeuBL7N9DQSkpgIfRXY9aq/G1AX9Z3y7m3DH+xucW+aDLPBtHQB6s8RT004EWPYlwiSjYIJ9w4jvgjqlzqv4GC/U8Q93whJHLyTyyhOUElGFxU9ALVnT+fQDifqYOQNUxShw9YVYNJLRJ+kVRf5NHPoKmQqMTx6SYfoD0FNggOijf1at5GcD5GtTwiDSBatjv3UpZELkHZzC7+Ph1P1gZZXOOgFi5+oiS1NjfI7DHlPeEWhW5ns74kZlKZYlj/BBoPFZsnj30IpNSLsUKOKshHRBsp8BKDPK4nW+Q6EA== 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=/mSDdMNarvSfC3EYm90gD1H8mI8I4yLiUSWz/pDQFfU=; b=KzocTOoZS0SFEdFrdrLMFt93A4IURQde0gJyXFa0rkBreWJ35rlvocs12NfGJJ32MW8ofTmSOLr3kzLHFZIV2fc7dfidcptaTES8Ch6Hgz8u0UwsrUS10XbjrqP9t6Moy//kNKCN1DtqGrY6T3IfwInURKvv0F+B/DOc/j9MyVJImYhFooG8wDP5BBO1JaUzNuJUzWgn8K9PJ6vPI+s2iJPrOfasoCFTjatRR1o3Vc46xnuzhucATKGmwDsfLB9SUHsKli7jXuCmS+j5u5EsCI4wuF/5NIk6SZo+2/8rbtLbkvKOspKXnkWjqw22hpsIbfPxeJZwAgekjUARjzbA8A== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by PR3P250MB0241.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:17c::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5588.12; Sat, 3 Sep 2022 20:36:06 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::f85a:9a02:f8be:7c47%2]) with mapi id 15.20.5588.010; Sat, 3 Sep 2022 20:36:06 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Sat, 3 Sep 2022 22:35:59 +0200 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: X-TMN: [ydhMRulL/Fw73/KoKQqZfKcClThpg+1SFv5i5+AoUaQ=] X-ClientProxiedBy: AM6P193CA0086.EURP193.PROD.OUTLOOK.COM (2603:10a6:209:88::27) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <20220903203559.1961353-4-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 57d22a4a-d639-4c69-20e8-08da8debed62 X-MS-Exchange-SLBlob-MailProps: EgT5Wr3QDKxJSTKLhBqHOpZClMCbmhyfgOB5RzCoNGc75e4ubbhcYh0sgKCF844dfwY9I7GdPkrDCX0X7EELcLyR7VrtaqIQJhRzW3vq4U+ZTKjzuE+Zjw5caHQIykwbyvzj8RlnElYi5GlYHtxasU8gntPB5aNydhxdiLre2M3r8zBshJ7df/YEiT3GAynIXdj4IY9X4xmRvKSsWlIDmg+3nKZ8Pv1jHoMk6AI8kRPTCzECDWWtgC5ydW4rMEotfaLy4na5FUclteJZSECm1Y41/ktjDCyutmnT2uOnYUSvNXXk2VMtg+Zye1S92NvGXLh6eCJJBlqyXVXAV6BTPbW+lShUAHhDdC8zNpyfKZifGYjhf2nuHPxl4dCU5MUPvD/8Sj5F3+eRIa3hLNOij1iHsRPENZbZQiIX+Gv3Ly5V1DdIkuxSEP55mPFn31fS5cBWAetmpKPzHknerUarAAcDhqnkwcz4gMYzH4G1FG79wWu9YyylYfKxnakwUnloAGrvqckaQ3scf3XBWDR/Oz0Jw1A6UEKcXDHRBEwmqPihdqjmE9dhhMahFuDNTiIYkNfjHWXdpIDJMluUr9l0tpxqK0V1pfnoPRo0VZjKas9z5rjfFum39UuCZKqXFhOo8tiGGb+kwr+0/S0hL/+8ZvtsfRrA+St6c2X5Y+wagGP0b4+YWflCMGoSH+zP5tvBNcNipnoZY+/agAWc/ZvCcBGutMj0yDheEPa1TM4gcSg= X-MS-TrafficTypeDiagnostic: PR3P250MB0241:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 8uL5JIn+/MOSrgAWzthDivJSgk9Ge46+n62dUVRjxs7GpT87XjmV0pm2CGUS9AXPg5Y41mm4wFG2/hxFgrrRP23XE2iJsC5zxvqMoj5nK7T85hJ3vE3vVXkg0GIGI5qNsTNa4Ofr3JDO71oSDTBPWq82p/M3WBl/eXaFIdGLpMhALfxewRhY2DmVaWswvJPdfwapElKK3h534RFociUyB31s3s6TTM5ueJYdoKF6MlQltLY4GttC9/2PXdrpdi1xCIHg+S0i+eMiXOUs2W7XjCLNEnKGO/TpjRNeeWq/jMVgG++FmG8OD+9HInlAoDVP7B82/t2vtYAGpRW3V9ZoPRGO9zrgKjRSbIe4JjayTGidbKwXqvWKzTGmpMK2vAAJKckZM+BhsAcdWOXBrPevPCnauaYSgaT5r4e+B+VcIbcReK2wtsa1nV3g1k3SxZc0AdEWrujsSY3rndqXdWwkkXjLrxOCt6U1hGIpFfkbuHYJvqLGUfWV+8WGUAYDSs+1sgB60apj7BOhiBxfOSDU7vV9aCXnS4QBtVOkiNE1dkpM4n+13uyNeU7gGEapnhbijpayQImF+69cXMsOyqyKknc28/0RNSplovujqqoRCzDRsyJlESm0uRIaeOcI9G3todk7aqB6vLG8DG8v840reQ== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: DFVFNnZSDW2liJyCerMaWa/8I0159mWzgg3cOB6J3qoMyH7s0tdW3K77JKC6ywo+lyN3iffiZzuQ3usMDLv0pYt/It5g7F9k3DfQpoMoDsZjP75NHXo5HequMWowN40PprGFstczVPjioI8zMHWw8/dS6o9Gg679sRrWZZ5AKgmplxSJFzeTNE6Nc05IkxvkY9VtoKHEXz/dRzn9SfRZf0jA+EPLcNWg1z2arA0V6ib4YzJUnHcx52k8iMP5ctKsWoKlhD372NWDMXZSTDnIOH0BxmGPU6Wn/amb7mtzaeFHhgJsZvgoTgzYUCKu4wuDb3MSknQSBnDhlDyy4mv8TKcp2F3sW9L4R46yv9TmOq5NLvt8sH1V75V+/3dCrwpV6906c19BreJO3cAIlH/wcf6oVl9+icu2e1SBydsPwYYB/JURDba7JMtkYwCFJtwW55j3dalCLrOoSqMCOfcGuUyxsJSDo2V33YxO0p/WDDiwTrK0pQlKxu1Ls2nj/2VtESatK0LhdaXO6sUERmwKPFEhBUa96JV+b72Cbi34dRaNBlH/BxFSIVNfodHRzIMLO2r9z/Yj6B+Tbm56KZxE6jOtV/58+fQZVPl6e7QXJSeRU5Fes7Y0OG/Pasa99ML7idzTiFP1mwSL/mdlWeszJMy3XNrxO+cWBDHPCImbZeCbvJyHGb8V7aC0/k+rOb5zqvtik/v0FWaRZwL+JpiPXGIb6vcbf7ZP7RmEFJVREYGPU+Zxnw3r4NKhNpnQ3HVawNQGF3l+u5tTjMFI2AZsT+4zXobZmD8Fs93PQ2XmsIQGDYItBpN1Myr8jgACFi1p1wNhuEapLZu1JDf8weBLM3/A/3TVbQmGo0tXO+1UVeCjB+f1jqxgxnTRjubebFzcR2ITurm4PLsSdg02GECRMis5H9iw2OIaQdlcOj7sUQH13tZI7SD80hG+cwbaHtiUiO0hR3awclZwexaEc+xyG/VULkJoxt2UmFCJ3MheY+RXAWo3JnNsuGNckHOX+hTd8lHuos5UORgxeg3oX5XXxih/zPkkSQ9ZfWYNwTCKZQ7eS8nGShbvypMiDctkExHADdSlCjuVboPLOcb4gg9hiGE61y9oWHYQGuAenB5SV2jZ+xKHK90KCTHfTroWt3Cwx66gWfhJ6DvHsO1XNWpfUF5SvQJv7f1+mNAbgpCLfF5Bk0l28Ilb5kBbVbsfrenUOk7u0KyyTxXpoe53waq12nA8m+PqrsPD2zZD1EIHBfNpjmYJ8jBfEzzpaiASzpgHWL6IcvpDTnhf6ghPbR++jMHfXYM/okICYgxzip98Y6bFnzGmucM+zMBrgQ2S2puQ X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 57d22a4a-d639-4c69-20e8-08da8debed62 X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Sep 2022 20:36:06.7839 (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: PR3P250MB0241 Subject: [FFmpeg-devel] [PATCH 5/5] avcodec/cfhddata: Reduce stack usage 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 Cc: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: 2YmUH7FDqSfJ Creating CFHD RL VLC tables works by first extending the codes by the sign, followed by creating a VLC, followed by deriving the RL VLC from this VLC (which is then discarded). Extending the codes uses stack arrays. The tables used to initialize the VLC are already sorted from left-to-right in the tree. This means that the corresponding VLC entries are generally also ascending, but not always: Entries from subtables always follow the corresponding main table although it is possible for the right-most node to fit into the main table. This suggests that one can try to use the final destination buffer as scratch buffer for the tables with sign included. Unfortunately it works for neither of the tables if one uses the right-most part of the RL VLC buffer as scratch buffer; using the left-most part of the RL VLC buffer as scratch buffer might work if one traverses the VLC entries from end to start. But it works only for the little RL VLC (table 9), not for table 18. Therefore this patch uses the RL VLC buffer for table 9 as scratch buffer for creating the bigger table 18. Afterwards the left part of the buffer for table 9 is used as scratch buffer to create table 9. This fixes the cfhd part of ticket #9399 (if it is not already fixed). Notice that I do not consider the previous stack usage excessive. Signed-off-by: Andreas Rheinhardt --- I actually regard #9399 as a toolchain issue and not as a reason to pessimize the code for all the other arches/toolchains where it works. libavcodec/cfhddata.c | 47 +++++++++++++++++++++---------------------- 1 file changed, 23 insertions(+), 24 deletions(-) diff --git a/libavcodec/cfhddata.c b/libavcodec/cfhddata.c index efe932dc3b..fd5cc8174e 100644 --- a/libavcodec/cfhddata.c +++ b/libavcodec/cfhddata.c @@ -127,11 +127,8 @@ static const CFHD_RL_ELEM table_18_vlc[NB_VLC_TABLE_18] = { static av_cold int cfhd_init_vlc(CFHD_RL_VLC_ELEM out[], unsigned out_size, const CFHD_RL_ELEM table_vlc[], unsigned table_size, - void *logctx) + CFHD_RL_VLC_ELEM tmp[], void *logctx) { - uint8_t new_cfhd_vlc_len[NB_VLC_TABLE_18 * 2]; - uint16_t new_cfhd_vlc_run[NB_VLC_TABLE_18 * 2]; - int16_t new_cfhd_vlc_level[NB_VLC_TABLE_18 * 2]; VLC vlc; unsigned j; int ret; @@ -139,27 +136,28 @@ static av_cold int cfhd_init_vlc(CFHD_RL_VLC_ELEM out[], unsigned out_size, /** Similar to dv.c, generate signed VLC tables **/ for (unsigned i = j = 0; i < table_size; i++, j++) { - new_cfhd_vlc_len[j] = table_vlc[i].len; - new_cfhd_vlc_run[j] = table_vlc[i].run; - new_cfhd_vlc_level[j] = table_vlc[i].level; + tmp[j].len = table_vlc[i].len; + tmp[j].run = table_vlc[i].run; + tmp[j].level = table_vlc[i].level; /* Don't include the zero level nor escape bits */ if (table_vlc[i].level && table_vlc[i].run) { - new_cfhd_vlc_len[j]++; + tmp[j].len++; j++; - new_cfhd_vlc_len[j] = table_vlc[i].len + 1; - new_cfhd_vlc_run[j] = table_vlc[i].run; - new_cfhd_vlc_level[j] = -table_vlc[i].level; + tmp[j].len = table_vlc[i].len + 1; + tmp[j].run = table_vlc[i].run; + tmp[j].level = -table_vlc[i].level; } } - ret = ff_init_vlc_from_lengths(&vlc, VLC_BITS, j, new_cfhd_vlc_len, - 1, NULL, 0, 0, 0, 0, logctx); + ret = ff_init_vlc_from_lengths(&vlc, VLC_BITS, j, + &tmp[0].len, sizeof(tmp[0]), + NULL, 0, 0, 0, 0, logctx); if (ret < 0) return ret; av_assert0(vlc.table_size == out_size); - for (unsigned i = 0; i < out_size; i++) { + for (unsigned i = out_size; i-- > 0;) { int code = vlc.table[i].sym; int len = vlc.table[i].len; int level, run; @@ -168,8 +166,8 @@ static av_cold int cfhd_init_vlc(CFHD_RL_VLC_ELEM out[], unsigned out_size, run = 0; level = code; } else { - run = new_cfhd_vlc_run[code]; - level = new_cfhd_vlc_level[code]; + run = tmp[code].run; + level = tmp[code].level; } out[i].len = len; out[i].level = level; @@ -184,16 +182,17 @@ av_cold int ff_cfhd_init_vlcs(CFHDContext *s) { int ret; - /* Table 9 */ - ret = cfhd_init_vlc(s->table_9_rl_vlc, FF_ARRAY_ELEMS(s->table_9_rl_vlc), - table_9_vlc, FF_ARRAY_ELEMS(table_9_vlc), - s->avctx); - if (ret < 0) - return ret; - /* Table 18 */ + /* Table 18 - we reuse the unused table_9_rl_vlc as scratch buffer here */ ret = cfhd_init_vlc(s->table_18_rl_vlc, FF_ARRAY_ELEMS(s->table_18_rl_vlc), table_18_vlc, FF_ARRAY_ELEMS(table_18_vlc), - s->avctx); + s->table_9_rl_vlc, s->avctx); + if (ret < 0) + return ret; + /* Table 9 - table_9_rl_vlc itself is used as scratch buffer; it works + * because we are counting down in the final loop */ + ret = cfhd_init_vlc(s->table_9_rl_vlc, FF_ARRAY_ELEMS(s->table_9_rl_vlc), + table_9_vlc, FF_ARRAY_ELEMS(table_9_vlc), + s->table_9_rl_vlc, s->avctx); if (ret < 0) return ret; return 0;