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 B8406485D2 for ; Fri, 8 Dec 2023 18:11:22 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id F390A68D056; Fri, 8 Dec 2023 20:11:19 +0200 (EET) Received: from nef.ens.fr (nef2.ens.fr [129.199.96.40]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id B70A768AC25 for ; Fri, 8 Dec 2023 20:11:13 +0200 (EET) X-ENS-nef-client: 129.199.129.80 ( name = phare.normalesup.org ) Received: from phare.normalesup.org (phare.normalesup.org [129.199.129.80]) by nef.ens.fr (8.14.4/1.01.28121999) with ESMTP id 3B8IBCYO031227 for ; Fri, 8 Dec 2023 19:11:12 +0100 Received: by phare.normalesup.org (Postfix, from userid 1001) id 353CFEB5BB; Fri, 8 Dec 2023 19:11:12 +0100 (CET) Date: Fri, 8 Dec 2023 19:11:12 +0100 From: Nicolas George To: FFmpeg development discussions and patches Message-ID: References: <20231203201027.2255-1-timo@rothenpieler.org> MIME-Version: 1.0 In-Reply-To: X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3 (nef.ens.fr [129.199.96.32]); Fri, 08 Dec 2023 19:11:12 +0100 (CET) Subject: Re: [FFmpeg-devel] [PATCH] avutil/mem: always align by at least 32 bytes 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="===============5730756060014119029==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --===============5730756060014119029== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="lF5xfs20uE5PjwVL" Content-Disposition: inline --lF5xfs20uE5PjwVL Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Timo Rothenpieler (12023-12-08): > Is mixing the libraries really a scenario we need to care about/support? No. We should merge all the libraries into a single libffmpeg.so. Having separate libraries brings us no end of hassle and drawbacks, starting with all the avpriv symbols and backward compatibility layers, and the benefits it brings could be reached in simpler and more efficient ways. But anytime I brought it up, the same naysayers would object, but when I ask what precise benefit they think the current situation brings, with the intent of explaining how it can be done better differently (I do not bring that half-backed, I have thought about it beforehand) or in some case explaining that no, this is not a benefit because linking does not work like that. And then the naysayers would whine that I am making too much a fuss. Barring merging all libraries into a single libffmpeg.so, have configure compute AV_LIBRARY_SIGNATURE as a 64 bits hash of the version and configuration, then have in version.h of each library: #define avsmth_version_check_signature() \ avsmth_version_check_signature_ ## AV_LIBRARY_SIGNATURE() then have avsmth_version_check_signature() in each library call the ones in the library it depends on, and core functions like *register_all() call it. Then if the libraries are mixed at run time it will produce an error message by the linker that can be searched on the web. Regards, --=20 Nicolas George --lF5xfs20uE5PjwVL Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE6ooRQGBoNzw0KnwPcZVLI8pNxgwFAmVzXDsACgkQcZVLI8pN xgzFaQ//T8HtX63lA4p35K1tuLsYmG4RMiUzViaFqKj14vudsWindLdYEuJCupBL 9sZRLji+IQnD8YUdyFALmEkR5pYluGkg0+8BYJtvFCCVwX2nWNJPWulQg3A56Ztw Rqriv0AIqzQ17aUKILjaGOM0OknStYSXs8mRPhUMTwR1lWc+SO8iHlf45z03jgDx Wq0a6M0Bv4TJcWx4e4QQ0B14fhweT7pOlnOL17Y4axtiOhzeUv856CB11TElmfF7 0TEWtUp/oqdR3PhdgQK+R5+HfqdOEur5N2RH9hBflbIjaI6obHPrtp1MxDEhXQU1 4k4LhV0icDZOQzBOCUxOr9Xd3i0lUKByuTBSSimhp9GIEEPxwL8zlRpSeM8vjC0+ zIayfsXPyOTP9BqBM2oJFBTMK/YLUHPgTA1hkBJOkfA/IZQMFtExWDCVUT5zILE1 NSYNg0Ot/0igJigbUxdyDiEvSFNGlEYmoKBVZC/Ujb3gURvCY3H8un3fGmMyptK+ JohLlrwAE0nft/DyLmPAtILBlIlZoUpOHtcvrmLBsNOiJ02fJcL2aYqsMzjysZgt lg3jx8rNd014YbDCjc4WAdt+Jzq1ZyuPixFK9I9ZY/3gKD9d3Mrq73VzceAyEbME ywKUM7wiY2+wOR2T4S4b6lYLa2YWPpc6Rk7Y/wFM0cn5fC5EeJ8= =IdY+ -----END PGP SIGNATURE----- --lF5xfs20uE5PjwVL-- --===============5730756060014119029== 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". --===============5730756060014119029==--