From patchwork Fri Feb 4 15:14:26 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 34118 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6602:2c4e:0:0:0:0 with SMTP id x14csp3491242iov; Fri, 4 Feb 2022 07:14:53 -0800 (PST) X-Google-Smtp-Source: ABdhPJxRFIIPy1Se8FPOr5ZXSSQORF21cN1cpcycUXvUD1ps4fJkGn1Wo8UqVnu5B+q74qjqCp5o X-Received: by 2002:a17:907:ea8:: with SMTP id ho40mr2806607ejc.654.1643987692681; Fri, 04 Feb 2022 07:14:52 -0800 (PST) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id 21si1310984eji.830.2022.02.04.07.14.52; Fri, 04 Feb 2022 07:14:52 -0800 (PST) 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=EEo3OB8a; 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 1B5A768B229; Fri, 4 Feb 2022 17:14:49 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-oln040092064065.outbound.protection.outlook.com [40.92.64.65]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6351A68B229 for ; Fri, 4 Feb 2022 17:14:42 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=oVUa2o12zj7/EpZaDYzTphYeM8dtIvE82xuhHhoKIWl2CUhh6aEIgTCDIaU2YcGL2s3Jvk09HzCIZpqsp6NhIsNK4G009ogMxSo2FmNw2Ihyh68VcejK4juFVC65IObEu8ArBOxWtBmZ/vnF2msLzY2+5Amq9C4z7nDpaL3eImQfI7ykLP3BugUg37R0EcdG8ITQJWiQvFUdFPgyFkYXN94Pv5IFfLCaQ2nCGV7tPmtsorYmLkyJMJ872y7Eip4NR7gkZV/cXPPL57DBlLSYNCeElDckJ2nuXyCxfqNzft0FyRl35hlD3EyVzwlXgpHPkCCKJSyU7sSkbbtqnBsF5g== 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=Ph/Ym2KcV6TrVHFvxIZb3RsBW4nN+JkI26Lq2/6St7o=; b=oDEzpc0jmhHPhdl4R5h94/pDphnGVooDgRFn+nMbfis0OMYoHqcPu2YuLoXgGIq96M4bX7M5sHNoUmMbs1TAFmHomfdTUS5MCSbXYJUnrYTsEuvo7hHRol40cA2ZLFn2xLFZcbql2eqoDASAvWohiiu2lM7ktTEAI2N9evjXRT8F1nsX62ICvXk/CHL2XumeuBA2ZoGFAr32jKTIeDcykwLM9hKAXwp6nRaNFYgLgIOYRIxlxwpvmsdxZbpSuvBGN72BshKTKu2CQaDAfe8B8FubBV1BTrVjOJXtLx2L7pvNemxNU1okmn6v17XlnOLWFJ5lnKOaYKQCeBZuacPpog== 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=Ph/Ym2KcV6TrVHFvxIZb3RsBW4nN+JkI26Lq2/6St7o=; b=EEo3OB8aiDl+xa0pP1/qMo33VOUdbCTNeTjP5EzNuSE9TvMyfTy8FB+8jAPm5PtRjG6nvxxamJJa+Jp6DB6F2DTMPIKHsCh3n3hzXn3PBz6pSuO9kElyaCgoZgIvxRDt8pBXW1kAbp5j1MS9VUbz1mX8TZocKHYPulrudfDCdj16F1ETX7yK5jPJXoCJ3YYwkGcBfD/7lGd9vUTjqxxdiHpajqyUgB9GGVtAXhvXJ6nE6tcF+Dm6CewzyjU5fdwzQ2Taa3Vx2lqDtC2RCTPBJUezgci2V0ndhoetUMfRLEXpPIb59w0cdG2aaSCKHpQcp0brJ5ED5W66OBy//bpw7w== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by DB7PR03MB4282.eurprd03.prod.outlook.com (2603:10a6:10:18::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Fri, 4 Feb 2022 15:14:40 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22%5]) with mapi id 15.20.4951.014; Fri, 4 Feb 2022 15:14:40 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 4 Feb 2022 16:14:26 +0100 Message-ID: X-Mailer: git-send-email 2.32.0 X-TMN: [00z38SkeQhykC3xe18sNH/d4id8Jktm8] X-ClientProxiedBy: AS9PR06CA0224.eurprd06.prod.outlook.com (2603:10a6:20b:45e::21) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: <20220204151432.2148838-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 66ca2ab4-ec36-4985-979c-08d9e7f110be X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiSjtu7sMDnYCqklgc/dtjmmFGVlpQafz+lAsF1MmFcP0DCNfaeTfafWn7P5BAjN5DQHb3KCDqFVbGyzHcBiSk0FnWhim1P4pXXqCejxKfZ9NdMjv7l0bkR4kpGWJ9ud+vdJ3GltgUjho0ZJw3i2LhUgTDg7Oe4TFxj/QoajRE5ld+2cwCuH2WmlED+kZDantcDRbAfsJQcUs1oRx18ocj04Vp7jubyPUdQ410PDHAIlEwrOoiZM2PaRdpBt6x344Ueb+bzFF6TvxDdKtVlC/MwLNCf/QJS4jhu60e2o12dM2Rd/WHp40mtjk2A17wnGOTVqytISrR8dZQ5JGriemJbpL4iFNWahIln3i3W6L+0sVWUojZsbOIJExeG7pE2BAvVEpEq/fG4CyYggb3GaurHYO5TpCAaZodMM3XP4esX0mb2CeCPB8iwQZWcPsmBds+WCLITN0TtA7flrhRPe0vw3+1+DFlXqZnLFAwuP9uCnxzkmvQ3bHo0N+lLPcMY3qchXxZUwxF1cVXP3QTcfC4zfMzX/8gz7SerJoT+GFggzJV8lI9RpgPY+nAlJUnlr9Vmt3cbxxDeIbZ8PHcLpYh4IsEMrbfZfL2E1oEMlHfiPw8ihiklpLWIKj2UXY+PgQbBe+qM3tl7rYIjSRlS8U7pdK3qOm6OX2swv+INaEUNrQnvFc4JWf5iqpD8gnxVgqs7woUVY0PMov1on4+BnVOjTxOo9DcVfg2BU6+Cr4tFi6KpMjmavJhq0TByEaUvfQ84= X-MS-TrafficTypeDiagnostic: DB7PR03MB4282:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: sv/bg7GnUzuxX8/PHNQifjxdBq031Rr2oWD6M+DVMd1mfEdee5Ql9xfz7Qc6MzB1RlXbvKgxbTt6FNECQheT8Zr2kaP1FFK+42VON/lcqXt8e2mV0vMtetByarpZHwXM2srwDEj/udygppd0ItsHGtmp/SnQ1Y9LrOgzDGVCHPvl/tHp9kczkyQF/kGhXlqFeH+BAeZ7gP/8Lfwf4g8Cd7UtjXdi0TQBriu1ggw1+/3Xsw8wRCeMQOiEblgXTe+bm6fsV37K1cA4DWNjpRfTiLFLnw9v4Tl2iogj1CkdBG3klUgGPKI3c9m6rO602t6Kk46q50nImuu8quvCCDmtlivyQ6yk8VHh+GESM0m/GIwOykJwUQRkRVtRXdtg73klZzQ5Fjlwj9J4PDqDNV09KM5mrF1zJNII6Cz4V0i9R07sL8iGfipxqDm3s9WXO0FCW6tOIlPKFvbAj5/DkFAi8akB09rc8ZD3KZzTf/0QER9iqEUcalCHSwJSX2jFamQQHA3U0NnMl4ogAgB8IbSdmujfOAW1YppbHP1Y4AwhV03S5Ce+iwbtApA2MrPNjB1pyihOj/C6C1SWvcI2XhZJfA== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 0o4oQQ/HIUIXuZl7XFBZN+sDVEYcPGzcCKiMFVb+LLCY3Rsa6vtq6RrK4xxqYGrv27WZvuAcUEnyUf2IvUsn6N9nXLRtWVf4o9bX3nKeFD5/dlvsPmfwbTOB5jt+biTvfNEVZyJC2GQ3OJQSUXiaOuu5ZiFk+qTTOEJBdw1ZCqmFwbtwTKh6GBXuMnxf9a9pZhRA8uhCpy7/YBkVaPanIekwKxvRTMic3JIK0IL61fPcO9QMxsIN6t9o8eN72B+RFVXzCAJWVF77KXxiXDvE7IlnvkdTsJzKR9iGgTRDy++Ij46Bikvbbp+ir4SlrjzrLRtiaXI66OkCMFpYUrXI8/wJWgaFC6z4M5L47+qqrlltaYgyx2qoVxKcBDzOkZXJzlA2rqYvIRYQMF0b4sDdCtwnRuBNg4AY5d2KutxEvYCrQf4r6dGnTVCfevhXbky+9+KzrbG6djEKzEGDoxnTGgsDW4v8ufWggggzfHtAP7+TKMUqawKCdQ8lLEMc8aty2QdTp5DNRmM/fSD9NN0xKk3t/9g9iNHvZYobJ9KJ8R0bxrqAcs6euJArxz8GvtYxS71UpWSzVToZTcvSpISKTivRLn/xZXGCM73irExnYOWIInHHhVZa7F+rPeLPP8xkUbgWwG/hvpHx3gpAzLXS1JAALp0d7dAz9Kf6LP0PAc6j3qNWWGCEFJ2uvpT2GBhYSQZ1oYfZklahJiYuRjnBbMFN2KKixvYbfILdviMMeiD0O4YABLv+ORRKnAJ7rEgUV9Vhz2yxFWl5obEc+m4Kxz4pzrcHq+5VtkPnzOKGwF6ZUSjG9V9lBqTOnrCW+0rw0W2RNWkVPeSQz9ZIj42iJnv0z90P+sMaw904FdbuyB2mY7+0idjvGoWnQft1encN630IzB11LFDoQwcRLyiHLsQfiZXupD1+eFbn/eENjC9q0TqsdiAEBkqykSUb9JgY6bwr3MBaXU3+4j3afUHrZQ== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 66ca2ab4-ec36-4985-979c-08d9e7f110be X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Feb 2022 15:14:40.6821 (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: DB7PR03MB4282 Subject: [FFmpeg-devel] [PATCH 1/7] avcodec/cbs_mpeg2: Remove redundant counter 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: bfW0o74GoTWo Use -1 as the position in ff_cbs_insert_unit_data() which implicitly reuses frag->nb_units as the counter. Also switch to a do-while-loop, as it is more natural than a for-loop now that the counter is gone. Signed-off-by: Andreas Rheinhardt --- libavcodec/cbs_mpeg2.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 26400f279f..4395bbf047 100644 --- a/libavcodec/cbs_mpeg2.c +++ b/libavcodec/cbs_mpeg2.c @@ -148,7 +148,8 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, CodedBitstreamUnitType unit_type; uint32_t start_code = -1; size_t unit_size; - int err, i, final = 0; + int err; + int final = 0; start = avpriv_find_start_code(frag->data, frag->data + frag->data_size, &start_code); @@ -157,7 +158,7 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, return AVERROR_INVALIDDATA; } - for (i = 0;; i++) { + do { unit_type = start_code & 0xff; if (start == frag->data + frag->data_size) { @@ -185,16 +186,13 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, final = 1; } - err = ff_cbs_insert_unit_data(frag, i, unit_type, (uint8_t*)start, + err = ff_cbs_insert_unit_data(frag, -1, unit_type, (uint8_t*)start, unit_size, frag->data_ref); if (err < 0) return err; - if (final) - break; - start = end; - } + } while (!final); return 0; } From patchwork Fri Feb 4 15:16:43 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 34119 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6602:2c4e:0:0:0:0 with SMTP id x14csp3493065iov; Fri, 4 Feb 2022 07:17:07 -0800 (PST) X-Google-Smtp-Source: ABdhPJwyS2kEZHd5b+4VK29ajtgxyWQGtGRNbjfZjQloxsZg6bZi5RumLLNaQO8scUNRfbI0+Mtv X-Received: by 2002:a17:907:86a2:: with SMTP id qa34mr2913977ejc.674.1643987827635; Fri, 04 Feb 2022 07:17:07 -0800 (PST) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id hc19si1437023ejc.382.2022.02.04.07.17.06; Fri, 04 Feb 2022 07:17:07 -0800 (PST) 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=Nba10YBr; 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 5426F68B278; Fri, 4 Feb 2022 17:17:04 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-oln040092064071.outbound.protection.outlook.com [40.92.64.71]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 7201C68B229 for ; Fri, 4 Feb 2022 17:16:57 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=T1YhBIunKU3n9nmrh1iSNHjruElbLiz29h7+7AB9Ro6AnsUYSF0iIyvCnI9c8yHqpgCHF6kE8GlRc0A4pt7y94xM7QdPYVRViRKvd+lAUFcFdDv2EMwmwUuM87qYOtVBq9iQmV7M+8sc3z33plf2gE5wEVRPfW2TwV/vVgxNmX5xRilEuBa8w0mHVthhQM6Hlg5JVL18TlzQ5Am4Y6JjWewDdXojqh/eA6jSy/jJurH/zPllUqLxEnUvNFHsDltZMFfseKgUrpDlSQ4GOqrSn+z3i9ssEWQrPUYjR2myZZshSLThDyCevmZPAy0E832a7S2b4IgAfXzzmuWEA8y3hA== 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=DxUF11Q8N1ZTXhPu7xA0vHjEHqkgc8fPZOp6Dal96D0=; b=GZbISh05p/ysRaqcBWtSuO5hy/e/PTDjp0GvZ7zW3hGN/jQf7DfpkNjGMB6Zq8JqSYNxxeSvUwiJEPaSD1wcjMiQq3eYlVfZ3uPYcnGevy3VXmwIDctFVwxN5oAbjMb+DJs2OMHf9am1aDYcBx7rRzsllUo34n+KZDq+I8ZX5eTqWWnvge+aGvYPsLJZfL26NxuG/DDPE4ga8KMcXAF8noWKk1YQwn4sRuse00VM8OoIjHoqVJtVFc7rtGznS3ARr8F2Z3vic/SrHe2A9A5n/i7BqtmSkhC7yBo1i2t/eugSbOgxhpPaeC8qbbzawqF52THohg3xD6DnFLfyCBgCjg== 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=DxUF11Q8N1ZTXhPu7xA0vHjEHqkgc8fPZOp6Dal96D0=; b=Nba10YBrYXcMoM4hNBM1SQuLqn3nHp7uCMFME+j9H63nrmWxxsXR+FE23RBcNNrVIPbk/9jzKg8tC/btGso30/vx9l8v31KZobSLFGhcFl1xfkW2EAHNeWMSclap+I16JIsEG6HhZ+4lQVWo3jpA7Q5W+HwlqXLkXgmzXXTrqi6nTPeWkCC35dz109j9ttyPucCyszwOTItowHuDW3t8PRZqPXzuktu/F+nITgZdTG9S/rPjBblIrTNMwOe4w1XhAtuMFes0ioAl6+g0PxRDcT++62tFmiiWvpX2cZcwmeAXqWvYVbFz7P/lh+PM9kSU0MAhACuBu0CJx7zbaaX3iA== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by DB7PR03MB4282.eurprd03.prod.outlook.com (2603:10a6:10:18::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Fri, 4 Feb 2022 15:16:56 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22%5]) with mapi id 15.20.4951.014; Fri, 4 Feb 2022 15:16:55 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 4 Feb 2022 16:16:43 +0100 Message-ID: X-Mailer: git-send-email 2.32.0 In-Reply-To: References: X-TMN: [0DpCaHQ+qV4DkNyboiTiZtiMuldoYeyG] X-ClientProxiedBy: AM5PR0101CA0032.eurprd01.prod.exchangelabs.com (2603:10a6:206:16::45) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: <20220204151648.2148917-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 4307dbd6-a0a0-4276-8d91-08d9e7f16170 X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiSjtu7sMDnYCqklgc/dtjmmFGVlpQafz+mp/J5B63NdXJ/xdyH2KRbMkq+ZJfV3Ezo6S2XfvlPDwm+YyjP2nUPrviRMTP2mbsfJUUGa/75dTeTNzO1Pi/EajiAQ21hN/VNEkO8xp+zd0WAHR6TFcM8RpySYrbaFf8m/tPc8M56rV3xuhaO1Dq2VT0dR+akDv5b7f7NeZDhFlOPC3Gp6SoFZNBZK7QpBlVgwCHmz2lXPB9V548bVjAxZLfb28hEFk1i2Y+04q8a5jxf6lM+6cHPCQq2muDKuPIjvWNklAAkwNu957uFQioKDkA6fdUup+jLk8u6hIU66FfnsvvzcxvsGeC1nBBGoOWmjhNx7tQNYoBp85FbkIpI5aeMEd8CnifmyCVoz5Ie4u7tMXcQQEFov0M3/G/JOT40dTyngP6q1OGdm1zN6bmdyQxYb6c1RKsTtc3DOeedEoUHLq2CGXUPxtJinwZeEmAzO0PAU1J1X4MPJtSNTS9kfujAvwcwUZMBv4VpNyp7d/FiHx5eBx6JQYeJsdEFjuaUL80h4ZpYsTHuUwwwvTWD/ggtVKZJQ89wPJFLaWkyH93wsf9C210qb9QKBZSwxLWk/bNOW/pUPp6vvcwQEZtoGzCo6ZtSOXNl/Rkf84Y0l3q0g5/1kJkMv+gbggFkF5ERGEXp6w02fvYAoqOCT8rEOgv7CW5Z6dOOiOwvS56gzZWkpMFGDsuHXIi9micNca62MHJRoxlrOPMDjRD9eSiFvAqdhafSV6t4= X-MS-TrafficTypeDiagnostic: DB7PR03MB4282:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: f3OsfSX3qjUH8SpvjKAsyxTqANVxGYqP4YQSI4rqF1O6uCa+AX23395gOXFj/o0mEOPe3b+rEq7CHRlvOYCU+7IxV0KD28fSo0aTYY1jnT6fCoCrhSvau+lVlr9eh3LKvr+xWBINLY/c1vBxMiHzHpTw6GEYhayiInW8HGXaTJxOrNL1qIwVXeXJ39cG0cdhxoYvVFzsfMsmc+sd6XnEnWFcwYbvyC6CC8+H8IaQXFD77FE0tElIt9D6XI0QEdWEGM2R/MMwd4Qb31Srjuq/5mX0/0O1+GEmUmzkkBPldCLZ4Hq2JFSRaMgAtgsrK3SdXUJGIm10wVobs0pHkyxGsukp5TJftIa5K6WuJYsMmeIeLy+6CtLhp+Z1aY6IY7Jm07X+cEcHgk0xGPBabVSugKsFBkbBk2jM9Iktp6BGidARNogHAdJ9MmdZqvPy7BKbrqLO9DdE5kbut7GQwop2xzef+rOgGkMr+avc121M5Ret6J+SYywWb/xK/doEukuWwgyKB7MDdN/8s4f6PHx2Ec2zdZhewNAOIngvpb5/ZoCzy5n2CYOEQGaOiv1/PSMZqWepAZ3lqgRPcNMUOjvHBg== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: jPqhRpot3+YY/ozdJKk0npUaiOwdFqYmUsDIBOhECPrwDtCLTpdJ/83cIUx+3g2Qkqkv3plBl0FRj6s0R0eAmDUoqP5iuhPJGLEuPL+L7v/jdXibR7PHc1tTiX/UtBa11681ytlngzCj5XcNWpq+mFLJTrIiZdAoyTD74I40WhOM/tFTw/tdVBuVMCYcMwpFVT9mnXxbHBHiMDFVS+jGFPHCJHA9FuEDTnsq9j+o7D0175Q4nd3brejxSFDgeXRecJDJrpGaHngUNzZdluHnKaf5/3FSoEqkB/HB5n8/vqz2AEpuTiJD7XitCAMRgKWsWecb/wQ13ALDK0L9zWhRYZvvOQ1C36eXm7YoIC+BytAIrVccZG3N9RMRVWw5P5otp7QfHJLaZZHR0cVLFVN0q7iqJPX1j4tvt7Zti7GS3SMw3YwuG6ocM+wM9t/o4wWA4HEyUY4wPc2bjCq1U6ZPTCCJ0H8aJD+7sWQRfZly7H4aJy77UTh8fuwVKivk34SQEi2NkWxC/jNVBUV/VpgbwSlTRhIwIBN12gmKtDA3LZoKmju2JZi8rFZdLGFFZzQ7tOmNLoMx/SqtTEZCyLpgATzNrD9Uh581oaxk8b688+uVZbvvl1vWR8XEDP6EMEmkHnA5ySNREC5T5CLn6s8FaUAqI+u8y0ZJaikWIs1Lws9OOBi2L5lG7PWqO/ebPPOARekanitnig5UNUKdf8kyEQHw8KMJp4JyBrMeaBjz0hn3ydl6WukH+iJVRCarnqQNe6y56CavwaVdVQl8lVcM/HZH+u6fG6KuRWuxwdwWSLReSot8MdIP9d5qkr/r3eVcv3NkNQHl6HPPIbKiSJvUChR+ynYG48Ala2/DHFlEa7MGLgSoyHT4nYDMCyQ7EYaUOrTDGamaLyMrEClSwwjBmyKbIN8TT3fPSYV/WzqVnOkAIcw1lwL44uki86z0gw0llYyx6QiIhS5wAZcXXy4DlA== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4307dbd6-a0a0-4276-8d91-08d9e7f16170 X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Feb 2022 15:16:55.9567 (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: DB7PR03MB4282 Subject: [FFmpeg-devel] [PATCH 2/7] avcodec/cbs_jpeg: Remove redundant counter 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: yjCpmckY7Uap Use -1 as the position in ff_cbs_insert_unit_data() which implicitly reuses frag->nb_units as the counter. Also switch to a do-while-loop, as it is more natural than a for-loop now that the counter is gone. Signed-off-by: Andreas Rheinhardt --- libavcodec/cbs_jpeg.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/libavcodec/cbs_jpeg.c b/libavcodec/cbs_jpeg.c index 7d3e10fcc8..ae263ba038 100644 --- a/libavcodec/cbs_jpeg.c +++ b/libavcodec/cbs_jpeg.c @@ -110,7 +110,7 @@ static int cbs_jpeg_split_fragment(CodedBitstreamContext *ctx, AVBufferRef *data_ref; uint8_t *data; size_t data_size; - int unit, start, end, marker, next_start, next_marker; + int start, end, marker, next_start, next_marker; int err, i, j, length; if (frag->data_size < 4) { @@ -144,7 +144,7 @@ static int cbs_jpeg_split_fragment(CodedBitstreamContext *ctx, marker = frag->data[i]; start = i + 1; - for (unit = 0;; unit++) { + do { if (marker == JPEG_MARKER_EOI) { break; } else if (marker == JPEG_MARKER_SOS) { @@ -226,16 +226,14 @@ static int cbs_jpeg_split_fragment(CodedBitstreamContext *ctx, data_ref = frag->data_ref; } - err = ff_cbs_insert_unit_data(frag, unit, marker, + err = ff_cbs_insert_unit_data(frag, -1, marker, data, data_size, data_ref); if (err < 0) return err; - if (next_marker == -1) - break; marker = next_marker; start = next_start; - } + } while (next_marker != -1); return 0; } From patchwork Fri Feb 4 15:16:44 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 34121 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6602:2c4e:0:0:0:0 with SMTP id x14csp3493340iov; Fri, 4 Feb 2022 07:17:28 -0800 (PST) X-Google-Smtp-Source: ABdhPJyo0J0Ou9VYJ4sMVYJ9OYDog7sdODCFAVZHjZquxd4ksHJj1XCNUUuKWP6AUi2X3OSGPYoy X-Received: by 2002:a17:907:2d25:: with SMTP id gs37mr2961260ejc.102.1643987847839; Fri, 04 Feb 2022 07:17:27 -0800 (PST) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id u20si1334123eju.808.2022.02.04.07.17.27; Fri, 04 Feb 2022 07:17:27 -0800 (PST) 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=lJituHxL; 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 63EA468B229; Fri, 4 Feb 2022 17:17:09 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-oln040092064071.outbound.protection.outlook.com [40.92.64.71]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id AF07668B286 for ; Fri, 4 Feb 2022 17:17:02 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=L9XFuKEa6MfZKAPpAQXdQXmZMw0MCnrlA61Idi5clM0FrcRDEum8mX1MNtH8bGqEQ60xxmKkvsXZ58z4g+Dch73wv1R47nGkxtwGOwmV0maxsdBVIxbYXF27LJCLB5LPFk1YI2Hi6qkDspPs+fAvDwra8Eci0Chv8mhT+g1WW2eaZaPKW0Bep3a61WnF9RBuXbCH+00kbD77w2WT1wLmX5VsB7PEIyO+ud24q+qQJTk+C3aJKWkMtZQgV6qOZ2OhrwwnwYnQx5zSGX7DnqKU/Qum2HnoUwA4DHtqd/xXWviYuRimV7/3DfydBezhDsQV2th06rqNI90ergDAtvJc1A== 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=chf81v6Clkn7Ml/X9T2IwCVGXGLsNe4swvEjux4Q6o0=; b=eEM7L3UyMgV/r6G9h3862wIgy2bV2FfoHndejOLPPI2B/Brg16jTwUtKVJZlSaU4d0ltynvugniu/zCuznpDqKycqQF1x9Bzqi00ebiNaYNSfloEIu/XJKxw4Bj6Vi7vUedzmP542vRl2qGXaU+uOrf3eioEB+nmJAh86HD5+HKtN3TmxGQp8ElbC5gRIUQFAF8GrY0AmokkYEkv8ArZyO8+k0lLV6qdejTdkYWQ98TVSbmb7FoNgWjqAZHLdN45d27cfLeQqVk+ns46YlZJlrgK122cW1fV0rdnHSYW0IU30ix6C0AuTD5rK6QN88dKhAg47mOaovLXSqhob7jI1A== 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=chf81v6Clkn7Ml/X9T2IwCVGXGLsNe4swvEjux4Q6o0=; b=lJituHxLA3wr2gjgscPLuiWA98w9q+1zo4v1H8y6t4/hxtBxE6cwbbg1WVhLLvt2aCAQrYGBiyEdvTZy5Fc5oh+pU4mm2CWxm399bTFgtXtGqSiuWMQ9cW0TWbCh5GAbD6vGVGOyscXCBMNIo6VMb4L63+Ux2tSP1NZYuZVk3cnIG/ya48n8mbThmJlyltW62mqdlcRicagF3QSUZ+UID3GtPtZ2T7JeNR4kyzRjfH7i2JV+lwIFJs/rhNg9aNT3Qp4vWclC9D8W0ln2RlO/z4KLcIFHsH5tTVzV0J7Xx4a6T7Ft2/Ha5+TMfr8p9+14TuDi0/+lzgKvKrLf4k5uiQ== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by DB7PR03MB4282.eurprd03.prod.outlook.com (2603:10a6:10:18::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Fri, 4 Feb 2022 15:16:57 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22%5]) with mapi id 15.20.4951.014; Fri, 4 Feb 2022 15:16:57 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 4 Feb 2022 16:16:44 +0100 Message-ID: X-Mailer: git-send-email 2.32.0 In-Reply-To: References: X-TMN: [XK2kx56ueLg7GhKyFyN8zW3CU+RePK43] X-ClientProxiedBy: AM5PR0101CA0032.eurprd01.prod.exchangelabs.com (2603:10a6:206:16::45) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: <20220204151648.2148917-2-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: c1eef3e7-23c9-4856-46b1-08d9e7f16266 X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiSjtu7sMDnYCqklgc/dtjmmspiwpx5PoZymC/aRnDSAtvh92fRb35ElQcmtG2Ppbmxg+oVZMO7EuaMXZkHKnMr1xF5A4Hqa1XbjnnPKEpD2Tohq5hy4qVgjIdl3rP0CDzo/lEguIJDKERhJM5cbosHlvzUigCZjvGU1P75dt8jOU6ZJJKPcG3UmO6fS86/r0z2u6QqnEK6lHm1iYnpT/WEQH/waAT7rVy7xk6L7Sw0rUj1a4bkegoWA5EbfV/KFw077UiSlKuD5eMFdBbYGUYYnCPePiINz/W8EXrBzAQBbySpTB+cKoFN3/w9Sj8HQ01RBmg+EsKC+qVXErLNekmzjon6JX9T/UvazozYXl+9aJMwNVCfzr9/+PbDeuk3ZYtCl9oBP67m5ohvQBQHx064YUVVotEv0rQcy14USy87p1T6eHOapXCtaqD8Z1uir8w9Ky/tACkQH/FnJNDWV8eo46HP9N7b1T/h5kiTgH0xv6gNZvxcCnf9JnGQiYp6f20LZ/MT3gXIEDcB91qI8aGwiaiR5W4WfFkpDybFznKxLBJqlxfP9RJ9cPrVnwsCkBF1rc+oqJhryx/Xr3WjSxrlu/qZSB+l3rSOBGvHUyf/DB0vEVuUDUrMg8nwnnFDwrhzqMIF0wMCJP6MWy6oIgvrBoo2JMjeKq5+npFbz1WNggfCq6B5LyPyERasihKK8DPwA7BkRbknz2GXveCd3NIQ0cVbS2IuN7jdVB38+8XIQh6YDjQhnWfbjKy1xK9ZNv7k= X-MS-TrafficTypeDiagnostic: DB7PR03MB4282:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: xHbrk+eGR8Ps0rWg1UZWRv1cl5QGKUSnWIhRYfuGxEQm6hIf387/xhG00j/rmwr9z47ETB39RPYK0zL0ASUcp7TqG+DozFPJzEbLPIk3s8gd11BRLjQBRnWR/OnPMKxjhFCQcYp/aXbcUvv5l4FP99gqKtrgTvnZoHsVVXZ594YN/Vx3RXAqo3Rj6wAvrrTULo+dPv8/ASoWnm1wHzdwbq43253JmlzhzmZaWjLevgCe4W8Secgd3mL2uAikI4KKJhK69Y9HZbf8L+d7i1eFv0OAfspfHYEXYCWm4rblnHXkOWe2d6o4WFps1hQqD/bj1V4FcwGiorN4hjKrnFTs0kF0Bsn1jP/R4OLyWwXvoyzz5TZd/F5xgUbJNzGQ8FZ3SY4NYk2qy8NliER3Fk/xzVwAhYzBaRxs6HdUucF7GCDo1Ouev1yLuwgM9IJsLBvp8+8ALN72gUeIe7upf46vhfTLPl+r/8CvagM21TruKqDyj2IJIQl5gI/DuzVTijn10WsK5PWhVSQZh83DYPCJpb4MQs10WhFiDxE3t2zKNkN8BWknqtyuY2rahkZnsO296A0P2N1X3K3l8ptPy0SwfA== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: DPILEGYOy2v8xBAdUlnrcKHbAoKScDT+E5E7/WUYU0VhPpF8TbyjdJfr0t37jwto9FbS3VIH1IL76pkKvZ4ZgTID3ehVfxKASJ+YfVTRGLKFcCzkSq9thOpMIPTtEq0KWbAtZbsDFL5m/dB+HBRGSVpEw5Vgf65tFzgJAEmIS9VjbcUiVkBA4qMDiP/uDr6Upf4/LfvAeRiQFN83DMZoxxixWt6TLWV9GBaikdXI8GONB7paiCgv2AQesjLo5XfbP7zpjuzwy5CuLG9CRspgENwCPh4uxOeOqGmJHZ+wKFHckyzEbj1RBdgXpHeD9kjeBF2ygBxhAZSTZGtNapdm3dowr86CeIVEnGGrplIO2HeZ6TES9Y6ILukLUJFYg4+FTo1bRY4Jix6HWmn5Sep78aIPRLF4/Cky1tfGUiNTV2wXRwDTJg+PIw4BQe+WRdGmLROqxV3wrU0h+P3VRkn++UDIehLxnsQ7hlAVAYD8Uyw0gtYtjHGQ1qhE/RwWTPNcgm4fHhFGJ93xtdYSVXMqqzTqUJPFJt9DB1iM5Li02f+KhIFDbSoNcIC946+rWKjUNjFzMA8r7Gf7MIuw0Q6ezBwnieb7P8IeeJLuR37rNoJqspG83L/+OosQXB0HAhdm/ZFp7MHNlBkjVNbrh07qS6eLSlVlXf58f48nccQGjunyQFZiSuEHcD8agZkUg4a+EZ+HprEomOQR5qUd3XeBz77BYwTrk9UKLXuWH8bFwcIiLzrKNeueLLcE8HYWdeaa48Ip9RIvqflQRiPI3WTiFtXtloQfqr4hmkq+y6KC/tm99Ac5VOLlFe3uZqDGHJu6SXACJGDfELXsCEvdmiWqan/06JanQrzx3DL2Fg+nXpSycDQljFQsFZ4Su+oQuIUIn6WmJKJ14Fq2xxLK1UQyb8CfRbNfKECi4VNuvmNp4e5XX6k/lBp8ZtxDLVVWirfcuEexDTbaPmLBBr4HijptHQ== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: c1eef3e7-23c9-4856-46b1-08d9e7f16266 X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Feb 2022 15:16:57.5671 (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: DB7PR03MB4282 Subject: [FFmpeg-devel] [PATCH 3/7] avcodec/cbs: Make ff_cbs_insert_unit_data() always append the new unit 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: TW0oucOH1Cnm All split functions (the only users of this function) only append units. Signed-off-by: Andreas Rheinhardt --- libavcodec/cbs.c | 22 +++++++++++++++------- libavcodec/cbs.h | 5 ++--- libavcodec/cbs_av1.c | 2 +- libavcodec/cbs_h2645.c | 2 +- libavcodec/cbs_jpeg.c | 2 +- libavcodec/cbs_mpeg2.c | 2 +- libavcodec/cbs_vp9.c | 4 ++-- 7 files changed, 23 insertions(+), 16 deletions(-) diff --git a/libavcodec/cbs.c b/libavcodec/cbs.c index d7aa67c3af..e829caa0a0 100644 --- a/libavcodec/cbs.c +++ b/libavcodec/cbs.c @@ -789,18 +789,16 @@ int ff_cbs_insert_unit_content(CodedBitstreamFragment *frag, return 0; } -int ff_cbs_insert_unit_data(CodedBitstreamFragment *frag, - int position, - CodedBitstreamUnitType type, - uint8_t *data, size_t data_size, - AVBufferRef *data_buf) +static int cbs_insert_unit_data(CodedBitstreamFragment *frag, + CodedBitstreamUnitType type, + uint8_t *data, size_t data_size, + AVBufferRef *data_buf, + int position) { CodedBitstreamUnit *unit; AVBufferRef *data_ref; int err; - if (position == -1) - position = frag->nb_units; av_assert0(position >= 0 && position <= frag->nb_units); if (data_buf) @@ -828,6 +826,16 @@ int ff_cbs_insert_unit_data(CodedBitstreamFragment *frag, return 0; } +int ff_cbs_append_unit_data(CodedBitstreamFragment *frag, + CodedBitstreamUnitType type, + uint8_t *data, size_t data_size, + AVBufferRef *data_buf) +{ + return cbs_insert_unit_data(frag, type, + data, data_size, data_buf, + frag->nb_units); +} + void ff_cbs_delete_unit(CodedBitstreamFragment *frag, int position) { diff --git a/libavcodec/cbs.h b/libavcodec/cbs.h index 87ea14cd07..5583063b5e 100644 --- a/libavcodec/cbs.h +++ b/libavcodec/cbs.h @@ -393,14 +393,13 @@ int ff_cbs_insert_unit_content(CodedBitstreamFragment *frag, AVBufferRef *content_buf); /** - * Insert a new unit into a fragment with the given data bitstream. + * Add a new unit to a fragment with the given data bitstream. * * If data_buf is not supplied then data must have been allocated with * av_malloc() and will on success become owned by the unit after this * call or freed on error. */ -int ff_cbs_insert_unit_data(CodedBitstreamFragment *frag, - int position, +int ff_cbs_append_unit_data(CodedBitstreamFragment *frag, CodedBitstreamUnitType type, uint8_t *data, size_t data_size, AVBufferRef *data_buf); diff --git a/libavcodec/cbs_av1.c b/libavcodec/cbs_av1.c index 302e1f38f5..04314443de 100644 --- a/libavcodec/cbs_av1.c +++ b/libavcodec/cbs_av1.c @@ -828,7 +828,7 @@ static int cbs_av1_split_fragment(CodedBitstreamContext *ctx, goto fail; } - err = ff_cbs_insert_unit_data(frag, -1, header.obu_type, + err = ff_cbs_append_unit_data(frag, header.obu_type, data, obu_length, frag->data_ref); if (err < 0) goto fail; diff --git a/libavcodec/cbs_h2645.c b/libavcodec/cbs_h2645.c index 3396c059c9..10b3bcc70b 100644 --- a/libavcodec/cbs_h2645.c +++ b/libavcodec/cbs_h2645.c @@ -493,7 +493,7 @@ static int cbs_h2645_fragment_add_nals(CodedBitstreamContext *ctx, ref = (nal->data == nal->raw_data) ? frag->data_ref : packet->rbsp.rbsp_buffer_ref; - err = ff_cbs_insert_unit_data(frag, -1, nal->type, + err = ff_cbs_append_unit_data(frag, nal->type, (uint8_t*)nal->data, size, ref); if (err < 0) return err; diff --git a/libavcodec/cbs_jpeg.c b/libavcodec/cbs_jpeg.c index ae263ba038..da7ee808cf 100644 --- a/libavcodec/cbs_jpeg.c +++ b/libavcodec/cbs_jpeg.c @@ -226,7 +226,7 @@ static int cbs_jpeg_split_fragment(CodedBitstreamContext *ctx, data_ref = frag->data_ref; } - err = ff_cbs_insert_unit_data(frag, -1, marker, + err = ff_cbs_append_unit_data(frag, marker, data, data_size, data_ref); if (err < 0) return err; diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 4395bbf047..2bf38c6001 100644 --- a/libavcodec/cbs_mpeg2.c +++ b/libavcodec/cbs_mpeg2.c @@ -186,7 +186,7 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, final = 1; } - err = ff_cbs_insert_unit_data(frag, -1, unit_type, (uint8_t*)start, + err = ff_cbs_append_unit_data(frag, unit_type, (uint8_t*)start, unit_size, frag->data_ref); if (err < 0) return err; diff --git a/libavcodec/cbs_vp9.c b/libavcodec/cbs_vp9.c index e0b8c02ac2..333d70ba4e 100644 --- a/libavcodec/cbs_vp9.c +++ b/libavcodec/cbs_vp9.c @@ -451,7 +451,7 @@ static int cbs_vp9_split_fragment(CodedBitstreamContext *ctx, return AVERROR_INVALIDDATA; } - err = ff_cbs_insert_unit_data(frag, -1, 0, + err = ff_cbs_append_unit_data(frag, 0, frag->data + pos, sfi.frame_sizes[i], frag->data_ref); @@ -469,7 +469,7 @@ static int cbs_vp9_split_fragment(CodedBitstreamContext *ctx, return 0; } else { - err = ff_cbs_insert_unit_data(frag, -1, 0, + err = ff_cbs_append_unit_data(frag, 0, frag->data, frag->data_size, frag->data_ref); if (err < 0) From patchwork Fri Feb 4 15:16:45 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 34120 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6602:2c4e:0:0:0:0 with SMTP id x14csp3493203iov; Fri, 4 Feb 2022 07:17:16 -0800 (PST) X-Google-Smtp-Source: ABdhPJwCfNZgRLsoFItGUAUHzf6QecAvZRp9N+J4FXKgdSGrvYbOvdT3KVSnOOKEhbc8j/NImUqw X-Received: by 2002:a17:906:c10f:: with SMTP id do15mr2937560ejc.474.1643987836630; Fri, 04 Feb 2022 07:17:16 -0800 (PST) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id i10si1523366ejd.885.2022.02.04.07.17.16; Fri, 04 Feb 2022 07:17:16 -0800 (PST) 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=eldfPeNe; 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 4F73268B280; Fri, 4 Feb 2022 17:17:07 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2060.outbound.protection.outlook.com [40.92.90.60]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 3DE3468B274 for ; Fri, 4 Feb 2022 17:17:00 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=HaVrbDkv/+SFh0LJ4NXT3FDMQsUpEuYBrsB2+LMp7jXxfE7nAPv67XqXTBEibBr3I9RYlicdNjLp/OefYWQcF27bzm3v3/gJoF8whLHaN1p6wJC2pewQD2uBCbNVzGL/dnmwliIrw3V7VIrirhQ61EWPqVL3LU+LJyEjDaliyPWzPeEhY/93YbAFBTa380TvovuWtn4rI3W9aLdKthEm92/9YzJGZYtSbGiCy9O8HG7qBzFi+QKiHmc1izwszysNwhycCkmfE/y/XtRAnmBp7WpYXfIc5UdwoW5qDGYQ4YSZ9RQImqB4hCf7hu5HaHn8z47aryj+7tsud5AvxSJSCA== 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=Zu/rd6TbmSv/02Gtu0YBT7Yd/AAB9RxF8xkFoh1MmCY=; b=lPTDXH3hhM5Zu6pPVR2HkGaabWCRkXmpW452HJVBTQIxP6hzmFbsSveAA5Rk3XJKcFAg04keFd41tcpxNSFhQkNcX2JdUo3SUtR7BaygIuC4cS0aWJlpn8HnZhP9j5YO/cXXm8QRf5n3LqmuKbVgmjmaURTzG7SZfvAO7kWlRj7q4v2aW7Ke0P98AUXNo5O4e4x+koYtzBe2s4JOikaxqKoGvM5awW3dis+iHGw+BuMCVxqLb7Vaps6KSF2YAXcXeC2F5atc63ae0xpgUTch7zyKUAz7hqvZRXor+a0ul1A6yXDOAwIfKyztLduAcvFQGWuEA3TcdzqpE5HWIEdtTA== 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=Zu/rd6TbmSv/02Gtu0YBT7Yd/AAB9RxF8xkFoh1MmCY=; b=eldfPeNeTgyfOHEwIWt6jDwxC0y/mb8/Mk4yRX8s4Q1eflJht5lZHqrfa02huFZPgClgXoFdXo/pq/YwyJrXVlP4Oyqw7vw3d1KsJhXalO/oruHE+VKMNja6boiczVHWL+WhxiLtdH6Pj1ZxyerBhq1SgK0BWRD68DgOVBNmrp1eOo3EV1k6EvqpeQzzYD4P099BjKOn2ddBgA6j8td1Yy83fTVNxaVnRR9YTkOkg/8H3cgFfZYnUxnvNMvcUckm0FQhujFv6I2kRq9bwrayoxQPHITDqjV+V6sJya3eHMWXbo4MVXdkB5T4hvoTdRHKU8UDF8DCGtkc+YodOaw59g== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by VI1PR03MB4703.eurprd03.prod.outlook.com (2603:10a6:803:5d::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Fri, 4 Feb 2022 15:16:58 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22%5]) with mapi id 15.20.4951.014; Fri, 4 Feb 2022 15:16:58 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 4 Feb 2022 16:16:45 +0100 Message-ID: X-Mailer: git-send-email 2.32.0 In-Reply-To: References: X-TMN: [YWvRYqq+IlUmJ3bDEZ7p30oIALPejjYV] X-ClientProxiedBy: AM5PR0101CA0032.eurprd01.prod.exchangelabs.com (2603:10a6:206:16::45) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: <20220204151648.2148917-3-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 2f64b249-2979-4f8d-ab1f-08d9e7f162fc X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiSjtu7sMDnYCqklgc/dtjmmeeI6J50wBVBJueMvqx6KlW+nCNcBSCFuZtdi1VPWkf4OZcj5TBrDxWEYEC0RfuInodbd9U/z0LTEB6lTE6LDgrjm26jdPquiF57WzADRemoBXWhB7Qk7WzN5LVT5yHJmv0gkHh3u2Pi8kIfIsrj86zS63/fSrz+ejhnV34wrl+6oXyKc+W7RrEZ/t4OjqKxN2gaSXazauDaYrNDldLJWCgCGtiZ9e8FXIEF3XPOnf1Ycz8Enc0wQrP+kGGupdhYPoglpS2mdjkHCa2bI6dthQ3tIhOQ6Bt5vrtoqzTuSOKQdBnANTXEV4E5ZFvyXtDIPqow1XXzH2xFeUOBm891k3p7TzSW76DGVPhPMElaJyRQRiofxZApMFA6P3m3qi2Sd1zQ8QtbHK6PLuu/vJt2/nFJ7LnjImjNfvKK9Q9eP6LNxnKy37yDsWwNFqJGWiwZzSd2MLAft6XOtAzL5Jxabkn0M5oqnwn29Wh+iqYXlZOETeWeTtNsAvSKWRuGt8XgWE6kt0nuPGQsdf+9isaleOOtSsfJRYUy6HC7WZf1Ob7V01wyeOtGSv+UsdGTysITk0H8YDiq0pAIB4lLhfCA78CnuBWPK44KE3sgGZCgXyxZ/hlF8ZQul0bGeblBAKzBaIcFqc389lH6x7tvYodZsbNGP31C9xK9xek8F92lHe3kX/YfxfoC8db7yk4R0lxoa6yM/Kpfvj1PSEOzU1VjiRbBIUCeaoDPEpEWXci+d8Sk= X-MS-TrafficTypeDiagnostic: VI1PR03MB4703:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 9/aQpsBGrzh2uiZ4gD+VxNqXq0UcrLVOGZLqE3RFJDDk7yiyE1ndTFLzhdIa+S9RXPh2zEaN/02kQzYcUK3kUTA33CQPAFEWTaiAjnDEsdF9U33qzkDBffT0g8IdmRfd9yLIQ9SdoZhqZMQlxR6TGptXJ3Dxsk36CqYGQDnBAR6XM7MnJJB9uHq3jJLH0/JP6sKsrIwyvMP576eSLxM8jTS/Cl7x2/NxQew7uRENdL3JU0vBbqoDfwyHuEYuheIZq+Gp524OoVyGkioHhWCkO5++vvytRsuGeHAZByhPlA9ydJPnlvRXqju1PE4eRYkrJ+RdlTjTZf4vw77sT9SH9TdQG3IIoPMW0XVv7uXnujRkAMOn0mmUrDssicFTZEWTu3k6oBluqfn0fCQOD2mAT4Gu1Xw/zxDOEkBgVtWn0wUkWFgJpU4EgpAhdVF8XvTn4X+AQJ/gBDiSc95oVyNV302ska7nJHUgizgLkNVM2x+1tH8E37Mf9GKGRRStq1QeTNAPMbpiFwm2KQE4Tczw6eBzEA+Upz9A57v3zgv3DaS/7Cc5TecfvEy9fyIOCss0bgQju7w07Jx2wcPaQeR5ng== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: elbuo3cZ91yrmesRgNkLVo9kKS6xcf8HQ/rrmN8oGjoB7X2oBSP+3SXD+3/RbK30vNL2RWsWfTRSMklOoDblXUy9LRy9/kTzeHwFBuMIFNXhjC5vODks4DLG+AftEAy77ZJYjHbvKCPq50yxi/MB7EiBHqELTZBeY/+Cp+7dowVtUmWl0jDafwUP3KfqKzhfiSaMSPbRNeT4ID/XJxxCZAvk7yi6vH6dFB0Y7rndc5EceNCp+E0srAacsw+lepwKJjj6ydLdv51j2gkU2CbFlBF4UA5JJqDP/IpXmmQcNbowxjwB9LgXZmL6JsPo2cbsckJrENs99arheQv2m813InCwgkQusggFYry4+4Q+N1hOA86peseid1fQVgiMJQwal+de4Do7zatssprvWO7lzlRtjpfU51ii/3qLLBqaizhqvqkkPZOc+zhaFUzcmZB4Gow7v5kTKxnWnJf73pwTKQ167cBJmSoSOP4re0wGhP3BELi0PIqLmsIhOPDUHfagBfaz07wGbWYOyGsDjieu9GF9guNj+mk1fhbXcm7HU6fkvYUuqMLBHD/0YuUiL99lqcmZnc73vsYAgTVOx948+cg8/c7PIOswmJWtugZgkyjAh7L1HDMxmVf4PTzt3MPMYq52optpjCoQvvda3Ca0KZh4lnEr3MNrccrlNXJu0ipVXpHA0o73YMdNZOUZ3AqU9pQHiaBrqCbLDRBm9BX/8ftcPnvncayf7z6r0eGke9Zpc0tSt2QM5ATbcVKZJCGqNrmieTHbfsT8mABeglK+PULKEzTusl9lZ5YqgH80p0c6q1D2I/625V+wtKPoavJHt81nSCon79xhUfG+rDNUF70fQ2BW0+E3BNaYQzMaxV5DdNZ/smQ7qCpRFcQyuDK3KkC/LyN7Ta6jW6ySbEc3g5bfxcXJk48CtuQO54Bc8SdfSZRfkObywK/9ZPdPC2SGL2WqIb/wVp9whAhC1wFqwQ== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2f64b249-2979-4f8d-ab1f-08d9e7f162fc X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Feb 2022 15:16:58.5371 (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: VI1PR03MB4703 Subject: [FFmpeg-devel] [PATCH 4/7] avcodec/cbs_mpeg2: Simplify splitting fragment 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: X+rNWE6+gcHj avpriv_find_start_code() supports non-contiguous buffers by maintaining a state that allows to find start codes that span across multiple buffers; a consequence thereof is that avpriv_find_start_code() is given a zero-sized buffer, it does not modify this state, so that it appears as if a start code was found if the state contained a start code. This can e.g. happen with Sequence End units in MPEG-2 and to counter this, cbs_mpeg2_split_fragment() reset the state when it has already encountered the end of the fragment in order to add the last unit (if it is only of the form 00 00 01 xy) only once; it also used a flag to set whether this is the final unit. Yet this can be improved by simply resetting state unconditionally (thereby avoiding a branch); the flag can be removed by just checking whether we have a valid start code (of the next unit to add) at the end. Signed-off-by: Andreas Rheinhardt --- libavcodec/cbs_mpeg2.c | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 2bf38c6001..90d667ddc8 100644 --- a/libavcodec/cbs_mpeg2.c +++ b/libavcodec/cbs_mpeg2.c @@ -149,7 +149,6 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, uint32_t start_code = -1; size_t unit_size; int err; - int final = 0; start = avpriv_find_start_code(frag->data, frag->data + frag->data_size, &start_code); @@ -161,14 +160,11 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, do { unit_type = start_code & 0xff; - if (start == frag->data + frag->data_size) { - // The last four bytes form a start code which constitutes - // a unit of its own. In this situation avpriv_find_start_code - // won't modify start_code at all so modify start_code so that - // the next unit will be treated as the last unit. - start_code = 0; - } - + // Reset start_code to ensure that avpriv_find_start_code() + // really reads a new start code and does not reuse the old + // start code in any way (as e.g. happens when there is a + // Sequence End unit at the very end of a packet). + start_code = UINT32_MAX; end = avpriv_find_start_code(start--, frag->data + frag->data_size, &start_code); @@ -183,7 +179,6 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, } else { // We didn't find a start code, so this is the final unit. unit_size = end - start; - final = 1; } err = ff_cbs_append_unit_data(frag, unit_type, (uint8_t*)start, @@ -192,7 +187,7 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, return err; start = end; - } while (!final); + } while ((start_code >> 8) == 0x000001); return 0; } From patchwork Fri Feb 4 15:16:46 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 34122 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6602:2c4e:0:0:0:0 with SMTP id x14csp3493503iov; Fri, 4 Feb 2022 07:17:39 -0800 (PST) X-Google-Smtp-Source: ABdhPJxLkwRHd1Qfbd9mwzP05Yjlzb/Zl+z3NDG4oBXwlK6kErplI4aULN+ZGQypMVy2evyfAUn2 X-Received: by 2002:a05:6402:438d:: with SMTP id o13mr3408293edc.258.1643987859404; Fri, 04 Feb 2022 07:17:39 -0800 (PST) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id do2si1435707ejc.362.2022.02.04.07.17.39; Fri, 04 Feb 2022 07:17:39 -0800 (PST) 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=CcdZKraX; 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 5EF1C68B295; Fri, 4 Feb 2022 17:17:12 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2060.outbound.protection.outlook.com [40.92.90.60]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 4E15968B289 for ; Fri, 4 Feb 2022 17:17:05 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=P6fCqeVdiHwLFjH/Cb6FJ0OsnPsTxxGfX//EKx0po43uc4QxwTYPzcLVKv25Gv/hJLLnRb+RHVTIdoevWoNvsd56ZMTOjspOBDb/urHr+Ap11NEQbj/hYEROAWV7du0xR1N6C1ynwJ+iEAPQpd+QIAD9N62R3Ef2G9H4TDuURQAyTtuKYuoTzGUDy9u5tCtwkFjpkGM4vfNpjwc2N0PcHLjFk2cceUjWVdllGo7nUohYdS18qSUiKO+PjtQr/T0zNlCBBYntBCtWQKqBcWrZ0D6r+2Bud2o6Mre7c4DLg4xyQQxo8jRxFgdBA169YxkqfC11UDPJkujnG+F0C/Chdw== 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=eGeBViGa9amvDUIraPrDKqbQOwtT9YjXhoc/Ok3+Yk4=; b=TpdqNwCRyRFbRgbPq9ygQygQJ7vKrfU8SCgUP8o9Om33PGhd42mUeB0Vsmgd1ZSwtt1ltwTqjCbf7V3G3TqKAux2QfiQdUHajULWO7dWmRwI4V654ihvG9q5XVtshyHLzwPajxZBa4cPwlnuZkr6zbHgIMsKWCKO837T9rr8y6UXJItdQO2DYISBQzLgPcMUEU4D+Lz2tMcQ0tO7Mwi2n9jEgx5oYhQOFNHzCgpqpbwyQO1fhUDmeN9hL6MHTLtk2+odJ7WD2lJzgY2Ekttnjt3mKsyr4IyMX0bySR8QHpNhere6ojwYX/HNUdOaS+PTUrKq4cSjyt3D7qBgjQDAHQ== 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=eGeBViGa9amvDUIraPrDKqbQOwtT9YjXhoc/Ok3+Yk4=; b=CcdZKraXr8kujCBhLnxeG7YW+FnUg8GQunbN3gb1dxVf4H5ywxxNtVRyq3mEFJgMNoDkChFXH9Jxu9SpNIuJIst7XJgS9DYsjHKgBKDMGGj/hrELCMMFVgsIXdyNgOg3oaB8IHBRttEunid67G+wycx8S1MdVNBS2BfTTXb8e41bKISH+8MOLwdeXZcflC+kIbhm6kct0pGlzNoH4LYBOYTEuJRMMhjHaCIF9LPalldmknCVAFVDDKOKhBuf0NlutaxZymbqgTvD/1xjKWLZRhLC+DOrrH4iVhyJWtJb5qvYNmm57NFwvOV2vZxHJ1Y5ltrcXYSCUB4YS+tc+EqGwg== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by VI1PR03MB4703.eurprd03.prod.outlook.com (2603:10a6:803:5d::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Fri, 4 Feb 2022 15:16:59 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22%5]) with mapi id 15.20.4951.014; Fri, 4 Feb 2022 15:16:59 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 4 Feb 2022 16:16:46 +0100 Message-ID: X-Mailer: git-send-email 2.32.0 In-Reply-To: References: X-TMN: [mEvUQYa1KylM3XleumHsvoo+dKqPR5OR] X-ClientProxiedBy: AM5PR0101CA0032.eurprd01.prod.exchangelabs.com (2603:10a6:206:16::45) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: <20220204151648.2148917-4-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 8362d0b4-76c4-42dc-88f4-08d9e7f163a3 X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiSjtu7sMDnYCqklgc/dtjmmUqGTl3bq4N/hlp049pXmZ1uvZJpDWB5beqGcOKpzHZRo5RZiDugtN5UWLoPuCQrvZ43DkxaMAvhWS7KVNal8OdIwANYGbVMXSsOsutZ1b+tDh/t0zMMaX7fA0+CWaONGznx4hrq7V1gubB8ZG3t3gOMRmt/HHVk2X3szMh26qOTGBbA0tey7E0Nz/as7rV6qZVDUNbSsAbrLOEaYGqk1+kwXSltKuFTYrzPetAKgjOCiRF8MUQf3df3Wsk1hfHWHTFoSkMuYQx9X+BdxJYsEa+dZEZYWboFcWaHvwIL7xDoI8OBwmm1Kib8UsFB4kTh2o1jNl/Wur6Vsi82aBOegFsfDoHlx4LkBbb7MXJ8K365soNaEOLdqnRC/34I6/LGukGumNIg2EPBWanSQqBooYF7v2SrDaqXcUNgwX9S73yQn2pTIvjILqNv+eMrDDeTjiHtQ3o2M/QMizpmIbtv1bVw4GT4zO/sFzK5YdIJHekyXIWVOW6cZro0C4jc/4GErPpnFRHk7JGayE0osrVIVIujQOidlzLpgjNRSzjfknjhvELcwni9z+HX7hB9LvmiPMmwEbpIDJDKZuBT85NAjuQc5gtw6E4DQb2TGvBF9SXkzpdFtsW7Na6tYkMi7bk8mJuQCGsuOb6ba/ZuclG3/ZC3zIUmwXG0NU37bEIRqaNIVyPNvaJZchzBN8jhAlUzF9bA7BHa27P66sf9ICyDp/aIErGLykHD9P8mZqgC26SQ= X-MS-TrafficTypeDiagnostic: VI1PR03MB4703:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 9oGdz0xpSGqvhbuQFKeEFzoG2tZMW4zLKSIP9WNWYArcNDJVW0t+5popc1xn5KKKTxgScesGY2VMWJ3lrtrjGCI6ceid/Pe0IhOnGVAujPvgI1GnQwVt7ZJR6+BHUJZ5eh3u/aX+rUBFNd7EEsPyW1Bul8wxqKV7qzkWnhKMg0RGWhACZX8eUXKKVkv63xWiLZwhNULKfV2MRDGMS14RyLvwJ1NCwURkKkIRa1Op7E5CkuIaMbUzrTd3mrwCbuXlBvVw7namGaMK+2Z0qTp1HUFk8u5qdLyhFKm+vHES9D81pP3Mr94Wv/fNja46rF3Osf3drI57YA3y22hM6AA8zswL5hXdo65mvLwI9lTC7E6YclQGZBKKDZYP3HU/RG4G56K+FxQ2kjJXx/c3tQEAJk9y5bnvI5PPX6OhaMCTzxeuQIGCgWGfE0+vQgBXMmMMMIOCiYszqFhc5S6o3lATRpqJTgbfnoCWbv+J90zxmZ4LM4oV3rKay1Zsd+e6GJ+SBfSPDMimCO8wCVoQkNlmKJUWaEJ7HZYlkJfDSqg7tW1+2PmtMKmrXzMoF8tdr7VWrIbFgDG3H7zfxkTYoeshVg== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: OGorecWdz7n97Kns2hlGyxaGibm0BT0s6HIyzxOVNdEfoeZ/ae3j9YME4mGt0IISgD/Bxw1L/y6c82vm5lRPSOID2XDe3Cf6pos7TEZFHbsaPhdXathMzSJcqNBE1v4cRosBDuZojOHDDSW6DViqGoX7WZs+KuxcSzcSAwckUxLmOMa24cS0dnoAbWVc2Adlh9ZHynXBOwjJiVFq4OYGmpR8b57qLPRzUq7Y+9Kkdla18zv8GAnV79g0GQMOh+8beNfoFBOYhynx/6FE9+hLWF0bVWPcbbKBQLaLIrUILYb4gzkCSk7PzpD29WbXBTIJtAddX/n91RUzIsjF9cnMx9Vp+9PVP9jmoYb65XCc7lHR14ZsKK8zxA7TYfXs92OJPHlSsQ9PqCTMrBOitpG/BjE5COzwDqDcJDeuTg/7iZpjt8J7/W7ymS5CFCEvqLsYNcWhtxOphl/Pa2qtvDmv+UNSSGgXMIMP4bCaslnza8gujSI6cgGYBe0+WAdj5qpl6zRxypOVTz3o7kIjSDkdByq1INAbwyHGuT2OvyN6C/SPNoNT93GWmCUPGw/K1tw36Qvfeg+cBWYO1+gobcX5IVlyN5pQ0pYf3RO4ShCR13v7nyuAsQ9vBQOGbIvX4P5Aw5yjiioGWmHTu7z9CN9k+QeCSEkwgdFYIR7kOghGB7ybXwsf5HzrH7G8/Ynl/uks9AVoZ57B9TM+bpAQYLt0vJu85Cl9AbggD9vKARlBG8DwPtMq5owA0drv3Nvws4XT686WPvODvPl0OVGOyb/J4m9byBZ2d7VTZvgRNqtxRya/0TujhHJ1vaxEngLC16hev6157/8xhrgJrE77RVhdbfngAnH9fM04UxUEvBj7O021objRBw19gjzS50rwW9tVHdj92Mv5iugCl/KbbfH2oj8LCiVKKUpxF93uzMA0UiWOfiGoQ2+diihUc3KcxHC0Vw4Jt2LxPZKOVMzhlaOpoQ== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8362d0b4-76c4-42dc-88f4-08d9e7f163a3 X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Feb 2022 15:16:59.6931 (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: VI1PR03MB4703 Subject: [FFmpeg-devel] [PATCH 5/7] all: Remove unnecessary libavcodec/internal.h inclusions 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: 3I1HiqgDw0bv Signed-off-by: Andreas Rheinhardt --- libavcodec/aac_adtstoasc_bsf.c | 1 - libavcodec/aliaspixenc.c | 1 - libavcodec/av1_parser.c | 1 - libavcodec/avpacket.c | 6 ++---- libavcodec/cbs_av1.c | 2 +- libavcodec/cbs_vp9.c | 1 - libavcodec/cljrenc.c | 2 -- libavcodec/dca_core.c | 1 + libavcodec/dca_core.h | 2 -- libavcodec/dca_lbr.c | 1 + libavcodec/dca_lbr.h | 2 -- libavcodec/dca_xll.c | 1 + libavcodec/dca_xll.h | 2 -- libavcodec/dcadec.c | 1 + libavcodec/dv.c | 7 ------- libavcodec/dvbsub_parser.c | 1 - libavcodec/dxtory.c | 1 - libavcodec/ffv1.c | 6 ------ libavcodec/ffv1.h | 4 ---- libavcodec/fitsenc.c | 1 - libavcodec/flacdata.c | 2 +- libavcodec/flacdata.h | 2 +- libavcodec/h264_ps.c | 1 - libavcodec/h264_refs.c | 1 - libavcodec/hevc_parser.c | 1 - libavcodec/hevc_refs.c | 2 -- libavcodec/hevcdec.c | 1 + libavcodec/hevcdec.h | 2 -- libavcodec/ituh263dec.c | 1 - libavcodec/jpeg2000dwt.c | 1 - libavcodec/motion_est.c | 1 - libavcodec/opusenc.h | 1 - libavcodec/parser.c | 1 - libavcodec/ratecontrol.c | 1 - libavcodec/scpr.h | 5 ----- libavcodec/sheervideo.c | 5 ----- libavcodec/ttadata.c | 1 + libavcodec/ttadata.h | 2 +- libavcodec/twinvq.h | 1 - libavcodec/vaapi_mpeg2.c | 1 - libavcodec/vaapi_mpeg4.c | 1 - libavcodec/vaapi_vc1.c | 1 - libavcodec/vc1_parser.c | 1 - libavcodec/vdpau_h264.c | 1 - libavcodec/vdpau_hevc.c | 1 - libavcodec/vdpau_vp9.c | 1 - libavcodec/vp9mvs.c | 1 - libavcodec/wma.c | 1 - libavcodec/xbmenc.c | 1 - libavcodec/xfaceenc.c | 1 - libavcodec/xwdenc.c | 2 -- libavcodec/yuv4enc.c | 1 - libavformat/assdec.c | 1 - libavformat/h264dec.c | 1 - libavformat/hcom.c | 1 - libavformat/ipudec.c | 1 - libavformat/jacosubdec.c | 1 - libavformat/network.c | 1 - libavformat/samidec.c | 2 -- libavformat/subviewerdec.c | 1 - 60 files changed, 12 insertions(+), 87 deletions(-) diff --git a/libavcodec/aac_adtstoasc_bsf.c b/libavcodec/aac_adtstoasc_bsf.c index 69453c706f..267ef97572 100644 --- a/libavcodec/aac_adtstoasc_bsf.c +++ b/libavcodec/aac_adtstoasc_bsf.c @@ -26,7 +26,6 @@ #include "put_bits.h" #include "get_bits.h" #include "mpeg4audio.h" -#include "internal.h" typedef struct AACBSFContext { int first_frame_done; diff --git a/libavcodec/aliaspixenc.c b/libavcodec/aliaspixenc.c index fa273df9c2..27b7d1d8de 100644 --- a/libavcodec/aliaspixenc.c +++ b/libavcodec/aliaspixenc.c @@ -24,7 +24,6 @@ #include "avcodec.h" #include "bytestream.h" #include "encode.h" -#include "internal.h" #define ALIAS_HEADER_SIZE 10 diff --git a/libavcodec/av1_parser.c b/libavcodec/av1_parser.c index d2dfdb3580..4cbd7408a0 100644 --- a/libavcodec/av1_parser.c +++ b/libavcodec/av1_parser.c @@ -23,7 +23,6 @@ #include "libavutil/avassert.h" #include "cbs.h" #include "cbs_av1.h" -#include "internal.h" #include "parser.h" typedef struct AV1ParseContext { diff --git a/libavcodec/avpacket.c b/libavcodec/avpacket.c index 4f7a6b255c..bcbc4166cb 100644 --- a/libavcodec/avpacket.c +++ b/libavcodec/avpacket.c @@ -22,14 +22,12 @@ #include #include "libavutil/avassert.h" -#include "libavutil/common.h" -#include "libavutil/internal.h" +#include "libavutil/intreadwrite.h" #include "libavutil/mathematics.h" #include "libavutil/mem.h" #include "libavutil/rational.h" -#include "bytestream.h" -#include "internal.h" +#include "defs.h" #include "packet.h" #include "packet_internal.h" diff --git a/libavcodec/cbs_av1.c b/libavcodec/cbs_av1.c index 04314443de..1229480567 100644 --- a/libavcodec/cbs_av1.c +++ b/libavcodec/cbs_av1.c @@ -20,10 +20,10 @@ #include "libavutil/opt.h" #include "libavutil/pixfmt.h" +#include "avcodec.h" #include "cbs.h" #include "cbs_internal.h" #include "cbs_av1.h" -#include "internal.h" static int cbs_av1_read_uvlc(CodedBitstreamContext *ctx, GetBitContext *gbc, diff --git a/libavcodec/cbs_vp9.c b/libavcodec/cbs_vp9.c index 333d70ba4e..ae7f88a8a3 100644 --- a/libavcodec/cbs_vp9.c +++ b/libavcodec/cbs_vp9.c @@ -21,7 +21,6 @@ #include "cbs.h" #include "cbs_internal.h" #include "cbs_vp9.h" -#include "internal.h" static int cbs_vp9_read_s(CodedBitstreamContext *ctx, GetBitContext *gbc, diff --git a/libavcodec/cljrenc.c b/libavcodec/cljrenc.c index aa53a110d0..21e033753b 100644 --- a/libavcodec/cljrenc.c +++ b/libavcodec/cljrenc.c @@ -24,12 +24,10 @@ * Cirrus Logic AccuPak encoder. */ -#include "libavutil/common.h" #include "libavutil/opt.h" #include "avcodec.h" #include "encode.h" -#include "internal.h" #include "put_bits.h" typedef struct CLJRContext { diff --git a/libavcodec/dca_core.c b/libavcodec/dca_core.c index 758e3447a6..80f3525b46 100644 --- a/libavcodec/dca_core.c +++ b/libavcodec/dca_core.c @@ -25,6 +25,7 @@ #include "dcahuff.h" #include "dcamath.h" #include "dca_syncwords.h" +#include "internal.h" #if ARCH_ARM #include "arm/dca.h" diff --git a/libavcodec/dca_core.h b/libavcodec/dca_core.h index 86bfa62203..cb8e38a94b 100644 --- a/libavcodec/dca_core.h +++ b/libavcodec/dca_core.h @@ -21,13 +21,11 @@ #ifndef AVCODEC_DCA_CORE_H #define AVCODEC_DCA_CORE_H -#include "libavutil/common.h" #include "libavutil/float_dsp.h" #include "libavutil/fixed_dsp.h" #include "libavutil/mem_internal.h" #include "avcodec.h" -#include "internal.h" #include "get_bits.h" #include "dca.h" #include "dca_exss.h" diff --git a/libavcodec/dca_lbr.c b/libavcodec/dca_lbr.c index 481a8df7e6..c1e3b0ddc3 100644 --- a/libavcodec/dca_lbr.c +++ b/libavcodec/dca_lbr.c @@ -28,6 +28,7 @@ #include "dcahuff.h" #include "dca_syncwords.h" #include "bytestream.h" +#include "internal.h" #define AMP_MAX 56 diff --git a/libavcodec/dca_lbr.h b/libavcodec/dca_lbr.h index 2eccbe1ff2..db7a676c31 100644 --- a/libavcodec/dca_lbr.h +++ b/libavcodec/dca_lbr.h @@ -21,12 +21,10 @@ #ifndef AVCODEC_DCA_LBR_H #define AVCODEC_DCA_LBR_H -#include "libavutil/common.h" #include "libavutil/float_dsp.h" #include "libavutil/mem_internal.h" #include "avcodec.h" -#include "internal.h" #include "get_bits.h" #include "dca.h" #include "dca_exss.h" diff --git a/libavcodec/dca_xll.c b/libavcodec/dca_xll.c index ab14dbcc88..316045b588 100644 --- a/libavcodec/dca_xll.c +++ b/libavcodec/dca_xll.c @@ -23,6 +23,7 @@ #include "dcadata.h" #include "dcamath.h" #include "dca_syncwords.h" +#include "internal.h" #include "unary.h" static int get_linear(GetBitContext *gb, int n) diff --git a/libavcodec/dca_xll.h b/libavcodec/dca_xll.h index 3aaad6d370..d223133019 100644 --- a/libavcodec/dca_xll.h +++ b/libavcodec/dca_xll.h @@ -21,11 +21,9 @@ #ifndef AVCODEC_DCA_XLL_H #define AVCODEC_DCA_XLL_H -#include "libavutil/common.h" #include "libavutil/mem_internal.h" #include "avcodec.h" -#include "internal.h" #include "get_bits.h" #include "dca.h" #include "dcadsp.h" diff --git a/libavcodec/dcadec.c b/libavcodec/dcadec.c index 43694b51f7..49b52f805f 100644 --- a/libavcodec/dcadec.c +++ b/libavcodec/dcadec.c @@ -25,6 +25,7 @@ #include "dcadec.h" #include "dcahuff.h" #include "dca_syncwords.h" +#include "internal.h" #include "profiles.h" #define MIN_PACKET_SIZE 16 diff --git a/libavcodec/dv.c b/libavcodec/dv.c index 8a4cac859e..e2550c4cc1 100644 --- a/libavcodec/dv.c +++ b/libavcodec/dv.c @@ -38,15 +38,8 @@ * DV codec. */ -#include "libavutil/internal.h" -#include "libavutil/pixdesc.h" - #include "avcodec.h" #include "dv.h" -#include "dvdata.h" -#include "internal.h" -#include "put_bits.h" -#include "simple_idct.h" static inline void dv_calc_mb_coordinates(const AVDVProfile *d, int chan, int seq, int slot, uint16_t *tbl) diff --git a/libavcodec/dvbsub_parser.c b/libavcodec/dvbsub_parser.c index 88bf5b9d74..b2d5446867 100644 --- a/libavcodec/dvbsub_parser.c +++ b/libavcodec/dvbsub_parser.c @@ -25,7 +25,6 @@ #include "libavutil/intreadwrite.h" #include "avcodec.h" -#include "internal.h" /* Parser (mostly) copied from dvdsub.c */ diff --git a/libavcodec/dxtory.c b/libavcodec/dxtory.c index 914131e742..36c30c1dbc 100644 --- a/libavcodec/dxtory.c +++ b/libavcodec/dxtory.c @@ -29,7 +29,6 @@ #include "avcodec.h" #include "bytestream.h" #include "get_bits.h" -#include "internal.h" #include "unary.h" #include "thread.h" diff --git a/libavcodec/ffv1.c b/libavcodec/ffv1.c index d8a0c39254..7eaa72f244 100644 --- a/libavcodec/ffv1.c +++ b/libavcodec/ffv1.c @@ -27,15 +27,9 @@ #include "libavutil/attributes.h" #include "libavutil/avassert.h" -#include "libavutil/crc.h" -#include "libavutil/opt.h" -#include "libavutil/imgutils.h" -#include "libavutil/pixdesc.h" #include "avcodec.h" -#include "internal.h" #include "rangecoder.h" -#include "mathops.h" #include "ffv1.h" av_cold int ff_ffv1_common_init(AVCodecContext *avctx) diff --git a/libavcodec/ffv1.h b/libavcodec/ffv1.h index 8ffe5ab433..ba612bc258 100644 --- a/libavcodec/ffv1.h +++ b/libavcodec/ffv1.h @@ -28,13 +28,9 @@ * FF Video Codec 1 (a lossless codec) */ -#include "libavutil/crc.h" -#include "libavutil/opt.h" #include "libavutil/imgutils.h" -#include "libavutil/pixdesc.h" #include "avcodec.h" #include "get_bits.h" -#include "internal.h" #include "mathops.h" #include "put_bits.h" #include "rangecoder.h" diff --git a/libavcodec/fitsenc.c b/libavcodec/fitsenc.c index dfaa04cdbd..3ac1141b69 100644 --- a/libavcodec/fitsenc.c +++ b/libavcodec/fitsenc.c @@ -33,7 +33,6 @@ #include "avcodec.h" #include "bytestream.h" #include "encode.h" -#include "internal.h" static int fits_encode_frame(AVCodecContext *avctx, AVPacket *pkt, const AVFrame *pict, int *got_packet) diff --git a/libavcodec/flacdata.c b/libavcodec/flacdata.c index 1954f32d32..d96e3e0966 100644 --- a/libavcodec/flacdata.c +++ b/libavcodec/flacdata.c @@ -19,7 +19,7 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include "internal.h" +#include "flacdata.h" const int ff_flac_sample_rate_table[16] = { 0, diff --git a/libavcodec/flacdata.h b/libavcodec/flacdata.h index e2c1e5d7f2..ef21840777 100644 --- a/libavcodec/flacdata.h +++ b/libavcodec/flacdata.h @@ -22,7 +22,7 @@ #ifndef AVCODEC_FLACDATA_H #define AVCODEC_FLACDATA_H -#include "internal.h" +#include extern const int ff_flac_sample_rate_table[16]; diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c index 6927fa7198..051f06692c 100644 --- a/libavcodec/h264_ps.c +++ b/libavcodec/h264_ps.c @@ -28,7 +28,6 @@ #include #include "libavutil/imgutils.h" -#include "internal.h" #include "mathops.h" #include "avcodec.h" #include "h264data.h" diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c index 94b473257f..50bbe94917 100644 --- a/libavcodec/h264_refs.c +++ b/libavcodec/h264_refs.c @@ -28,7 +28,6 @@ #include #include "libavutil/avassert.h" -#include "internal.h" #include "avcodec.h" #include "h264.h" #include "h264dec.h" diff --git a/libavcodec/hevc_parser.c b/libavcodec/hevc_parser.c index c944a6aacd..59f9a0ff3e 100644 --- a/libavcodec/hevc_parser.c +++ b/libavcodec/hevc_parser.c @@ -28,7 +28,6 @@ #include "hevc_ps.h" #include "hevc_sei.h" #include "h2645_parse.h" -#include "internal.h" #include "parser.h" #define START_CODE 0x000001 ///< start_code_prefix_one_3bytes diff --git a/libavcodec/hevc_refs.c b/libavcodec/hevc_refs.c index 06e42d9c53..d496918feb 100644 --- a/libavcodec/hevc_refs.c +++ b/libavcodec/hevc_refs.c @@ -22,9 +22,7 @@ */ #include "libavutil/avassert.h" -#include "libavutil/pixdesc.h" -#include "internal.h" #include "thread.h" #include "hevc.h" #include "hevcdec.h" diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c index 8d7a4f7147..ed012c4e75 100644 --- a/libavcodec/hevcdec.c +++ b/libavcodec/hevcdec.c @@ -44,6 +44,7 @@ #include "hevc_parse.h" #include "hevcdec.h" #include "hwconfig.h" +#include "internal.h" #include "profiles.h" const uint8_t ff_hevc_pel_weight[65] = { [2] = 0, [4] = 1, [6] = 2, [8] = 3, [12] = 4, [16] = 5, [24] = 6, [32] = 7, [48] = 8, [64] = 9 }; diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h index 157bc6926f..8e26c852db 100644 --- a/libavcodec/hevcdec.h +++ b/libavcodec/hevcdec.h @@ -26,7 +26,6 @@ #include #include "libavutil/buffer.h" -#include "libavutil/md5.h" #include "libavutil/mem_internal.h" #include "avcodec.h" @@ -41,7 +40,6 @@ #include "hevc_sei.h" #include "hevcdsp.h" #include "h274.h" -#include "internal.h" #include "thread.h" #include "videodsp.h" diff --git a/libavcodec/ituh263dec.c b/libavcodec/ituh263dec.c index 7d7a1f01a2..445d5f3f36 100644 --- a/libavcodec/ituh263dec.c +++ b/libavcodec/ituh263dec.c @@ -40,7 +40,6 @@ #include "h263.h" #include "h263data.h" #include "h263dec.h" -#include "internal.h" #include "mathops.h" #include "mpegutils.h" #include "unary.h" diff --git a/libavcodec/jpeg2000dwt.c b/libavcodec/jpeg2000dwt.c index f9a0977407..ed24091fe8 100644 --- a/libavcodec/jpeg2000dwt.c +++ b/libavcodec/jpeg2000dwt.c @@ -28,7 +28,6 @@ #include "libavutil/common.h" #include "libavutil/mem.h" #include "jpeg2000dwt.h" -#include "internal.h" /* Defines for 9/7 DWT lifting parameters. * Parameters are in float. */ diff --git a/libavcodec/motion_est.c b/libavcodec/motion_est.c index 5b0958733c..9c548c1567 100644 --- a/libavcodec/motion_est.c +++ b/libavcodec/motion_est.c @@ -32,7 +32,6 @@ #include #include "avcodec.h" -#include "internal.h" #include "mathops.h" #include "motion_est.h" #include "mpegutils.h" diff --git a/libavcodec/opusenc.h b/libavcodec/opusenc.h index b623b3e948..32d23deead 100644 --- a/libavcodec/opusenc.h +++ b/libavcodec/opusenc.h @@ -22,7 +22,6 @@ #ifndef AVCODEC_OPUSENC_H #define AVCODEC_OPUSENC_H -#include "internal.h" #include "opus_celt.h" /* Determines the maximum delay the psychoacoustic system will use for lookahead */ diff --git a/libavcodec/parser.c b/libavcodec/parser.c index 3b459cc125..24fe44eb30 100644 --- a/libavcodec/parser.c +++ b/libavcodec/parser.c @@ -25,7 +25,6 @@ #include #include "libavutil/avassert.h" -#include "libavutil/internal.h" #include "libavutil/mem.h" #include "internal.h" diff --git a/libavcodec/ratecontrol.c b/libavcodec/ratecontrol.c index 6b77ccd006..9cfc21df64 100644 --- a/libavcodec/ratecontrol.c +++ b/libavcodec/ratecontrol.c @@ -29,7 +29,6 @@ #include "libavutil/internal.h" #include "avcodec.h" -#include "internal.h" #include "ratecontrol.h" #include "mpegutils.h" #include "mpegvideo.h" diff --git a/libavcodec/scpr.h b/libavcodec/scpr.h index 15cb87c464..d161a42c6e 100644 --- a/libavcodec/scpr.h +++ b/libavcodec/scpr.h @@ -23,13 +23,8 @@ #ifndef AVCODEC_SCPR_H #define AVCODEC_SCPR_H -#include -#include -#include - #include "avcodec.h" #include "bytestream.h" -#include "internal.h" #include "scpr3.h" typedef struct RangeCoder { diff --git a/libavcodec/sheervideo.c b/libavcodec/sheervideo.c index fb5a39303e..dbe8e4f516 100644 --- a/libavcodec/sheervideo.c +++ b/libavcodec/sheervideo.c @@ -19,17 +19,12 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include -#include -#include - #define CACHED_BITSTREAM_READER !ARCH_X86_32 #define SHEER_VLC_BITS 12 #include "libavutil/intreadwrite.h" #include "avcodec.h" #include "get_bits.h" -#include "internal.h" #include "thread.h" #include "sheervideodata.h" diff --git a/libavcodec/ttadata.c b/libavcodec/ttadata.c index aa9f418a7d..297d709480 100644 --- a/libavcodec/ttadata.c +++ b/libavcodec/ttadata.c @@ -18,6 +18,7 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include #include "ttadata.h" const uint32_t ff_tta_shift_1[] = { diff --git a/libavcodec/ttadata.h b/libavcodec/ttadata.h index 48c4cd0354..1ec1d749ce 100644 --- a/libavcodec/ttadata.h +++ b/libavcodec/ttadata.h @@ -21,7 +21,7 @@ #ifndef AVCODEC_TTADATA_H #define AVCODEC_TTADATA_H -#include "internal.h" +#include #define MAX_ORDER 16 typedef struct TTAFilter { diff --git a/libavcodec/twinvq.h b/libavcodec/twinvq.h index 24e5ebcd0b..234604f581 100644 --- a/libavcodec/twinvq.h +++ b/libavcodec/twinvq.h @@ -29,7 +29,6 @@ #include "libavutil/float_dsp.h" #include "avcodec.h" #include "fft.h" -#include "internal.h" enum TwinVQCodec { TWINVQ_CODEC_VQF, diff --git a/libavcodec/vaapi_mpeg2.c b/libavcodec/vaapi_mpeg2.c index 26e0cd827c..ba70ddef31 100644 --- a/libavcodec/vaapi_mpeg2.c +++ b/libavcodec/vaapi_mpeg2.c @@ -23,7 +23,6 @@ #include "hwconfig.h" #include "mpegutils.h" #include "mpegvideo.h" -#include "internal.h" #include "vaapi_decode.h" /** Reconstruct bitstream f_code */ diff --git a/libavcodec/vaapi_mpeg4.c b/libavcodec/vaapi_mpeg4.c index 71e155154c..bc11a7e417 100644 --- a/libavcodec/vaapi_mpeg4.c +++ b/libavcodec/vaapi_mpeg4.c @@ -22,7 +22,6 @@ #include "h263.h" #include "hwconfig.h" -#include "internal.h" #include "mpeg4video.h" #include "mpegvideo.h" #include "vaapi_decode.h" diff --git a/libavcodec/vaapi_vc1.c b/libavcodec/vaapi_vc1.c index 4e9607d9be..d82b5b31f8 100644 --- a/libavcodec/vaapi_vc1.c +++ b/libavcodec/vaapi_vc1.c @@ -21,7 +21,6 @@ */ #include "hwconfig.h" -#include "internal.h" #include "vaapi_decode.h" #include "vc1.h" #include "vc1data.h" diff --git a/libavcodec/vc1_parser.c b/libavcodec/vc1_parser.c index 00896ddb09..8257e0ccfa 100644 --- a/libavcodec/vc1_parser.c +++ b/libavcodec/vc1_parser.c @@ -29,7 +29,6 @@ #include "parser.h" #include "vc1.h" #include "get_bits.h" -#include "internal.h" /** The maximum number of bytes of a sequence, entry point or * frame header whose values we pay any attention to */ diff --git a/libavcodec/vdpau_h264.c b/libavcodec/vdpau_h264.c index 5ba73dafd4..525e208495 100644 --- a/libavcodec/vdpau_h264.c +++ b/libavcodec/vdpau_h264.c @@ -24,7 +24,6 @@ #include #include "avcodec.h" -#include "internal.h" #include "h264dec.h" #include "h264_ps.h" #include "hwconfig.h" diff --git a/libavcodec/vdpau_hevc.c b/libavcodec/vdpau_hevc.c index c17de0c5d0..2669040f78 100644 --- a/libavcodec/vdpau_hevc.c +++ b/libavcodec/vdpau_hevc.c @@ -23,7 +23,6 @@ #include #include "avcodec.h" -#include "internal.h" #include "hevc_data.h" #include "hevcdec.h" #include "hwconfig.h" diff --git a/libavcodec/vdpau_vp9.c b/libavcodec/vdpau_vp9.c index a8609d647e..49fe18189b 100644 --- a/libavcodec/vdpau_vp9.c +++ b/libavcodec/vdpau_vp9.c @@ -23,7 +23,6 @@ #include #include "libavutil/pixdesc.h" #include "avcodec.h" -#include "internal.h" #include "vp9data.h" #include "vp9dec.h" #include "hwconfig.h" diff --git a/libavcodec/vp9mvs.c b/libavcodec/vp9mvs.c index 88db1c341c..774aaa3910 100644 --- a/libavcodec/vp9mvs.c +++ b/libavcodec/vp9mvs.c @@ -21,7 +21,6 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include "internal.h" #include "vp56.h" #include "vp9.h" #include "vp9data.h" diff --git a/libavcodec/wma.c b/libavcodec/wma.c index d9fda2dbfa..c73711df48 100644 --- a/libavcodec/wma.c +++ b/libavcodec/wma.c @@ -22,7 +22,6 @@ #include "libavutil/attributes.h" #include "avcodec.h" -#include "internal.h" #include "sinewin.h" #include "wma.h" #include "wma_common.h" diff --git a/libavcodec/xbmenc.c b/libavcodec/xbmenc.c index 5e552ce820..05e014a0bf 100644 --- a/libavcodec/xbmenc.c +++ b/libavcodec/xbmenc.c @@ -22,7 +22,6 @@ #include "avcodec.h" #include "encode.h" -#include "internal.h" #include "mathops.h" #define ANSI_MIN_READLINE 509 diff --git a/libavcodec/xfaceenc.c b/libavcodec/xfaceenc.c index 55815e5c2f..ae258f559e 100644 --- a/libavcodec/xfaceenc.c +++ b/libavcodec/xfaceenc.c @@ -27,7 +27,6 @@ #include "xface.h" #include "avcodec.h" #include "encode.h" -#include "internal.h" #include "libavutil/avassert.h" typedef struct XFaceContext { diff --git a/libavcodec/xwdenc.c b/libavcodec/xwdenc.c index a65f83abdd..31a263b8df 100644 --- a/libavcodec/xwdenc.c +++ b/libavcodec/xwdenc.c @@ -20,12 +20,10 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include "libavutil/intreadwrite.h" #include "libavutil/pixdesc.h" #include "avcodec.h" #include "bytestream.h" #include "encode.h" -#include "internal.h" #include "xwd.h" #define WINDOW_NAME "lavcxwdenc" diff --git a/libavcodec/yuv4enc.c b/libavcodec/yuv4enc.c index 5181a46403..11dfc68a0d 100644 --- a/libavcodec/yuv4enc.c +++ b/libavcodec/yuv4enc.c @@ -22,7 +22,6 @@ #include "avcodec.h" #include "encode.h" -#include "internal.h" static int yuv4_encode_frame(AVCodecContext *avctx, AVPacket *pkt, const AVFrame *pic, int *got_packet) diff --git a/libavformat/assdec.c b/libavformat/assdec.c index 2ab7fb9efb..0915f6fafd 100644 --- a/libavformat/assdec.c +++ b/libavformat/assdec.c @@ -25,7 +25,6 @@ #include "avformat.h" #include "internal.h" #include "subtitles.h" -#include "libavcodec/internal.h" #include "libavutil/bprint.h" typedef struct ASSContext { diff --git a/libavformat/h264dec.c b/libavformat/h264dec.c index 199e87fbfa..ac804c0192 100644 --- a/libavformat/h264dec.c +++ b/libavformat/h264dec.c @@ -23,7 +23,6 @@ #include "libavcodec/golomb.h" #include "avformat.h" #include "rawdec.h" -#include "libavcodec/internal.h" #define MAX_SPS_COUNT 32 #define MAX_PPS_COUNT 256 diff --git a/libavformat/hcom.c b/libavformat/hcom.c index b7fb7314c8..97f0003210 100644 --- a/libavformat/hcom.c +++ b/libavformat/hcom.c @@ -20,7 +20,6 @@ */ #include "libavutil/intreadwrite.h" -#include "libavcodec/internal.h" #include "avformat.h" #include "internal.h" #include "pcm.h" diff --git a/libavformat/ipudec.c b/libavformat/ipudec.c index a314a89001..4e346f9638 100644 --- a/libavformat/ipudec.c +++ b/libavformat/ipudec.c @@ -25,7 +25,6 @@ #include "rawdec.h" #include "libavutil/intreadwrite.h" -#include "libavcodec/internal.h" static int ipu_read_probe(const AVProbeData *p) { diff --git a/libavformat/jacosubdec.c b/libavformat/jacosubdec.c index 68ed7511d7..0ee4820f62 100644 --- a/libavformat/jacosubdec.c +++ b/libavformat/jacosubdec.c @@ -28,7 +28,6 @@ #include "avformat.h" #include "internal.h" #include "subtitles.h" -#include "libavcodec/internal.h" #include "libavcodec/jacosub.h" #include "libavutil/avstring.h" #include "libavutil/bprint.h" diff --git a/libavformat/network.c b/libavformat/network.c index 246ffd406d..96fe636745 100644 --- a/libavformat/network.c +++ b/libavformat/network.c @@ -22,7 +22,6 @@ #include "network.h" #include "tls.h" #include "url.h" -#include "libavcodec/internal.h" #include "libavutil/avutil.h" #include "libavutil/avassert.h" #include "libavutil/mem.h" diff --git a/libavformat/samidec.c b/libavformat/samidec.c index 698164b6a2..0da299343d 100644 --- a/libavformat/samidec.c +++ b/libavformat/samidec.c @@ -27,10 +27,8 @@ #include "avformat.h" #include "internal.h" #include "subtitles.h" -#include "libavcodec/internal.h" #include "libavutil/avstring.h" #include "libavutil/bprint.h" -#include "libavutil/intreadwrite.h" typedef struct { FFDemuxSubtitlesQueue q; diff --git a/libavformat/subviewerdec.c b/libavformat/subviewerdec.c index bcd103bd86..e3a950fce3 100644 --- a/libavformat/subviewerdec.c +++ b/libavformat/subviewerdec.c @@ -28,7 +28,6 @@ #include "internal.h" #include "subtitles.h" #include "avio_internal.h" -#include "libavcodec/internal.h" #include "libavutil/avstring.h" #include "libavutil/bprint.h" #include "libavutil/intreadwrite.h" From patchwork Fri Feb 4 15:16:47 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 34123 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6602:2c4e:0:0:0:0 with SMTP id x14csp3493667iov; Fri, 4 Feb 2022 07:17:52 -0800 (PST) X-Google-Smtp-Source: ABdhPJwnP5XQt44O4ciV3xbFqYr3uSB5WF1DjzN6wHQd0ENhG9xPE5ydNFf0GDrgDYyrnx7E3euQ X-Received: by 2002:aa7:cb17:: with SMTP id s23mr3564486edt.282.1643987872050; Fri, 04 Feb 2022 07:17:52 -0800 (PST) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id u8si1398494eds.8.2022.02.04.07.17.51; Fri, 04 Feb 2022 07:17:52 -0800 (PST) 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=OAQ3D7UI; 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 87A9768B2A4; Fri, 4 Feb 2022 17:17:17 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2060.outbound.protection.outlook.com [40.92.90.60]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 5E74468B297 for ; Fri, 4 Feb 2022 17:17:10 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mB5lNGh7gnefLEytczC3mIyQFUny6Yaq5BJo2i10OfmCDpYcozWFyWZPxOmvxWIzjmoZthTya6DJNaqX8TqZUnROFK4gKDIJNooJRet+Y0zyiHpCDjhrYE9fVFRJl3EuehhotpfeSMbVQ7FboQHTi7SMWDoocDJ3x7B4yFRXCGnVaKISIf62+jwvQ8BMyFllnrVBUBjQp9wb6ejLeVKj/lfpxwH0/MwdVcsBQC3iU7Orup9mMz8oMD661u3wD1osVgZ+SDbPfehmxCTqbVnYdTOQkDsmwkwhS86ex+adQeV1L6pPcoPsL5I2nduzV0NU9MObjdATsgWi9RNvGxOWFQ== 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=UTJXa52lkDewAUa4aSZF2JcqznvSA0sGIV3RzaIGZlg=; b=NZoek7YO4LSP4WG0M+9OmcK+JTp2L5w2bIsoI8TYLxpLramFyC9oeYoKvuYjvXrV2f0eTvufDJIEzbman010uajX++B9EuKqn5xm1U7pCzttNWpYSLHoJj+SATRZTs/WLasQxarzJdSvZAmoGqx4mFfxbCwPh1UcOTfDnHutgamKJJ61gbD2DzUgmpauRgGYdfXBqfSu2+fdc6gUWm06AMcMQsWym7oHrX/7Sgr7RHTBqLi9eiZiik4kLHuttl0SnHDKYzDPv9sCDS0DzFTUtWPB31GRcaDl3X4IvW1xfVZaueB/iRA5niJOPqQaYITXhSTVUm/1fNTTJS/zIx8nog== 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=UTJXa52lkDewAUa4aSZF2JcqznvSA0sGIV3RzaIGZlg=; b=OAQ3D7UIkvDkTllsB2ROoj0jAK19uAyxDY7mkhVhnNfYGfeXJmVs5q9KetZm35s0uoEOZ4hquXWJOKHRMpxJr85k2HS1hUeT9eahvenXJG0s4fn57GzatI2PWpOB4UTDTH133viLRxPuesxaWBf+0FfQQu3dTyo4pzkSrSMayeHzIp9oo9Zah1xXvioj+syIUHq/Pell4MkRFpBIiRr5TqbzTGWRLgaDFPmwYd+nFWfVBOW/U9OYouFhzrFnCCFrQI1DcsNMwS/Orrx0mdaTgeailpMXX8PEPbChTwE1s2QkjS7tN3bYMyzznjko8Ln3vMI3lMZ2Jb6REw6iFx4y9Q== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by VI1PR03MB4703.eurprd03.prod.outlook.com (2603:10a6:803:5d::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Fri, 4 Feb 2022 15:17:00 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22%5]) with mapi id 15.20.4951.014; Fri, 4 Feb 2022 15:17:00 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 4 Feb 2022 16:16:47 +0100 Message-ID: X-Mailer: git-send-email 2.32.0 In-Reply-To: References: X-TMN: [H+98irXNo96BxXlS7/FIKa+YASzbTikq] X-ClientProxiedBy: AM5PR0101CA0032.eurprd01.prod.exchangelabs.com (2603:10a6:206:16::45) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: <20220204151648.2148917-5-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: e1173529-143d-4778-61c8-08d9e7f1644f X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiSjtu7sMDnYCqklgc/dtjmmdKOFrrtb3v6Vhnj/KWPZQBSHA1JVl0g+g5wqu0HS71iG/RYpkr0JJoyVt9+TbgOiOKvuiExKq8KWtO/PyDYHzifHaT6TeDqWC+XtZFOSuSE1J3Yykq9VIHU2fqBFXX9MosoBuUilrCGQgcZt3qaZQoq2EemyJtijhtWCcqdr/oD6rDL+n69/WfDE5+bv3ocGPxetwif514iT2hvmFCrAH8hoirSzVFnRUZWvlsVnk+oOgMH6h6R0KcOTEx6/+T9J9+TWmBPH7pbDGZg7lQE+QQkN+j7NS2zEBWvXwe3pyTkiPRen//q+B89AWwWUITdXGLrOp033BQfy30m2zKOU6IL1GW1tZIZxLEZwt2VPveHND/IEEVnXlKdB9rPlujJxQakTzA0UbycoQqbC0stINwaRcTsu441uaTEBQ1D8oHMt6Wi+YZPJEn4p78JFFvq3JsSY9LMr/vNdBgXyptQsZte6CEkNKocvK9Dg0gy9MIatNk/HsqXKbarB6f2SRQEtzvjXoZNXKrtu/4VYiNDOiPRUKvaWBuLluxLiR83HR/sJSSDsPA9BwxN70RcC6/gkr8SMgGrc01uoN2OWMa2Hj3GObzfPGdIHQ0CpqDg1o4DD2LbW+lUb+BZyRyCd8mkF7K5F2US+IMle6YiEQhUSxJgdYGEtwijpPTtzywoX7fGrXXDFdQkmwt6k0/17A92G0KPsILlBYPHeBuZbvAO6zJWHjjLYgP9LtBhktlFMD/8= X-MS-TrafficTypeDiagnostic: VI1PR03MB4703:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Pq2IQsBDiwWmCopAbZDrsoM/gLPWaJ+juCZ71WLp+38BjuxK+ct8GwrxeECSp5kX6OxDL4VNQX2EO3xZSUBMuZ2DgIwOwxzTsnEkVr2bZkTWiEG7tZYVdHUKpSeOuIMc4w8eBYis17rb8ic/d0IzzCy4K2W3HPxl8gwY0p3BpCDUFRoQvDDgYgEn4h+EU760/9ytgCY+rZF28c1Bc8sS1DAf/nFwu2S9wofrQ1/LwDSaXylSlVK5f+XIQnzNrbFOG8+dJMv2+Rme380RbvnkOGPHKL8qwXI1FIeDM5olKisedGjAq7PKXdjpTI+PI/3NADF1t0tSF2szLNPQxHwARlG25/7BkLHeWD2RqnuDLHsLAOPWzNqzZ17ZCoKswqlKFRsRxYGkYkfRo6/DW0HvhveHLHBD0e1drgAdvuMUph74IpPacA0y1rhZ6jcfzL78ewjXTnPKqnkXeBfVM9+DFeqqmomWV/yda46W+GKVvaUrW8uqDPPPXC1Ws/5BG2kiCTyliOKQlD+tq6E9ij/Nus57ds5k7U7tuJc+cg0JdyUydtq/lK4Pig2z+VSXnShvV7+dRHRqqnY+lL1HPYRigA== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 5ALAHQHK0JiPGGP9A5zGSNcjs6dEHISp+IDyEFdKLEBGKT2f5N2is8JUfjrpZMMuv6BhaPiQ3AXOj6oACc256qxF37jG4H9+Do8C25U/S0KcXTaits/EnDLIUYJGfWccriMHoQIdZ7ZjZmdcTLxwe/2z8r06xd1foKf8S4t/Nv1kErt1/HtRDZEEIj6Br5oxz3/vhaqel6e6p8K2cmt0j3OfPSyzpEmkyWdYUdetE7uTqcVJL18Tsz51FQ3co+5CDOU5Xq6rifL16DJheSP2N0ESM41PJxKiP0juyMGQ40j8sopv4WW0kTWZR98X+nX7lw+sal7qChFE+M6ukGDhX3UeGd+ZXUfpvxohKLJdq6xP5egDCXCzWcS+64kp/n4P7Y5J3HnX6jZBtchcEwnYHGpvYL8qnRqwVlJMq39HMJBrEvshBSm6Qt/dCzebYJYXuVhdxwtbGWxb+w5k6H974MrIzvN+nhL8NxqEcT1isCwS8zLoPjPwhR2O9F3HkTmIE242uGmHOOV0P/xNrZbVYlBdeCPK1f1R5joKQRQGr/bN3hcz+mlN+vXorkMNwMVYu70uhNQSwZpf1g0b6u4zrL8UaoX1lH+ydQNRXkQpMSm3ghkLzHeE9yZrFZxRP3t8vz1DZc5yYbyjiNzejn2w04fBpYjWzDDLIF2y/eh4B9yuwM5xylIQhfyu8BduN2v0VM/gSUtqz0Hc/OTlktfkXnRinMCmKAucNNGIMMRrJ5Jytiva3g/MnafnIGDNfZYDR3skbjOrV1rUmX5cvKEkjAfO8v8Eb4qRK4skyL2XAJgkMibvc35AXeaHsJQ7QCmbFRWTmSbHk9Ce6fI9VlsmWBUf5a34xQA0A++vfJL8G1LlAadcfM38TRiK5VAvs3K8g6fCfOiJXaHCVwFpvlPCOdQU/sKQf0FGdTIw57Ticry37jQq1RvOgoJdJWqlqSyC6a2t3PQgt9PVRJjf6T0tfw== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: e1173529-143d-4778-61c8-08d9e7f1644f X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Feb 2022 15:17:00.8194 (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: VI1PR03MB4703 Subject: [FFmpeg-devel] [PATCH 6/7] avcodec/internal.h: Move avpriv_find_start_code() to startcode.h 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: OoI0nzZRnW9T This is by definition the appropriate place for it. Remove all the now unnecessary libavcodec/internal.h inclusions; also remove other unnecessary headers from the affected files. Signed-off-by: Andreas Rheinhardt --- libavcodec/cavsdec.c | 1 + libavcodec/cbs_mpeg2.c | 2 +- libavcodec/extract_extradata_bsf.c | 3 +-- libavcodec/h264_parser.c | 3 +-- libavcodec/internal.h | 4 ---- libavcodec/mpeg12.c | 6 +----- libavcodec/mpeg12dec.c | 4 +--- libavcodec/mpeg4_unpack_bframes_bsf.c | 2 +- libavcodec/mpegvideo_parser.c | 2 +- libavcodec/remove_extradata_bsf.c | 2 +- libavcodec/startcode.h | 4 ++++ libavcodec/utils.c | 6 +----- libavcodec/vc1_common.h | 2 +- libavformat/avidec.c | 2 +- libavformat/avs2dec.c | 2 +- libavformat/avs3dec.c | 2 +- libavformat/cavsvideodec.c | 2 +- libavformat/mpegtsenc.c | 2 +- libavformat/mpegvideodec.c | 2 +- libavformat/mxfenc.c | 6 ++---- libavformat/rtpenc_mpv.c | 2 +- 21 files changed, 24 insertions(+), 37 deletions(-) diff --git a/libavcodec/cavsdec.c b/libavcodec/cavsdec.c index 692c77eb39..894aa1b54a 100644 --- a/libavcodec/cavsdec.c +++ b/libavcodec/cavsdec.c @@ -32,6 +32,7 @@ #include "cavs.h" #include "internal.h" #include "mpeg12data.h" +#include "startcode.h" static const uint8_t mv_scan[4] = { MV_FWD_X0, MV_FWD_X1, diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 90d667ddc8..8d45d1ceeb 100644 --- a/libavcodec/cbs_mpeg2.c +++ b/libavcodec/cbs_mpeg2.c @@ -21,7 +21,7 @@ #include "cbs.h" #include "cbs_internal.h" #include "cbs_mpeg2.h" -#include "internal.h" +#include "startcode.h" #define HEADER(name) do { \ diff --git a/libavcodec/extract_extradata_bsf.c b/libavcodec/extract_extradata_bsf.c index dbcb8508b0..027a578af1 100644 --- a/libavcodec/extract_extradata_bsf.c +++ b/libavcodec/extract_extradata_bsf.c @@ -18,8 +18,6 @@ #include -#include "libavutil/common.h" -#include "libavutil/intreadwrite.h" #include "libavutil/log.h" #include "libavutil/opt.h" @@ -31,6 +29,7 @@ #include "h2645_parse.h" #include "h264.h" #include "hevc.h" +#include "startcode.h" #include "vc1_common.h" typedef struct ExtractExtradataContext { diff --git a/libavcodec/h264_parser.c b/libavcodec/h264_parser.c index 4002bcad77..50810f1789 100644 --- a/libavcodec/h264_parser.c +++ b/libavcodec/h264_parser.c @@ -27,7 +27,6 @@ #define UNCHECKED_BITSTREAM_READER 1 -#include #include #include "libavutil/avutil.h" @@ -46,9 +45,9 @@ #include "h264_ps.h" #include "h2645_parse.h" #include "h264data.h" -#include "internal.h" #include "mpegutils.h" #include "parser.h" +#include "startcode.h" typedef struct H264ParseContext { ParseContext pc; diff --git a/libavcodec/internal.h b/libavcodec/internal.h index 72ca1553f6..57a1c6206f 100644 --- a/libavcodec/internal.h +++ b/libavcodec/internal.h @@ -285,10 +285,6 @@ int ff_thread_can_start_frame(AVCodecContext *avctx); int avpriv_h264_has_num_reorder_frames(AVCodecContext *avctx); -const uint8_t *avpriv_find_start_code(const uint8_t *p, - const uint8_t *end, - uint32_t *state); - int avpriv_codec_get_cap_skip_frame_fill_param(const AVCodec *codec); /** diff --git a/libavcodec/mpeg12.c b/libavcodec/mpeg12.c index 58e03c05d4..5520960b74 100644 --- a/libavcodec/mpeg12.c +++ b/libavcodec/mpeg12.c @@ -29,18 +29,14 @@ #include "libavutil/attributes.h" #include "libavutil/avassert.h" -#include "libavutil/timecode.h" #include "libavutil/thread.h" -#include "internal.h" #include "avcodec.h" #include "mpegvideo.h" -#include "error_resilience.h" #include "mpeg12.h" #include "mpeg12data.h" #include "mpegvideodata.h" -#include "bytestream.h" -#include "thread.h" +#include "startcode.h" static const uint8_t table_mb_ptype[7][2] = { { 3, 5 }, // 0x01 MB_INTRA diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index 4a7bd6d466..860e86aa74 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12dec.c @@ -34,14 +34,11 @@ #include "libavutil/mem_internal.h" #include "libavutil/stereo3d.h" #include "libavutil/timecode.h" -#include "libavutil/video_enc_params.h" #include "avcodec.h" -#include "bytestream.h" #include "error_resilience.h" #include "hwconfig.h" #include "idctdsp.h" -#include "internal.h" #include "mpeg_er.h" #include "mpeg12.h" #include "mpeg12data.h" @@ -49,6 +46,7 @@ #include "mpegvideo.h" #include "mpegvideodata.h" #include "profiles.h" +#include "startcode.h" #include "thread.h" #include "xvmc_internal.h" diff --git a/libavcodec/mpeg4_unpack_bframes_bsf.c b/libavcodec/mpeg4_unpack_bframes_bsf.c index 6f8595713d..ae2c129d88 100644 --- a/libavcodec/mpeg4_unpack_bframes_bsf.c +++ b/libavcodec/mpeg4_unpack_bframes_bsf.c @@ -21,8 +21,8 @@ #include "bsf.h" #include "bsf_internal.h" -#include "internal.h" #include "mpeg4video.h" +#include "startcode.h" typedef struct UnpackBFramesBSFContext { AVBufferRef *b_frame_ref; diff --git a/libavcodec/mpegvideo_parser.c b/libavcodec/mpegvideo_parser.c index c5dc867d24..22666c85d9 100644 --- a/libavcodec/mpegvideo_parser.c +++ b/libavcodec/mpegvideo_parser.c @@ -23,7 +23,7 @@ #include "parser.h" #include "mpeg12.h" #include "mpeg12data.h" -#include "internal.h" +#include "startcode.h" struct MpvParseContext { ParseContext pc; diff --git a/libavcodec/remove_extradata_bsf.c b/libavcodec/remove_extradata_bsf.c index 1d5f193f89..c698d1d7f9 100644 --- a/libavcodec/remove_extradata_bsf.c +++ b/libavcodec/remove_extradata_bsf.c @@ -26,7 +26,7 @@ #include "bsf_internal.h" #include "h264.h" #include "hevc.h" -#include "internal.h" +#include "startcode.h" #include "vc1_common.h" enum RemoveFreq { diff --git a/libavcodec/startcode.h b/libavcodec/startcode.h index cfa02b0860..8b75832aaf 100644 --- a/libavcodec/startcode.h +++ b/libavcodec/startcode.h @@ -27,6 +27,10 @@ #include +const uint8_t *avpriv_find_start_code(const uint8_t *p, + const uint8_t *end, + uint32_t *state); + int ff_startcode_find_candidate_c(const uint8_t *buf, int size); #endif /* AVCODEC_STARTCODE_H */ diff --git a/libavcodec/utils.c b/libavcodec/utils.c index b19befef21..c7c7323351 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -27,7 +27,6 @@ #include "config.h" #include "libavutil/avassert.h" -#include "libavutil/avstring.h" #include "libavutil/channel_layout.h" #include "libavutil/intreadwrite.h" #include "libavutil/mem.h" @@ -40,12 +39,9 @@ #include "thread.h" #include "internal.h" #include "put_bits.h" -#include "raw.h" +#include "startcode.h" #include -#include -#include #include -#include void av_fast_padded_malloc(void *ptr, unsigned int *size, size_t min_size) { diff --git a/libavcodec/vc1_common.h b/libavcodec/vc1_common.h index b46c33f7e2..c0f0c8c2eb 100644 --- a/libavcodec/vc1_common.h +++ b/libavcodec/vc1_common.h @@ -26,7 +26,7 @@ #include #include "libavutil/attributes.h" -#include "internal.h" +#include "startcode.h" /** Markers used in VC-1 AP frame data */ //@{ diff --git a/libavformat/avidec.c b/libavformat/avidec.c index 86f857b1e3..0f0e6846da 100644 --- a/libavformat/avidec.c +++ b/libavformat/avidec.c @@ -36,7 +36,7 @@ #include "riff.h" #include "libavcodec/bytestream.h" #include "libavcodec/exif.h" -#include "libavcodec/internal.h" +#include "libavcodec/startcode.h" typedef struct AVIStream { int64_t frame_offset; /* current frame (video) or byte (audio) counter diff --git a/libavformat/avs2dec.c b/libavformat/avs2dec.c index 51908d2b63..0d5b23b65e 100644 --- a/libavformat/avs2dec.c +++ b/libavformat/avs2dec.c @@ -22,7 +22,7 @@ #include "avformat.h" #include "rawdec.h" -#include "libavcodec/internal.h" +#include "libavcodec/startcode.h" #include "libavutil/intreadwrite.h" #define AVS2_ISSQH(x) ((x) == 0xB0) diff --git a/libavformat/avs3dec.c b/libavformat/avs3dec.c index 253caa7c1d..2395df171b 100644 --- a/libavformat/avs3dec.c +++ b/libavformat/avs3dec.c @@ -22,7 +22,7 @@ */ #include "libavcodec/avs3.h" -#include "libavcodec/internal.h" +#include "libavcodec/startcode.h" #include "avformat.h" #include "rawdec.h" diff --git a/libavformat/cavsvideodec.c b/libavformat/cavsvideodec.c index 8900b97597..1fa3aa5e92 100644 --- a/libavformat/cavsvideodec.c +++ b/libavformat/cavsvideodec.c @@ -19,9 +19,9 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "libavcodec/startcode.h" #include "avformat.h" #include "rawdec.h" -#include "libavcodec/internal.h" #define CAVS_SEQ_START_CODE 0x000001b0 #define CAVS_PIC_I_START_CODE 0x000001b3 diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 92b4cc8087..971b3f55d8 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -28,7 +28,7 @@ #include "libavutil/opt.h" #include "libavcodec/ac3_parser_internal.h" -#include "libavcodec/internal.h" +#include "libavcodec/startcode.h" #include "avformat.h" #include "avio_internal.h" diff --git a/libavformat/mpegvideodec.c b/libavformat/mpegvideodec.c index 2d6f81aaa1..33c5d79794 100644 --- a/libavformat/mpegvideodec.c +++ b/libavformat/mpegvideodec.c @@ -24,7 +24,7 @@ #include "rawdec.h" #include "libavutil/intreadwrite.h" -#include "libavcodec/internal.h" +#include "libavcodec/startcode.h" #define SEQ_START_CODE 0x000001b3 #define GOP_START_CODE 0x000001b8 diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 5e068c8220..1e87dc6111 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -48,12 +48,10 @@ #include "libavutil/mastering_display_metadata.h" #include "libavutil/pixdesc.h" #include "libavutil/time_internal.h" -#include "libavcodec/bytestream.h" -#include "libavcodec/dv_profile.h" -#include "libavcodec/h264_ps.h" #include "libavcodec/golomb.h" -#include "libavcodec/internal.h" +#include "libavcodec/h264.h" #include "libavcodec/packet_internal.h" +#include "libavcodec/startcode.h" #include "avformat.h" #include "avio_internal.h" #include "internal.h" diff --git a/libavformat/rtpenc_mpv.c b/libavformat/rtpenc_mpv.c index 4b45f51772..c18c75082f 100644 --- a/libavformat/rtpenc_mpv.c +++ b/libavformat/rtpenc_mpv.c @@ -20,7 +20,7 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include "libavcodec/internal.h" +#include "libavcodec/startcode.h" #include "avformat.h" #include "rtpenc.h" From patchwork Fri Feb 4 15:16:48 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 34124 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6602:2c4e:0:0:0:0 with SMTP id x14csp3493825iov; Fri, 4 Feb 2022 07:18:03 -0800 (PST) X-Google-Smtp-Source: ABdhPJxJ2o+xJitiD2mQsd1inxgxyIHTxwq90E3xppDqR3yjG8MnlI0YcIuKIvy8e36b9PVmCY6g X-Received: by 2002:a17:907:2d2a:: with SMTP id gs42mr2982036ejc.106.1643987882924; Fri, 04 Feb 2022 07:18:02 -0800 (PST) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id b13si1466026edd.601.2022.02.04.07.18.02; Fri, 04 Feb 2022 07:18:02 -0800 (PST) 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=EvqdeVdr; 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 A2BBD68B2A9; Fri, 4 Feb 2022 17:17:18 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2060.outbound.protection.outlook.com [40.92.90.60]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 8060368B2A4 for ; Fri, 4 Feb 2022 17:17:15 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NAmuOwBT6wq0Y+hzAtlNpGVlb5HWoswxFCLue0kxzs6rRUNVSpgGwTLMO4jzDsqdrXztV++gkHsOOzjluCWUVYLDhwazhtNjijtfMH1HxA4jotoO8ud+HIj8klA2yuMlYa56Mv932lDjvtnbAAkdOC3QZ+U/WwCK/Kb2K5F9Nqt0mEd1Ekuxt8ivzr9Oisih8gfJ3uKpGxH/inu4W6+TFjv9qqtmCIXmpss8q2e9cZ/GzBLXSIzcMPNMbdfrEXc7JlMBRAQzi3ckBwuwyH9Qde5fYXEFd+4jHC7v0OAFkEDmNz+24heilLRSi4UuuG1/gcl3vJvsMyPhBRpCgGUxHQ== 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=++cncbfRhvkveHkNsfy9lsTDX8SEzuWXoqAySs4LQug=; b=H/rgcGM0tkRb4Qp1i3cbO8gxFfXbnnX341La1G31Jp9LTa4RN29QYXHswO8NcGHPjDiuLNyTqlSda0IGOSAfHN8KF4uWBLsJYTkqMyT4SW68mhUXsfAzSw1Ok7rsQKVJbAYFgjQzfWGLZgsw1i6Dsm9Za+hkO++xUW6IP/rnwyaUB78TghcaDqSL12GZhnjP20Vdfb483uyHcFfuRKdX5D3y6oay8FtwFEF2zWvpcd6P5pEcZ52ZlJJLQph52UXRnYywrs5fCdDXTD2zN+0wmfQPD8aQa55DO5MZqKix/UBvSqY9PbXlwfSSAPZftPjDJukIv94d1jZY2YGE+APj6A== 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=++cncbfRhvkveHkNsfy9lsTDX8SEzuWXoqAySs4LQug=; b=EvqdeVdr1BALPM9MLb7C/l2j0N6gv/0j8wjI327Uf6pCi+pEe15O2QH/iufYU+/S5/f7FMgU2MJFQzVjNP8okmiIOz9+j21NMqlrIF7ZGUKfrD09hvpJdXxhtdAd9zVIxE6dpAQGWH1cDywkGKU43YL3mYcNLVU6goXU/M4O1b9I5WjXEBRQ+QBvVqFz+MFznykKr/fkHJXP4gX3Kjq5mTr2Jlk4cJBAkI8+eDhHM0CodNEAlWaoAOXwadLwxTWSd6Yjq/lFbymx/J8KuWQzaMV2IdowjzMIUqjmit2hWE5MNquwCxFmVpeIynjIyht7TaS4ppROWOGh9waeXVSDGQ== Received: from AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) by VI1PR03MB4703.eurprd03.prod.outlook.com (2603:10a6:803:5d::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Fri, 4 Feb 2022 15:17:02 +0000 Received: from AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22]) by AM7PR03MB6660.eurprd03.prod.outlook.com ([fe80::ac56:2ff4:d304:ab22%5]) with mapi id 15.20.4951.014; Fri, 4 Feb 2022 15:17:02 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 4 Feb 2022 16:16:48 +0100 Message-ID: X-Mailer: git-send-email 2.32.0 In-Reply-To: References: X-TMN: [ukiI95vjBcGmd0DBtAMOympW3+fbTOQJ] X-ClientProxiedBy: AM5PR0101CA0032.eurprd01.prod.exchangelabs.com (2603:10a6:206:16::45) To AM7PR03MB6660.eurprd03.prod.outlook.com (2603:10a6:20b:1c1::22) X-Microsoft-Original-Message-ID: <20220204151648.2148917-6-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: c8a7e3de-0b67-4150-d15d-08d9e7f16552 X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiSjtu7sMDnYCqklgc/dtjmmRjHoIG/rcZfagTGpIJRp8+sAc50Lc2oxGxtuVUDfqVznrylW5apLSEWPKB6pB55UDpBPHaqWtGCdmLixycxSUB0RqHxd7k/pmuJqWnC0P1xt5sl6eA4WcnEeVsPcD1oNhD7Tz2V6layzwARrs622ys8J7zeQbrs+i/lo8gBDDmJYRjt3iaPwVTNL4eeq+guocCT9Qc7zJC3+i/cNpct8LrTC6Ek53NKNgPKfDMn7uZdVU+ejEhBXeswB5N4fOBGc/8WkiIpAUrYvOk8utWNXg9DJrYLDzDyzk4Zqclb4GC5puRa0WxfZA4Lbv+Q++VqpWcNwqRkId00mMjYIjOc1RaJTimb5nsScGqOgIvt3esMqaoNXWxxeCJ5YVrwnPMAwOKjnnfCuFvVeuH87AVOiCy+w5z6dop5JCGb6QBkfForALC4qA6xRpaIryxcV+GrxysILMMNlXKZvGGcsCz7iGcRQj6HTvBf0ZsfE0KPaZRCQc3gxC41E6cd1qfJ8cI3uQsB7DLF3YSnMGwKNgtvexjRcPOBR3csKcZc/AM6crcTKXyAQbD5mS8efldu79ch0zJEm1pNwjqBYZRyK7+CmpBOiykNNY7LnPHhsGRT6o/3YjGN1qmF8Q51lmMF4V59sYf6HfXa2EQUvMzBeYqGpBbd9khYovbbQsRWTvChdsiQKraHLUkTZylI6iE755dHXAPu1iW1ZjJ1/O123wYVthDdgwbXG8w9NKcf2lPjQUog= X-MS-TrafficTypeDiagnostic: VI1PR03MB4703:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: rVaATMqIe7yJiO5yUCRUJtewlxE2DNZ8XZpdEjUqBzUbn75o7V5ee5GyZH2oEN9+rbZQ5fCZS24Nve5w2z68jvSR/lgrETj2ZeH6KmLO+1eyyF10nQ9F9+SOUWNqRWh7tVRRBs7abaSa5UGoAeWy06OnOFr0x8q3GOQdcPGA7u6VwOpdOD7Q6+QkAnHJQqQ8y16kUpZ9Gp8eAnSGYAA7Y7M4u40t9Vo0cSRbzn1f17jCYlFWnb/HDFZoWKqkiRKqYIH5yQjRZSahyGPAgD4ze5AU3T3csmTNQz96x2gqA0Zi69pZgvZ7jAevaXYPdd+wgh1n8Z7153ED1esXcqP8Lsi92duCO23uhPQqVFMVtnKk/d/Zqr/EE0k3A6PUZnap2sW2zEOpNJzGlFgqC28602l87ggLFWwgmC7QvnHxVoz90NYmb4L0PoP271DRr/ci5vikMxN4Z/9mSETfnRFx5B3KdtrlX4GxhLaxPkjSdcs9f9Rv/Px3SIDZgJsxkpFiWxUeL566fI3bCTtrPHq1dFSda0qUptUtKoN+sgmLSM7PIE6DcmlcxprswzxXzQQpgMKrz4SEsLtVgk5ILJj2Bw== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: /tAl/XMMaPRwDyBt6XZEUfsjxknabBOVuLFduaYBgyPdbHcWUhqxl+vcIYPZ/I/YN3mgekqI4PwuGYT3mbTqW9CmAHCncMSXtPEyRgIxdtaHXoZ/eUXHc0TtHEUaNYDQJx30E4kmPWxrSjC05UWiyyfejT6M9lccHWwy4QGCfvSho/sVukRVhBy9o6fpqbkJplbrN9vHPHxc5FmRHQTuEAmpEgFZGpeubWz8KNr7HCnx0sy+tsNpIfa4w9TJ/1EnXO2YvbVlGEu97DTGU9psskVy1+CesE5sCeoIqgsitfGPGi349QJ45gWz1x9vcGlym+GonQW+S8Qi3G1vEBZOZ3i8l17zM0LgJju2RFAac1axOmHcjWpp4kCf1Y4y6Qtd/xoqpoP411T97EJDMWWVv4CUPGF/rSDiWZi/ixHpkxbxLsYhwpk1luRvJgePgAB1ztlCYYA1J/O+tomAe1Op7P4NXeGhaQB2FcN9jupLYyN5B2u9FMlaOfij3BZQgCB9lxQ4PJHz1YCAanD2HdfdwTuf8Ocsp0pn9d//S+07e5tdfnqpEeW7EBjqyGhgwMVUpdRD0d/E2YjQgeo3qpu5IycB/ex1NXkGp+irqlWcKB6zF1QtGmcExvrPrV8yowAwDmhXgPuKLOtgTswp+QLinIf2ZvIsA8Mwq9+7SaxFbelNpBQNu77g1V5/8DGQICpK6ygX1FX2/LGiR+8EPvDdxV5Ppa60S0qkVyi3JMfcxQgSudBauIhlSc/pJ3Wn6b8ALiM3XmhMGpmVGHeVrtVxr34JHHi5lgt4SMiA5hWAtu5SqZYV+1+KBITeZD1E4rWXnzgAkNdj3WDzBnzxqxyNDk7F0R9CB7raLHsOdaB5/i9/00cwdff+hzbdHAZHJ5w/wjnAsyOdonp13aw/q4Kph7Ge1LWbv2wl9gnm84HgjHLW8n8utntCsTnnLW4sN96OaIMpv5aBiFBjSo1nKsonig== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: c8a7e3de-0b67-4150-d15d-08d9e7f16552 X-MS-Exchange-CrossTenant-AuthSource: AM7PR03MB6660.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Feb 2022 15:17:02.4883 (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: VI1PR03MB4703 Subject: [FFmpeg-devel] [PATCH 7/7] avcodec/cbs_mpeg2: Use smaller scope for variables 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: XBIJRMmYioZI Signed-off-by: Andreas Rheinhardt --- libavcodec/cbs_mpeg2.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c index 8d45d1ceeb..dfea12ef1b 100644 --- a/libavcodec/cbs_mpeg2.c +++ b/libavcodec/cbs_mpeg2.c @@ -144,10 +144,8 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, CodedBitstreamFragment *frag, int header) { - const uint8_t *start, *end; - CodedBitstreamUnitType unit_type; + const uint8_t *start; uint32_t start_code = -1; - size_t unit_size; int err; start = avpriv_find_start_code(frag->data, frag->data + frag->data_size, @@ -158,7 +156,9 @@ static int cbs_mpeg2_split_fragment(CodedBitstreamContext *ctx, } do { - unit_type = start_code & 0xff; + CodedBitstreamUnitType unit_type = start_code & 0xff; + const uint8_t *end; + size_t unit_size; // Reset start_code to ensure that avpriv_find_start_code() // really reads a new start code and does not reuse the old