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 36BEE4B325 for ; Wed, 5 Jun 2024 00:31:03 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AF01768D703; Wed, 5 Jun 2024 03:31:00 +0300 (EEST) Received: from BN8PR05CU002.outbound.protection.outlook.com (mail-eastus2azon11023019.outbound.protection.outlook.com [52.101.56.19]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 5896268D6A5 for ; Wed, 5 Jun 2024 03:30:54 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=O3B7AlE5wjFNGGqfN/qlrDI6NUKeW5xEjeKbk7hpZBq/xzPZHSwLVs7X4XpLhYAUae/5pSSlBNvOz/T2/I1bR+8WaHCjA+qnDpHmHDELkY5m/dYO3fc4OOrqPUfgXPxJ2za3IdPH1QrIoh+QVCNRH/SQ3CRBrwHHzadJXMQF9n9Mz0syZxs/wgV5qbG8zgLJoiqfRXBLDxJJucNMD/Iis9MZpdl3CGTdvzcYejGdG3SJ2tUKWUX9x3FUX3tkdZ5zcDrVFGP7vRycqqyMXa46ZwlgDzyBwf6mkx1FyC5cEn64Kz23jj6oOSt1ZArqENkuMLL8LH1r0v9spC9qTyJTmg== 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=y712yidI9VfnDu6FNVpZ8tZKq+EEMbx8Cd+wPomu7QI=; b=W+WMstA7Ayy4BKqBzOO9gEzls+hb/9LTrJ8+t6Xtb1u4/sdRteLk/9Lr7bR5FO705BkS+C697d8/hxRzgBfvuwGjQOwwKWGvznoQI2t0ZzSkLtsItshfymEZBQI+0FtZTT+I3wSwGSs3CG8oUnAZ8ma7AUZwdQrQLe7tMiXPi9KswyS2/u1i9T16iYme5yUSI9GkzPrtBY0WFGOwtsn8BMKkwfYzhnhoLjniOGMBSrSdYmnxognCCCwLyGJd8Utjny5F8QtatDX3/BJ0aqE7ydtMhhpYhUq4pw4TPcHaYg27iDw1+Vrv3m+zQiAdBchS/SBDpEaw+pDyYSbi/E0ljw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none Received: from BN7PR21MB1636.namprd21.prod.outlook.com (2603:10b6:406:b8::32) by DS1PR21MB4177.namprd21.prod.outlook.com (2603:10b6:8:1e1::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.6; Wed, 5 Jun 2024 00:30:49 +0000 Received: from BN7PR21MB1636.namprd21.prod.outlook.com ([fe80::7dc7:a9e6:aed1:4506]) by BN7PR21MB1636.namprd21.prod.outlook.com ([fe80::7dc7:a9e6:aed1:4506%7]) with mapi id 15.20.7677.001; Wed, 5 Jun 2024 00:30:47 +0000 To: "ffmpeg-devel@ffmpeg.org" Thread-Topic: [FFmpeg-devel] [PATCH] Add extra const to FF_PFNGLSHADERSOURCEPROC. Thread-Index: AdqzjFACoILuP5/3TyiHmguB6eVpKwALqNqAAISm21IADT51AAA1h4V4 Date: Wed, 5 Jun 2024 00:30:47 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ContentBits=0; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=True; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Method=Standard; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Name=General; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2024-06-04T23:56:30.382Z; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: BN7PR21MB1636:EE_|DS1PR21MB4177:EE_ x-ms-office365-filtering-correlation-id: d360f949-b0bf-44ac-acc4-08dc84f6bedb x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; ARA:13230031|1800799015|376005|366007|38070700009; x-microsoft-antispam-message-info: =?iso-8859-1?Q?Y9r80FDLcXqFYvYolo/mMawIUFryoSzEXwOqg1nONN1KJ8AjoLMneLNIRR?= =?iso-8859-1?Q?Bq+G5FE+YhKifZtnFFokXxN47k/yjxeTVyKuzWyYpjrSr+PpxJaLfBIr0b?= =?iso-8859-1?Q?+oFGPFxAwHfQs50yS849/Q7SSSOcQKHwgOlzTRiUkTfMbJ52kwaw0iIHNX?= =?iso-8859-1?Q?7J60wvW0aeMeB3U4jC/ZCstuk+ck+NCY5NFJk4F0FuJECIEhfgiVDMMT6S?= =?iso-8859-1?Q?rmtrdQn7BdTqHhZrOxxeRp8DBiGyxWVthEIghx5F4Y12ablO8xonHhAyLp?= =?iso-8859-1?Q?h9fmJgOGakpre1PWesJDwvfm5ut79A3NOfhrojXbz4yOg1b1m1VKEkEh6Q?= =?iso-8859-1?Q?3cWUBDc//BKJrqACWCIOKP83dnkMhlqWMP6OPEp6glbMroO32bJNxrcuej?= =?iso-8859-1?Q?wksW7+t9QCaOJW/CTVlanL7doIUN0/LMSstqP3oyySyFdEEFa7Cs/Bg+Um?= =?iso-8859-1?Q?ZJzZRWU1Jm8jCAMN9V95YWA1MF2B4FCpcSoFmsEz3wyD+xLFJpqAPlwPci?= =?iso-8859-1?Q?9JNAgGS9exXTYv+xCDzdP9CMkz40SVunXJvaZ3+g9WHUcpqGXHTkdn2Oug?= =?iso-8859-1?Q?fg0Q9EQuZUkYKgcViBubYTA4QRZ0JknQETpVDmc+y7t+yBM9zzvkc/L6m4?= =?iso-8859-1?Q?NY8cskDl9AgfKiBNSjaLuC0IuTDhyrE/oWbPYicRQJvjMhUhUxbLpRqFqf?= =?iso-8859-1?Q?AKHhOKqyMq++CUyplMWtZtiPVE+e3CaPApOHQ9MDOtkRx4gTc6O6VB7kXo?= =?iso-8859-1?Q?J+RieoueKkZJODqsIJHUfwqEWl+jsuIETSiOPTi46JNOGtTKt1bt/VM8N0?= =?iso-8859-1?Q?rnUhKS7z+vs03QzKPpQUSf0bk8EKFxhRP/VpqIRjr95GlvmkBExDF1z2Xv?= =?iso-8859-1?Q?EWFaP0ml3zOpTcN76Mch9JCdNk4vR4JgXJT6HsunV+5VVrDXGnySsn0KBj?= =?iso-8859-1?Q?DJmClvxEL713a8ijrgme3Ci1w0nG6Bo8d0M68iU2awOjAnIUqj4LuiBHKk?= =?iso-8859-1?Q?0t8HBEbRlrpTyimZJZQZBiteDGQnlUdpJG8GaTU5Fn5GWVh98va4OcUV26?= =?iso-8859-1?Q?PGrQjYIS7iMvP4gaU57yrctEMiYi6l1XGhO57NSyMZ3Jocg9uA1pRy45+M?= =?iso-8859-1?Q?ZLwAcTcS3G61NrU7r14qJve/oQXaCyVjTWZ43Wlqg8cT35j0O6QpuRURQ4?= =?iso-8859-1?Q?43BICMUCMdoYqj1s5Ly23trdN5I+V92AnH7ti1zdmLSLu5rdqFRCG9HX4q?= =?iso-8859-1?Q?3/W9v6GtiOTd9sG/DSdQ9jELPdzGOTPNTZhzAvWW4gZ9/MIroQgliquTXu?= =?iso-8859-1?Q?ejzIfBl/VSWeHn5fTZfProMfQ2dprd+VUfX6ZPIHL9JkkdvSSYjF2rVjH8?= =?iso-8859-1?Q?1Px7nGpiR4?= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BN7PR21MB1636.namprd21.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(1800799015)(376005)(366007)(38070700009); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?K40xz7F67NVZqwupejLN8hTjNMl7D4QxvrVIUV+VMpuFO8q+XrgXcfE+kB?= =?iso-8859-1?Q?+JjCzaWSuMw4Ifxsm+9i4s5KmqJnhPBl4bq5isZTGBsbF1T58HUdheN+Gq?= =?iso-8859-1?Q?sLpeGlCO77RZDg5HP0hmVDW+f7mvVwRb8jrKaUvm2GiF1042q7+QPXG1VP?= =?iso-8859-1?Q?hkk6m1fmZsoSsF9iq+2lYxomdezN1m9NrqDzgv4W0NAxnzwW0kP7uY4+31?= =?iso-8859-1?Q?iQzfkEH/Zwf/csh3eJCLcqsskTQhr9OmVkfz6FH984BxoMaqwhze5mZFka?= =?iso-8859-1?Q?k9bKIP7x8P7DMBxhAA3Tgvq02bWvBuhREMsb88VQ6ZfzRpJShd+B37yEN1?= =?iso-8859-1?Q?SbRyb+6x0o/kyE+U1fodqwzZVqIpPuzpOJBAnlrOfTw8UDN9XUXHdv85a/?= =?iso-8859-1?Q?4Q0StbxxxjdgRYHe9CYGpw0I453CksvFgfPTapmjBiA9A5WZ3MBnAUANTv?= =?iso-8859-1?Q?zTuNMLYzUt3o4YHKFel6/f+FY8l0THQHt8JvdSMOpSXVZwTD4zBOELSl6S?= =?iso-8859-1?Q?5qRQt/rg5wgynwemqBdI9k/8QoonIj0cg4hXr7I09o8OWba7YnLJYAi1Qe?= =?iso-8859-1?Q?HbzeCdNipitW5Fll3bei4yLhLnmdm/sZaNFNrNkJ5A9oasdgSgX9ItPN4a?= =?iso-8859-1?Q?3G/OyipTV/u222P28C4zsnCKMVs3Y5Z0F5hBhygSkdN0m+Da21WO0kj6Xz?= =?iso-8859-1?Q?JVCAbxg/Oh4pcf46GWrkPIP9TVATlNf5P1LislAoP7XcmCuasuCx4CHNnW?= =?iso-8859-1?Q?Esz3fXq0n4+KlAk/W6YQisRr1LGKw4dKhrHZ9hd1xQSg9FoFIwAjCcRCbg?= =?iso-8859-1?Q?CJ9TbIGp1ZHjZH4008jV6RjiL6XDFQn1HtGZFnUtQtSftNf/NVQpYXYd9J?= =?iso-8859-1?Q?bj4BHH34HI/1RanxM12VytGXbOVABchryJxSYdWe6pFobTXOsgaQ+Zpeae?= =?iso-8859-1?Q?ATyg+dDPj6+2l3ZaQhRrLJEs2BlgVL0gVVqcvO/L2An9TWZdCtVoCePjdo?= =?iso-8859-1?Q?wH1FC3XHYpUm9oqwSoC54yTPDWrBuUtI5KvT4Y1Hk6AlxYQHWH6JmHZeoS?= =?iso-8859-1?Q?rfAlbr12HSnTqL5OYljynbnRLxMDLo/1HbljRH3QM7FV8CfFvWug5H9Z5f?= =?iso-8859-1?Q?ovLBIMADgQrgYGlUiG3HtwnIwlfGLwrwJWWVNQYmmR1+dq4Pl8TWmk4Muo?= =?iso-8859-1?Q?vfcXjMmy6bBi+PWEdhKjbcEQNsl0UtnqtiuGxG5tDS3d6so0UNgA+0l3zU?= =?iso-8859-1?Q?wf6hIoUXb+kP5/KJzKtLgxPMolct0mQOR8V8VcZJd1pODovi7SiMUP9p93?= =?iso-8859-1?Q?g9wlCgnMoQKRE6JS8VzM+0ZCM1C4vWJWgxU+hGte77R/MMdhHTuoGH8jpz?= =?iso-8859-1?Q?9769iVgC8uSbbdkBZjmsp43lZK/ksMXgETyyg7Smb/ZOys2/zixdnn/Zf9?= =?iso-8859-1?Q?+JaPFUsHugUUA8eL3DcdzF9ogBTckC0E0CBlJtwS9mgyBc+8T0rwqKLOFl?= =?iso-8859-1?Q?2RMlxekZIcSOXHEpuplL2dTZPyvx3Qg3jtF3irn50tEOgX6Hrw7Hd25OdM?= =?iso-8859-1?Q?j267MphgpoqK7Z3n1DvkWyQTqg8uaohBrakRgA6JepbBkVsMW/QLPEblXL?= =?iso-8859-1?Q?ITFe3sWh9lPXlV8coi8DstLcbNRyYcIvggV0VLL6JcVXQ3U59PFac0OQD9?= =?iso-8859-1?Q?51qNbAhXSu6n/oWY0EE=3D?= MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BN7PR21MB1636.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: d360f949-b0bf-44ac-acc4-08dc84f6bedb X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Jun 2024 00:30:47.8028 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: hEtvmhE4BQXcF5jjDIRsLnkh4khleOMEyS/0Q2jIazUNz/k2DjLuVi8048lqMmIq3tkYXUtRO6fyMMoiDU4zSQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS1PR21MB4177 Subject: Re: [FFmpeg-devel] [PATCH] Add extra const to FF_PFNGLSHADERSOURCEPROC. 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: , From: "Billy O'Neal \(VC AIR\) via ffmpeg-devel" Reply-To: FFmpeg development discussions and patches Cc: "Billy O'Neal \(VC AIR\)" 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: Makes sense. I'm validating that a patch with the #ifdef succeeds and will attempt to add it as a reply if that works. I apologize for my misunderstanding of the situation, and the horrible Outlook signature. (And in advance if my git format-patch use afterwards accidentally makes a new thread rather than attaching it to this one) Thanks again for the feedback and consideration, Billy O'Neal Visual C++ - vcpkg I believed that void f(GLuint shader, GLsizei count, const char**string, const GLint *length); was compatible with void f(GLuint shader, GLsizei count, const char* const*string, const GLint *length); based largely on that assignment succeeding everywhere, but I was mistaken. See C11 6.7.6.3 "Function declarators (including prototypes)"/15: > Moreover, the parameter type lists, if both are present, shall agree in the number of > parameters and in use of the ellipsis terminator; corresponding parameters shall have > compatible types. OK, so this reduces to "is const char** compatible with const char* const *" 6.7.6.1 "Pointer declarators"/2: >For two pointer types to be compatible, both shall be identically qualified and both shall >be pointers to compatible types. Validation build running: https://github.com/microsoft/vcpkg/pull/39132/ ________________________________ From: ffmpeg-devel on behalf of Andreas Rheinhardt Sent: Monday, June 3, 2024 3:08 PM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH] Add extra const to FF_PFNGLSHADERSOURCEPROC. Billy O'Neal (VC AIR) via ffmpeg-devel: > Assigning the non-const version to the const version is valid (at least, builds on other platforms succeeded). > It is not valid: "If a converted pointer is used to call a function whose type is not compatible with the referenced type, the behavior is undefined." (C11 6.3.2.3 (8)) It will mostly work, because there is no problem from an ABI point of view. But it is nevertheless undefined behaviour (and recent version's of Clang's undefined behavior sanitizer actually check for such mismatches). > Would you still like Apple guards? (I'm hesitating only because most upstreams ask for minimal preprocessor use) > > Thanks for the consideration! > > ________________________________ > From: ffmpeg-devel on behalf of Andreas Rheinhardt > Sent: Friday, May 31, 2024 5:30:49 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] Add extra const to FF_PFNGLSHADERSOURCEPROC. > > [You don't often get email from andreas.rheinhardt@outlook.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] > > Billy O'Neal (VC AIR) via ffmpeg-devel: >> Resolves the following compiler error on macOS 14.5: >> >> ```console >> /Library/Developer/CommandLineTools/SDKs/MacOSX14.4.sdk/System/Library/Frameworks/OpenGL.framework/Headers/gl3.h:1891:21: note: 'glShaderSource' has been explicitly marked deprecated here >> GLAPI void APIENTRY glShaderSource (GLuint shader, GLsizei count, const GLchar* const *string, const GLint *length) OPENGL_DEPRECATED(10.5, 10.14); >> ^ >> src/libavdevice/opengl_enc.c:482:27: error: incompatible function pointer types assigning to 'FF_PFNGLSHADERSOURCEPROC' (aka 'void (*)(unsigned int, int, const char **, const int *)') from 'void (GLuint, GLsizei, const GLchar *const *, const GLint *)' (aka 'void (unsigned int, int, const char *const *, const int *)') [-Wincompatible-function-pointer-types] >> procs->glShaderSource = glShaderSource; >> ^ ~~~~~~~~~~~~~~ >> ``` >> >> Signed-off-by: Billy Robert O'Neal III >> --- >> libavdevice/opengl_enc.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavdevice/opengl_enc.c b/libavdevice/opengl_enc.c >> index 6f7a30ff9e..691bee22ad 100644 >> --- a/libavdevice/opengl_enc.c >> +++ b/libavdevice/opengl_enc.c >> @@ -117,7 +117,7 @@ typedef void (APIENTRY *FF_PFNGLATTACHSHADERPROC) (GLuint program, GLuint shad >> typedef GLuint (APIENTRY *FF_PFNGLCREATESHADERPROC) (GLenum type); >> typedef void (APIENTRY *FF_PFNGLDELETESHADERPROC) (GLuint shader); >> typedef void (APIENTRY *FF_PFNGLCOMPILESHADERPROC) (GLuint shader); >> -typedef void (APIENTRY *FF_PFNGLSHADERSOURCEPROC) (GLuint shader, GLsizei count, const char* *string, const GLint *length); >> +typedef void (APIENTRY *FF_PFNGLSHADERSOURCEPROC) (GLuint shader, GLsizei count, const char* const *string, const GLint *length); >> typedef void (APIENTRY *FF_PFNGLGETSHADERIVPROC) (GLuint shader, GLenum pname, GLint *params); >> typedef void (APIENTRY *FF_PFNGLGETSHADERINFOLOGPROC) (GLuint shader, GLsizei bufSize, GLsizei *length, char *infoLog); >> > > It seems that the extra const is apple-only, so this patch should be > restricted to it. > _______________________________________________ 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". _______________________________________________ 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".