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 747E6431B0 for ; Mon, 22 Aug 2022 22:09:01 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 7326968B9A9; Tue, 23 Aug 2022 01:08:58 +0300 (EEST) Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10olkn2056.outbound.protection.outlook.com [40.92.41.56]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id BC44868B85A for ; Tue, 23 Aug 2022 01:08:51 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Qy8Oyvb/M112PJWq5VlqrakerPYGM+QZj300bs+0pZpe+XPpfK7X7BDdllBTFsEQVrJw8j4RPEiIqc8cmi34IzhmmsIeY5DR4ZhgKEVxT+RBhBTx9kEHld9zzeKrmRiFFY5yI5f/lsPdjaDKbPX23eQYs48fCYoHrrRf4CvO8gCXZx+Ig9AI5Z1HeJpQoSP3zl1kimZn68F22ajFkhlmCKc6JIo3VJ+yOYfSa/e44u+88jYA5y3HCVq2CpnNJWnSf4B4+FmGT1uWCBbgKjZXNDmhayXh/Nd0LVuQxlPv+voBw+WuD/+bIUS5xpJBJ7yy6StjYjmfXYqrtUCzQDLFXw== 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=Xvz9nifN/JZH3M+/lu9BzTNQqJqr9/UXi1CML6ZilRE=; b=AE7O8ZkR42uLL6HFrS9FQ+0Jo5gnQ9tSKoTfjx69jDYSXwkPfJ12zJyxjcORHc0kTsusHWeDBUK8tKQ7RRZrteRLsYMnKxAov2wNnLrcTI6Hk5uuCWX5ibipvNrNs08jqTbn9AASK500+biKvE1bXd/w4BFt0ZhqvdM77HEN9i1LPu4S1N9Z9cdzKUh5iKCz1Ob9/VW1vHY/WgaB/xaTuEcL/ba5wl94Vkq/9fSDtzC3IxX7e2JMyPCLLWbZxOk/X/LuZlU3Z5Ovsn/t/OG3mL6U++2VeKYVuMiJiaER5ofQrIglZFekS/qQGjh9L0KEmepEFJXNWBzW6QPTUL5J2A== 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=hotmail.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Xvz9nifN/JZH3M+/lu9BzTNQqJqr9/UXi1CML6ZilRE=; b=kxH/zdErxlMqgeQ8bHId8BI42O9RMen0Y7Dfn13iYbH0r7FG9EgoVVLKjOm9fmvpACnUO4vJhICF5sPUHAe93J4VswcZLXGqQPIjb9qLF6eHdoDGUGxp7cyA37+b1sV8wN/faWO0P3Q8cZdJntqPV/KaFkMnNoiOadNM5SXPl6pkb0DvBxUbUQHvHJWAA2WjcTExQTqecPrAFuD2JcVuMiRHJj0vfSxrxTW6VLPCaGnq07+2MXXaI0U1jnTRPRwen+Sd6xsNq6e+gKbXk4cy1z26SEjDbGR2ouQKSF+Xn5JNGHlcti8dcp2eZm7D6LuZ/mEKS3oIl5vALVaRn9S/jA== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by CH0P223MB0059.NAMP223.PROD.OUTLOOK.COM (2603:10b6:610:f5::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5546.18; Mon, 22 Aug 2022 22:08:48 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::59af:f946:e59d:784b]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::59af:f946:e59d:784b%6]) with mapi id 15.20.5546.023; Mon, 22 Aug 2022 22:08:48 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022 Thread-Index: AQHYiHoQ/s8zpruhyEis6n+SHo1FCK2qkhHggA7hQ4CAABDRAIABrZcAgACUof0= Date: Mon, 22 Aug 2022 22:08:48 +0000 Message-ID: References: =?utf-8?q??= =?utf-8?q?=3CDM8P223MB0365D91C08ED6BA00BCDE342BA649=40DM8P223MB0365=2ENAMP2?= =?utf-8?q?23=2EPROD=2EOUTLOOK=2ECOM=3E?= <166117073498.25402.3952806733544526391@lain.khirnov.net> In-Reply-To: <166117073498.25402.3952806733544526391@lain.khirnov.net> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: x-ms-exchange-messagesentrepresentingtype: 1 x-tmn: [EVgWKCNH/MomMZfrNmjg6WnauQug/yx2LhF+zpW98blSU1wSJVuC+R7PIW2dxXzI] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 675bd8b5-94f3-4f38-a413-08da848ae3b7 x-ms-traffictypediagnostic: CH0P223MB0059:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: NKtMGPa8D5Ovz3vt00RC+xMIEKgDq2qG8z1xRUraqL6cd5LL/KUfrgV5eKBjVa6wzuXVskQycJT6Kbj8n2ZAfBNcoJQx/5IIC7x+2mCnbjPhD1v6w4KTMAxrh7wa2xbSVqy02uu71vLJ2fEwwEuM9wLmU4T1StlZgjXC0u5sTAIJsJbltQLrGR2HOqJvhVB1F23zZMWyVksRnThJo6/qGynz/JKgiVpIkXaGjf105xn4OtkRLFkZ6LDFHHvXqnQyyxZC1eV/FmJ6Ls21Y/K7aQAzdF5YDGOw7rUjYlO1V71mns/mc21FyxB18pymLtukxVPrVq2XdD+G1wSJcC68teN60nWBYjyYxknFno2axOFeqATWFOCxbqe0KTFhlMBHzsxCZVwLCWDP3PupRZdyUeZ6rZwcobra8Pw4Bex13fu2J2UoxYLeGCsXpOoyq19EaOIgUvPOJr9DLRF+c1vcylA9fcsq1z+ECKR0A4tfqvz0lRBxg/metymdquXW9//hWUhGnI0KfVM535cpNx0no/bcK957Kk9N6eddX3IRvvGv6GNusiGTD1hDMczuZCRcDR9y6YDwZpKRMeqaxc27/qJPvbQmh9Uhmnz4btE1/ToiPdC+foMB3KGIu6GKQMiQZUdUP41SksZKZXjAx//TOyCVFWeDh7Lq0vWeV/gYWBwc1MOcV8cV4kC/qOQMXSxD x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?GoMNqHdoPxfzbkmQeoR/WDKJKlp8WAApEDpKS1vltxjKlqdQAFP1pCRhGT?= =?iso-8859-1?Q?BPAgnHwdih7eNVQ/4WVhchA0hjR/jm/r442ti+hGuH31sDFBPkfXK2qpNC?= =?iso-8859-1?Q?o6E/oGGiFTfctBeA3CYnZ1GqqjFu61gKJVzSzYBiKicL2ynJlROyxg1L4s?= =?iso-8859-1?Q?1BBsubnN4zv2M58yp4SFn47udw62zxfiNHsLnXnbwnI47U+ufCNuVOfSd3?= =?iso-8859-1?Q?kyVtCP9HmzAM2v9OMeHVridVSCPpcfu3Gt+6ucJOdysJMWifBvx/znJolk?= =?iso-8859-1?Q?46tFN0lPVpJoUU4QpFSfg5XTh7VhPfJ/wkHdg09BQF3G9pFupILqD1GUxa?= =?iso-8859-1?Q?iuuN+atPIMtrvNlgvYfFmAWmjvDnOt1qVuM1u4N3vSlGx1SbuCKs3UhJAs?= =?iso-8859-1?Q?9YYIbfSDhQixDTrWUUQk8rLABYYiivTunNA8RD1OaiOhOd4Gy17nifjh8f?= =?iso-8859-1?Q?LBGYyjZrqKJp5xy2UqaWZyjKVlILdl2ww+e/xKFOx+Gsa6GiRsQnu0TvT3?= =?iso-8859-1?Q?y8Mt14GQ3YeOViysCFoNXnR6+8e4iS9Be35X3Waks1Vo38DCJq8M5zoVL+?= =?iso-8859-1?Q?YeGMSkvP20dpod47NanuiUa+vz/ivYT9aWX1uEyk6TaDZTmYOYC65P2VNH?= =?iso-8859-1?Q?XNXpUvp37a/gqflgXI8aiX0PH61hEGQ8zK5dj9eyhFMFg3AvGP01Z2feNg?= =?iso-8859-1?Q?ok1rCq2ezMghqErHLTo+1M14v3GpkREW7n0gWYxJLVu0jfQVigOa2D8afO?= =?iso-8859-1?Q?TXAEH6Pmx0nI5vYWHlVQm98b4wOUjn1sYGzce5CQQn1poH2BVkzqXhw/Js?= =?iso-8859-1?Q?LFMAvb3PUroeX00a4Io647rs8SDl6tWlDyN8rTxrB7nTqV0DcxZhPnbaNX?= =?iso-8859-1?Q?a/5LoHhmVgixOoV4zxDrMCJHq1hQaft79UK6cyqPT1CiJkvdxhVXqxYmpu?= =?iso-8859-1?Q?+nxWQQeZWbVhRkxZcX9Y8w6fsvffx6qAVSL1VDOx5hl9jt3jNV3M/kN4y0?= =?iso-8859-1?Q?0veBLA6EOF3n9NK4/XaDH/3Y2lPher/E4phoN1VwJNSzS1//Fi7nwzkG01?= =?iso-8859-1?Q?SA05vXIeAjSE5RFCIQZkzrF+aYa1XZdDh0lBVEwB9DMyVzrXG+wX5dWq13?= =?iso-8859-1?Q?iB0XViz+5ySeaAm8oy7JEbV5w5Wn1lb864oyqXYu1Q/UQNm2n/gw74cwPy?= =?iso-8859-1?Q?NBJyeBZPIyoontR3o5idlZli63So2Ez4GdDLeDcHS9MtthG4wm9PM6JTjf?= =?iso-8859-1?Q?RYNjs9HvI5XsmAJU87YzD4BkximLgtKIwHgzNmKLNS/6IChy4W6vWuQzqu?= =?iso-8859-1?Q?KxAARVOQm5QVB0UC/lvIq9neJdJRUHH1XaBO0ySoOFaPR7SVWO82LshTmo?= =?iso-8859-1?Q?M6oBSO2Lag?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-1ff67.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: 675bd8b5-94f3-4f38-a413-08da848ae3b7 X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Aug 2022 22:08:48.6694 (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: CH0P223MB0059 Subject: Re: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022 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: ________________________________________ From: ffmpeg-devel on behalf of Anton Khirnov Sent: Monday, August 22, 2022 2:18 PM To: ffmpeg-devel Subject: Re: [FFmpeg-devel] [PATCH v5 00/25] Subtitle Filtering 2022 Quoting Jean-Baptiste Kempf (2022-08-21 12:41:20) > > On Sun, 21 Aug 2022, at 11:41, Paul B Mahol wrote: > > We should more forward and merge this considerable subtitle work > > Are there parts of this work that have reach majority consensus? Almost exactly identical objections to the basic aspects of the API were raised independently by me, Lynne, and Hendrik. IIUC Soft Works still refuses to address them (though it's not so easy to tell in a 200-email thread). --- Anton, thanks for the reply. Please correct me if I'm wrong. From my memory and understanding, the one any only remaining point of discussion was the necessity to have a separate field for subtitle start PTS in addition to the AVFrame's PTS field. I wasn't refusing to make a change, but I have taken a lot of effort to explain the reasons for that necessity. I did that in several chats on IRC, on the ML, and recently, I have written an article especially to address that concern and better explain the background: https://github.com/softworkz/SubtitleFilteringDemos/issues/1 It remained unresponded (but maybe unnoticed?). Bottom line is that without having the separate subtitle pts field, the whole patchset cannot work. @Anton: you said the reason for this is because I would have designed it like this. But this not the case. The reason why this field is needed it due to the way how libavfilter is designed to work. The only way to avoid that second field would be to fundamentally rework the scheduling of frames in filter graphs and making changes to a core part like that (which is working pretty well and reliably btw) would impose a huge risk of regressions and incompatibilities (it's more a guarantee for issues rather than an abstract risk only), which doesn't make any sense to do at this time and in this context. My conclusion is that having that one additional field in AVFrame is by far the better option. Please let me know what you think should be done and what kind of concern you have with regards to the additional field in AVFrame. It might not qualify as a top candidate for "API Beauty", but it's none of the worst ones either. Your preceding arguments were based on the assumption that it could easily be avoided. I hope the referenced article helps to understand why it can't (without fundamental changes to libavfilter). Please also let me know whether there are any other objections or desired changes and I'll try to address them. I am in no way refusing to make changes, as long as they are feasible. Thanks, softworkz PS: Please excuse potentially bad e.mail formatting, I'm off-site _______________________________________________ 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".