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 E8D3747C4F for ; Sat, 14 Oct 2023 17:00:47 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id A668068C6D1; Sat, 14 Oct 2023 20:00:44 +0300 (EEST) Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1A6D768BED8 for ; Sat, 14 Oct 2023 20:00:38 +0300 (EEST) Received: by mail.gandi.net (Postfix) with ESMTPSA id 2E2CC240005 for ; Sat, 14 Oct 2023 17:00:36 +0000 (UTC) Date: Sat, 14 Oct 2023 19:00:36 +0200 From: Michael Niedermayer To: FFmpeg development discussions and patches Message-ID: <20231014170036.GV3543730@pb2> References: <20231013191934.GQ3543730@pb2> <20231014003409.GB5462@haasn.xyz> <22919b35-2f3a-494f-8f34-e40e628263f2@gmail.com> <20231014005457.GB123737@haasn.xyz> <8A960BE2-8364-4AF8-A9B5-E0551C19F9DF@cosmin.at> <0101018b2b540db8-9535df37-83d6-44fd-8a1e-a5fd99185315-000000@us-west-2.amazonses.com> MIME-Version: 1.0 In-Reply-To: X-GND-Sasl: michael@niedermayer.cc Subject: Re: [FFmpeg-devel] SWS cleanup / SPI Funding Suggestion 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: multipart/mixed; boundary="===============6491791111974529096==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --===============6491791111974529096== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="1Ot6giJwhtzp6kCs" Content-Disposition: inline --1Ot6giJwhtzp6kCs Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Oct 14, 2023 at 03:19:49PM +0100, Kieran Kunhya wrote: > On Sat, 14 Oct 2023 at 00:17, Cosmin Stejerean via ffmpeg-devel < > ffmpeg-devel@ffmpeg.org> wrote: >=20 > > > > > > > On Oct 13, 2023, at 4:00 PM, Vittorio Giovara < > > vittorio.giovara@gmail.com> wrote: > > > > > > TBF this is in part why i was suggesting a new library - I feel like = sws > > is > > > affected by bad brading because of these caching issues and imprecise > > > conversion, and a new clean api in a new library would make a lot of > > sense > > > in my opinion. > > > > I think the branding issue would solve itself in short order if the act= ual > > implementation of swscale started to be good. My concern with adding a = new > > library is that we'd end up in a situation where we have both swscale a= nd a > > new library side by side for some extended period of time. > > > > By comparison adding cleaner APIs to swscale and then slowly strangling > > the old APIs (along the lines of Niklas' proposal) would allow for a mo= re > > gradual transition that has a higher likelihood of success compared to a > > full rewrite IMO. > > >=20 > The issue is not the API, the issue is that swscale is astonishingly > complex and difficult to understand internally, there are lots of differe= nt > codepaths > and randomly you'll end up with a buggy or slow one randomly ? code in general doesnt give you randomly something very different. So, why do i complain? because swscale has real issues and needs to be improved. And these comments point in the wrong direction > and have no > idea how to fix it. If you dont know how to fix it yourself, sending me a bug report is probably a good start. >=20 > It's probably easier to start from scratch than to try and understand and > then fix swscale (years of work). Well there are 2 further aspects with that. The first one is bluntly put. If you dont understand the old code, then you probably are not qualified to write better code. People tend not to successfully improve things they dont understand. The 2nd issue is, ATM, i maintain swscale. If iam involved in the new effort and understand it either because of that or because it has some similarity then i can continue to maintain swscale. If its totally different and i was totally not involded then i also will not maintain it obviously. This is something to be especially aware of in case the cleanup/new code would be done by someone who comes, does it and leaves. you could end up with nicer code thats then unmaintained. PS: whats the real issue with sws ? it evolved out of a piece yuv->rgb converter from a video player. It evolved from that and stuff was added into it. This is a similar situation to why ffmpeg.c needed cleanup thx [...] --=20 Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Dictatorship: All citizens are under surveillance, all their steps and actions recorded, for the politicians to enforce control. Democracy: All politicians are under surveillance, all their steps and actions recorded, for the citizens to enforce control. --1Ot6giJwhtzp6kCs Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABEIAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCZSrJMAAKCRBhHseHBAsP qwmNAJ4i4OpfZQVKuzKkJTcOhzfhripTMwCeL0I0d7El12IMnGaCwhR0l4nSUNw= =jL/F -----END PGP SIGNATURE----- --1Ot6giJwhtzp6kCs-- --===============6491791111974529096== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ 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". --===============6491791111974529096==--