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 B42F444CD0 for ; Mon, 14 Nov 2022 22:50:08 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 30A8A68BBF2; Tue, 15 Nov 2022 00:50:06 +0200 (EET) Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10olkn2062.outbound.protection.outlook.com [40.92.41.62]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1B77F68BAEB for ; Tue, 15 Nov 2022 00:50:00 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nEmVXB3//rwM70ITbjSV0U4jP/ybQWE77AutJsWdevBB6/SYgNhEhiyYhQDH31E/QnnoOzlxNBNQWR+9EgTm0kXjVhlHshrp1pdiT4xDIWu3I1DSrrO+eDfrfVQLAYYZzMf0CtLqX7rEVpBRXyfonLG+bCNbBX9u48hyxodATkqMvBP9Yfmoq6bdnb5j9Fr76NiQhjVwRizUhEYg/AxAMQoijtw3cTh6m+qjvmG/36ZpGlaDnED9fqQ948ffhIjZA/pU5UAjl17auikiRQRe2vKX91wPKeRzzYAvWYmxBhEyeYAAwaX5p22TkuaisG6l1i+oVZnkE077zUdgAhnIrg== 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=IXYjL9o+b9r7E07sEEzZVZdupL/Xv5sgitfi3f6v1Hg=; b=NiwuW9GsMDYO1sCSlzV4b1m5nZu0eP2TiMFzMnzXxQ+2w+ZectdzhyZIU1lUrOYvabwCejHtSa+ljRiYKeidTTqqPRgu++k7twTXZu9su6NLUDu5mGbFw1OA9ljD1lHEcDETsWe4uun3af3WhYE04q21TeJbvWe/jxNLNjxzO0v2krOUXklDJjAlCPlJSi+NbA/sJedUnN2fCmr+j0Lko44i1ehCWPvYQpXRGnUxJ1flYOR/iMg7aHgmm0xEtQuThqQmL4Ro/B73qsNGcrZXp3E6QGz6XSoMzGGXtlMnMz0v7efjUNX4ZMcBkRp/MU0sOK75quCSpD8RZBPrr4qNJw== 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=IXYjL9o+b9r7E07sEEzZVZdupL/Xv5sgitfi3f6v1Hg=; b=d1Gg9l5p6pOVcuW/GlWRRYnS9nCv2pRThmOlNRNy5T3MdeeeXWhO+4rsQCv8fKX7wOvYk5RdpYZutvblbMIqkI7+gMd155u11WLw47kgkDXyLLPxSGMcNRE+xM1lnTLzCVcAXA6gXQooKUSXG6EQSl66fN44vA/5UPFBz01U8Po01LVIzkntTBpLuDvu4KztrdWMi+R1uFs5zc7xxNw8IjPleLVgrEg4sr9buhTZr4/JBVssOzqH1ik3qGQ8Oj8lTG6zNcoKY96drhbCRpN7U/mkqXnIFMv6mgBidGpc13pB4CBIhkdYvtHUMlPHEM636hGqHLbrjbjFSwXdLPA5oQ== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by SA1P223MB0677.NAMP223.PROD.OUTLOOK.COM (2603:10b6:806:25c::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5813.17; Mon, 14 Nov 2022 22:49:56 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::45ea:be80:31df:38c0]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::45ea:be80:31df:38c0%9]) with mapi id 15.20.5813.017; Mon, 14 Nov 2022 22:49:56 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH 2/4] doc/developer.texi: extend the argument for submitting patches Thread-Index: AQHY+AlpczOesuRDPEqrzDs8qAxq1a4+QqZVgAA55yeAABuX+YAAYkcAgAACYUA= Date: Mon, 14 Nov 2022 22:49:56 +0000 Message-ID: References: <20221114091309.15755-1-anton@khirnov.net> <20221114091309.15755-2-anton@khirnov.net> <166842406666.1198.894449822855148822@lain.khirnov.net> <166843649962.26119.12307610011247756962@lain.khirnov.net> <166844242020.20155.16539393327108846682@lain.khirnov.net> <20221114220546.GA1783292@pb2> In-Reply-To: <20221114220546.GA1783292@pb2> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [P/H3Ie56D5clg5zKwIlG5MVezJwvbmwc] x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM8P223MB0365:EE_|SA1P223MB0677:EE_ x-ms-office365-filtering-correlation-id: ca1f5921-6fed-4920-e38d-08dac6928d53 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: n8J9u1e9VklleqRLWkuxo27gXgdOo+i4ER9asok0uR3Nb0PQTGKZoj0zoYNhjGbRJwE0IIQz9bAk4y+E2OsxHcCQReXQZH2//l6/24C1Er+eLYGFyXjS4StFAoUtlmMKAp9gLZPAvNmI2XD1vVCqzGhnm7LWy4K1aqoQ9RlE2MNFnv5MMnmFxopV6JbHmbRBT07ATVc93ff0WiNHRASF6PeOOq5qecWwsixROO1pA38U4LB1D2wanOywNn2gdiHYQ8D9qrzkTQGF1IbQTCGiHAceiENphi5ljrXVY57A7Qj6IbO9dTB88ElQY6fS1F+mesPopPLKrVj0/6dvaN7+5qvksDNd9k39RuD/BypYc9eekwmbM0yR9FFlSFfM9OSzIOQzzQpuBsI5iZz+NI3HPoKObQVK0tVu0m3gkOaFuQyj6xezWSo5yLgprlgWC+vBkl2RO071c5TUfKbyp8IjnrsZTSH2Y1aGyUkUQHwVHI+fEI9bF2aUMexrFasrcJHr5bHfOzcUPphiDxubcY4PM2euvIqTi+eLD0QHtvTjBoDQHJJpfmpXaRDCvG+q7vUXLfskmYBTjN2hLk4qAQLqi0amvPhGgYDcSUH8/pdPCc7+J4CVnMr+mkzvGR7O6eeOb4YF4xkD28jK2CVC69FYRYzDr6Ygs+8nteMhzqZX+ba6tzMnYxxZ+vJ+58okZf/Dj9ETyQdaTWUAWYd+kRalaeuKI5mCQWoawlfCKA3HDbI= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?S1k4U1hkWVVqQ0pTVFg0NkVocEd5cVdCaGFRZXhnV2MvTHluT1BBL0J6OFdu?= =?utf-8?B?Z3NmM3R2NjVxVkJ2Wmd3eVcxNTJlM0hkaFNzbjRDNDhtd3BjUkQyemlwYnZ2?= =?utf-8?B?a2x3MTlVdSsyOVIzNWpQR3pCdGJkT1hTbDArNStuWDF3cTlhQ2QvL3p5UGZT?= =?utf-8?B?WUxVbTh0U2VPMjZQb3VzS0hJUDZOdUxEQXF0azZucEM5d0RyTndUVllrdTVn?= =?utf-8?B?VDRuZUpmOHBROGo3Yy9MMHNiRlQwd09qc3h5SzhqajZmblZHWDI5YVNnUXdK?= =?utf-8?B?WEc5Y1VqTTBzSWRnNm1pZkwrNWdScE9wa2g2SzZ2bTYvM2JPcWJrNS92aWlv?= =?utf-8?B?cWZqalRDcUxTb1UyVGcrSXhZcExtck9lbnFpTDVNZ0lWbHhBM0NvcUNJcEg3?= =?utf-8?B?VkVDRzRoVzRhdWtXcFZYaGlaUC9nM0JUVytxblVtSVBMMGdqeGNvVHM1ZUpY?= =?utf-8?B?aEJYaGNNVHlhc0RzQmRLeWM4QmhEclVSWk5FSTJEbFlUcTVCTk9XSTYrZWxi?= =?utf-8?B?SEo0aTZTYm05T2NTLzB2SGFTUFJycXFON3NYNGM1aVZqSWRFZ2pFcUhlQWR6?= =?utf-8?B?QzM1NEt4NEE5Y3plL0dXUjFTK1prN0Fvc3pXZXg4aUpSRHhWMFRwcXY5bjA5?= =?utf-8?B?Z1NuSFpQRjh2TEhSMy9QbmVmZHNEeTBPOTIvdElERjUxV0R5c1JJTGREeVdC?= =?utf-8?B?bStVN1REWDlvUHhCSGtKTmN5RzRqdWtsMlJMRkNxTUxhU1FJQ1RYUzNadmhn?= =?utf-8?B?MGRuUXEvTFB1WXdVa3NKcXBZSzRUaHdLSDJXQi9kN0xFREl2ZHRiYXNjOXdu?= =?utf-8?B?dnVaNzZ4QzBBYVlZRGpZdUZFbXdPVE03YUpMZjJDRi93VlZ0R1Fhc1QwdWJ6?= =?utf-8?B?T0YvQm1naWJWNXRBY3BweUJJUXNyY0xKTTdoZThVN3JCVS9jSStWY1pEUXlk?= =?utf-8?B?NDA5dFJ2T0k2ODVjTjNnb3A3bnhnTWZ6VHFkNXFaTXE4MlIxaGt6ZE5qdzRI?= =?utf-8?B?dFp5SFN1TnBCeEtON0luSU1MSURCWW92UW03VTJXVDBWakpPVkt0dzhiRmdt?= =?utf-8?B?Undmc2Nuc24rYVNMY1B1N3FkUTE3NnRZekcxTWFOdjNZVDRPVHgzU3BXTVJu?= =?utf-8?B?QzFqMHBLcXNJVFJNczVQS2ZpN0pXRWtLalpxV2EwMlZmeGFNNG9mMVUzZ3U0?= =?utf-8?B?LzJJcEYyTDljbFM1RjhqM1ZBeitsVnBKODRPTjM0NnpUb2J3OTdaZUxDNlV5?= =?utf-8?B?VVRINVFDKzlJMzJxMUh2WlQvcklPU0hrRzhtVHNtNUtzcFJaTGJ5ekhwTFNQ?= =?utf-8?B?amRaRHBEZnRad0crZEp2VlIzcGVGbldoRXBEZ3c1ZWxJUW0vZUF2RitHN1Zv?= =?utf-8?B?Sy9UcnVoMFMyQ3BEYnlYQmxXRFNtdzJSaE5ZdzFLR08zdDg1L2JqeG1uaVl3?= =?utf-8?B?RzJ4Q0JCSklnbzJNdkR3L3pVMmMwL2lobGdaaXlXN1A4c1dGVEFjYWhJNzhX?= =?utf-8?B?cm56cHZQWDI0VDNBRXlDRWlpTmFoSHlqZlA5UGJVL3prL3BGTUhEcXV3VkNW?= =?utf-8?B?a2VWUmMrVnNMVlF1ckJwcHNIL2oyV1NobHVSRWZMZ1dreDg5b1JsUytxUkFv?= =?utf-8?B?V3l0eXRDMmYwUnc4d0t4Rk1WSXhpTWVGdlIxbVB3OHBPN1NMcTh1dlpmclFW?= =?utf-8?B?V1pFWWkvNStlTTB4SHVDZS8rUnlDNW1OT3hTcndmWk43U3M3dGlEV3pnPT0=?= 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: ca1f5921-6fed-4920-e38d-08dac6928d53 X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Nov 2022 22:49:56.4645 (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: SA1P223MB0677 Subject: Re: [FFmpeg-devel] [PATCH 2/4] doc/developer.texi: extend the argument for submitting patches 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: > -----Original Message----- > From: ffmpeg-devel On Behalf Of > Michael Niedermayer > Sent: Monday, November 14, 2022 11:06 PM > To: FFmpeg development discussions and patches devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 2/4] doc/developer.texi: extend > the argument for submitting patches > > On Mon, Nov 14, 2022 at 05:13:40PM +0100, Anton Khirnov wrote: > > Quoting Soft Works (2022-11-14 16:13:29) > > > > I did read your document, and my takeaway message from it is > "doing > > > > it > > > > properly is too hard". As long as that continues to be your > position, > > > > you might as well not bother. > > > > > > This is ridiculous, and you know that. Or at least you would know > > > if you would have really tried to understand the problem. > > > > > > And that unfortunately applies to some others as well. Nobody is > > > willing to go deep enough to the point where it becomes clear > > > that a "perfect" solution would only be possible by making > fundamental > > > changes to libavfilter, which are complex, risky and something > > > that would never be accepted from me, even when it would be > > > the most excellent solution. > > > > Stop with the drama, please. You are not a persecuted misunderstood > > genius. Nobody here has a personal grudge against you. The reason > > people, including me, are objecting to your patches is that they > are not > > fundamental enough. You want to redo the subtitle API in a major > way, > > but keep it saddled with legacy hacks right from the start. We have > > enough of those already to know we don't want any more. > > Maybe people should take a step back and together discuss with > softworks > what changes need to be done. > > This thread is a bit odd becasue from reading just it its very clear > there > are objections but what exactly needs to be done to move this forward > is > not so clear (to me at least). To me neither. Until today. > I think a patch review should result in a clear list of things that > need > to be done. > Then these things can be gone through one by one. What can be done > what not > where there is consensus, where not and why ... > > If a request to a fundamental redesign is there then thats what it is > and maybe noone will do it but IMHO it should be clear so everyone > understands > what is requested > > As it is, this thread simply makes me sad because its deadlocked, > there is > some will and effort on one side and that isnt directed into anything > that > goes into the ffmpeg codebase ... I had a friendly chat with Anton on IRC about it. Bottom line is that he insists that I re-work libavfilter filtering from the ground up to support non-monotonous pts values in graph processing. The reason is that the single time_64t field that I want to add as a member to AVFrame would not be acceptable because it would make timestamp handling "too messy". Other objections haven't been made. I asked multiple times whether he would be serious about this, demanding me to take on a possibly multi- month work for the reason that having a single additional field in AVFrame would be messy, which he confirmed. I said it would be a huge amount of work and too much for me to take. He doubted, saying it can't be that much, I asked what he would expect me to change and how: Nov 14 19:08:02 i can't say what to change exactly without actually doing it Nov 14 19:08:22 i didn't write that code, I only have a very rought idea what it does Full transcript on request. I got no more words. Except about the irony that reveals when looking at the subject and who wrote it. Thanks, Michael! softworkz _______________________________________________ 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".