From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 6328E48783 for ; Fri, 16 Feb 2024 13:53:39 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 56CE068D293; Fri, 16 Feb 2024 15:53:37 +0200 (EET) Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-he1eur04olkn2039.outbound.protection.outlook.com [40.92.73.39]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id E136568D260 for ; Fri, 16 Feb 2024 15:53:29 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gzyHQmwWcR1tSOhswTPNBg2Lp1xmRXFZTkadiv/op2Bqpt2IpwxwQQuqnD/KA5pW5S5JbykjB23Eil9rJ4OUj8sJvqwLJxFnZBnNMAp2KVBw3zHg2rHSLftrzTk87A7lmLgjEWout64rfUBglPzNOHluywvrDuvYd+ybxv7500GXrYNaTmu7XuaYYXnbitDqEtlRQF/KxCN2S6JkGgHtyHvzn5CKMRliWkiJl1CzEROmpIF20W18jGMq4GWReZyzstduUA0YLSlKBYnChFttgdsTHndpEwd9iauTD0b1k/rFYV3jQDtj21VjcaQvdKGNeLlBzskjaSeYPM85anqNVA== 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=ampxsryTrQ7RLMpuMkwKIx1zHIWN+Q+FQ1Yxq67KMn4=; b=dC+Z5BeiRli4SEkY95u1Y5pcdLbgKwtWIWC9VQFis0cTSlBWOAG19//x+WGVCQZ7LKoTExz5fkdtVj4ftQqqGgirTGTAxlS4Q/jmduo8iNmuUI4jTQt+WLQyT+k52LIIOOtlYBYNXUJv9l6Vs0PNPP7zHASG5XYFvScmem3Vl9YO/S1hPbUOrZMpCUGcgFtmZacPS6ouEgTnfRan58S+EctkqHqhG6ZGxG+oO3dua0n7O9y6o5yABq2H3XW/zuE90yZnx0EAd94y7LTYBSVVfAX8BaPKnRKuiLLivi7AwsjvvQSRfXVI5rZfe8sQ0c79vRkhdZ3FX44k/m3TtMfgyQ== 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=ampxsryTrQ7RLMpuMkwKIx1zHIWN+Q+FQ1Yxq67KMn4=; b=iGzOX7LmSMkVF32DFKw/rOMRUxRysGThaXdI4YoGiDEZnGoxlYRvJEFDr+cHb+HtofBkGTkH2V4Q1cmDW7T2+OXr9tjWSWZdm5LEyO6i6C/1DKdqs6ODQI+qV6O4Ybdl5cTCNhskriPpUWH5HlDv0+etM8y5w6PtinsGwhCSRE474GRBqU6GP6ueDw0pIHPGDbLP7n2xnH4qMRSrLwi0aND3SiVc6OT9Bjw/WDuQiQ0wXcZPAdWxWwR2/+g5YxvoPDqmLwlhbgUReii0209TaH+7cDGApEmVzeVynntGT+Ha51HtG6jgxP2B+CPz+Ha7bLFc0SgCSxy3XP/vbBnwXw== Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) by AM8P250MB0375.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:32a::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7292.31; Fri, 16 Feb 2024 13:53:27 +0000 Received: from AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::65aa:deb0:a18e:d48d]) by AS8P250MB0744.EURP250.PROD.OUTLOOK.COM ([fe80::65aa:deb0:a18e:d48d%5]) with mapi id 15.20.7270.036; Fri, 16 Feb 2024 13:53:27 +0000 Message-ID: Date: Fri, 16 Feb 2024 14:55:23 +0100 User-Agent: Mozilla Thunderbird To: ffmpeg-devel@ffmpeg.org References: <20240123064948.455-1-ffmpeg@gyani.pro> <36aab7a2-3590-4cc1-aeb4-8805d6484de9@gyani.pro> <677047fe-11e4-4a62-a003-912471ff81ef@gyani.pro> <170643927262.8914.6288212134706680203@lain.khirnov.net> <594a2da4-0693-483e-82ab-2924b16d8dbb@gyani.pro> <170799405543.32390.14003661310804711658@lain.khirnov.net> <0c3c8b9b-927e-470d-9272-67536279ba15@gyani.pro> Content-Language: en-US From: Andreas Rheinhardt In-Reply-To: <0c3c8b9b-927e-470d-9272-67536279ba15@gyani.pro> X-TMN: [RcJdHLyLdqT8XTvEa4D0Em2LDWZrIyy6Bq1z6eEVjQQ=] X-ClientProxiedBy: ZR0P278CA0195.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:44::11) To AS8P250MB0744.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:541::14) X-Microsoft-Original-Message-ID: <9832247e-de1b-40bc-b3ba-edc6adec8ca9@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS8P250MB0744:EE_|AM8P250MB0375:EE_ X-MS-Office365-Filtering-Correlation-Id: 45a637a5-8cc4-4b89-2a89-08dc2ef6a67f X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: xpziJocrSvXwwq8ZMW5qeC6lLIaE2CTpuJA8luMkxuBF7Xi7CwNisiW3Z/+zBSYBFynztTNOyugDgRvMaXRzOM1OCdmfle98HqRc2x3kT+ibxRMmrzZ/QyukA90FihN0Ovxv7DAq2ryjmCVfBy7xovbkSUTmTuMRA1x/Om01TNEpvHniF5hnUZHQTye/BHJQkHI0vp3ckE3ycJvQRfPzYqps2lLwzSibEZBmMCqoGBSDIKLG2pUFIA89agA1O5fotrIW2nyzZExToP8H6p+Z1YhIucmcKMCksp0dSCTJkZOQErktj20+e9fuJQCtxi3JXIGWHQ8aluJYWUWVm9yXgyogzhupS+SAu5lVXgY/zJW+Mi6HwnUBRPunAZzI0DryWwEwkzdPmdcDPdVmYvHKl+1XQFIWSanjMQpD0emYrKpn/LXxCGqr8bH6/SU4+xG2G1oBrCQz2q/ff+xST9JGYkqN5EwFWHzmLwW1cHeWfej1/eOLn1Q3XmKw4yGxFH0no6kFA1jJFLIN2IL2o6Y/Gf4qe5P/GnNmwDfmiLE/5LRJcMI819vzxYNpxfCYPGTL X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?M1FEbmxyOERQTklZWmxpUGM4VGc4bGFXOUhQb0dLcmpkRlAxNHE2T1pwWHdB?= =?utf-8?B?N2JVTGVCYTJwa0p5ZGxZNU1heXVxeEU1UVZCMmNGMkIxVXRwQzhQZXlTcXRn?= =?utf-8?B?bWVzSEZqQXpqeDhQYlNzN1FMbm4yS1greEtWNkpud3c5ZjIzWkNTV3ZKRUFl?= =?utf-8?B?bDhiaTUvN3grbjBxbDV6RXBhL1BLTE1YSnNFQnB5ZjNKRmlzWHZVYWVmUTFx?= =?utf-8?B?aWNid3doeUV0K3IycWhjcXlxUXBGTGRhUFRrcFdvWSthQ3pORWJWcWtheWph?= =?utf-8?B?WE1kT1luS3VsWmRIbEpEekorb21VcUhGMFhxYWlVWXlrdklITkRYcm4vZmZK?= =?utf-8?B?Z2VQbmtHUzFOemVFY3BwcHhRcllsSmlRL0hUSVdzbERCczRSWU1RdWxyT0ZC?= =?utf-8?B?WE1TQVVKTzVMTTc3UkRQTzd4Qk50RFB1dU9NVGhPZm84bWJrWXMyTk0ranI3?= =?utf-8?B?ajgvczJGajM2Y0RvQkVQd1dzZzczbUo4SVZmZGgzZFJQdXFSalFQSzVVMlha?= =?utf-8?B?SXBQS1J1d0k1L1VyTVQvQ3p1RkcyTCtRUWxRUzhzVW5pOTZlMTZYR0QzazJO?= =?utf-8?B?ekRrYjhGTzNaZWtIeEdDYTM5dmk1amV5T1JhenM3OFlBQjl3dTdtUG5yVUtB?= =?utf-8?B?SWtKanZQWUR1QUhYSSs2V0ZKVnQ2cFJFWEJIN1VjYjkyYmM2Rmo5T1FEc1M3?= =?utf-8?B?MzE0TDkwTk43STIveHBFQk9lYTNzN3IwTjRLYzNrUWRZMUQwVVhycXhtdG5z?= =?utf-8?B?QXIrblZUK0F1aGlwbmw3MFJENjk1L2YrcFRuRjI4V1g4bmttTjJZbWFORzZP?= =?utf-8?B?bTlGeG1BSG9LK2tvWXYyT3J0dzFJWE5oSnNld1c3elpJZGtGcWRlVlhORTVw?= =?utf-8?B?aEdBS0h4ditOSEQzc0h5VnpycEdVcmxmWGxDTVFYN0ZHNmdnRnlKTU1SVDkw?= =?utf-8?B?Q0VmN1h5aGpOVW9QVUJsbFFNZS9Zc2Z1cnpNcDVmbGJtSk9wdW5Fb0JYcjA1?= =?utf-8?B?T3pqYzVJV1ZKb09xTFZ5WVcwZ2FuaGxmL0l6clM4TWtBNjBBV1FJNGtrRWJ5?= =?utf-8?B?aWk4UUJ1eDVlamRyV29OZ281R1ZrNXQvMWhvYVI1eEthbjVNckVGQmE2TGVE?= =?utf-8?B?NlRVRGhGZ1pnOENxUWdNOVA1UzI0VEV2QktDTXhRakJwRVVJWnh3NlZoOVB3?= =?utf-8?B?R0F0Qmp0RkFkeFREc1ZTK2ltWFI1dUVrYTllZG5xUkhWOHJjcTJTc05TYnM4?= =?utf-8?B?a0hIdlExeWVJQmR0RWFQdVU3eXdFblltMUU0YjBDSGpXeU9IUlE3UXEzcHVi?= =?utf-8?B?aHNleThoQXUyaTBTbVhLanZSTEEwdm0ySzBySmFaTlNxQmljbFZVaXZmVU0w?= =?utf-8?B?LzhBOGtoRW8weWVDTkdWTENNZlRQNHNQc2ZqdVlLcWxFNElQQWNhSTlENzBq?= =?utf-8?B?TWFRcm00Rzk3OUJHUy91OXY0Y3dKZ0NRYnBkWnpOL2JCd3c2UFJZSnl3Vi83?= =?utf-8?B?Tmw2RmJQTmVjQ1E4RWZ4amZqaVFMaEN3TGNYQ1BjSm56MkhVWUx1UG1SdnY2?= =?utf-8?B?TnA4d2NYWGt1R1lZWDYwNTBLY051a1lKczRtMkplSThtVGlWNExrYVJlMExG?= =?utf-8?B?QzExa1VROHVKbHNTSXFBVTlaelN6SVQ3VmlBOWdGeDA0NndPOXpncFRiUW1Q?= =?utf-8?B?eFNvYnJxc2VyRFNQKzExRTJ5RjFlKzQ1RXdUY0ZKRDZ6Sm5OWkNuVnJnPT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 45a637a5-8cc4-4b89-2a89-08dc2ef6a67f X-MS-Exchange-CrossTenant-AuthSource: AS8P250MB0744.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Feb 2024 13:53:27.2657 (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: AM8P250MB0375 Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/s302m: enable non-PCM decoding X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: Gyan Doshi: > > > On 2024-02-15 04:17 pm, Anton Khirnov wrote: >> Hi, >> sorry for the delay, I've been busy fixing things for the release >> Quoting Gyan Doshi via ffmpeg-devel (2024-01-29 05:00:33) >>> On 2024-01-28 04:24 pm, Anton Khirnov wrote: >>>>> a) it would mean essentially inlining this decoder in the demuxer. >>>> Why is that a problem? This decoder seems like it shouldn't be a >>>> decoder. >>>> >>>> I agree with Andreas that this seems like it's a demuxer pretending to >>>> be a decoder. >>> This module transforms the entire raw payload data to generate its >>> output, even if the syntax is simple which >>> essentially makes it a de-coder. The de-multiplexer aspect of multiple >>> streams is an academic possibility allowed >>> by the standard but not seen in any sample which makes me suspect it's >>> used for carriage between broadcast >>> facilities rather than something ever sent to an OTT provider, let alone >>> an end user. >> If it dynamically generates nested decoders, then it's not a proper >> codec in our model. It should be either a part of the demuxer, or a >> bitstream filter (possibly inserted automatically by the demuxer). > > s302m is a hybrid creature and does not slot cleanly into any role. So > there is no theoretically proper place for this component - any choice > is a least-out-of-place accommodation. > > But it is much more out of place inside a demuxer. Analyzing packet > payload and then manipulating that entire payload is much closer to a > decoding role than data chunk extraction for packetization. And the > stream extracted from the container is meant to be SMPTE ST 302 not PCM* > or Dolby-E/AC-3..etc, which will both misrepresent what the container > carries > and possibly discard S-ADM metadata, if present, in the packet. With > passthrough demuxing, a stream can be mapped for both decoding and > streamcopying. This is not true: It can not be streamcopied into formats expecting ordinary PCM or Dolby-e/AC-3. Which is why exporting the data without the unnecessary packetization is preferable. > > A bsf in principle would work but in practice, can't as Andreas > clarified that bsfs can't set or alter codec_id after init. And > resetting the codec id requires packet inspection. > > Nested decoders are used without issue in components like imm5 or ftr > (upto 64 nested decoders!) among others. There's no breaking of new > ground here. > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".