Date: Sun, 21 May 2023 09:14:02 +0000 From: bugzilla-noreply@freebsd.org To: x11@FreeBSD.org Subject: [Bug 271533] x11/xorgproto: ensure Xos.h does not define index() as a macro Message-ID: <bug-271533-7141-90LU0LRrX3@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-271533-7141@https.bugs.freebsd.org/bugzilla/> References: <bug-271533-7141@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D271533 --- Comment #3 from commit-hook@FreeBSD.org --- A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=3De5a0215417930a60ac8fbbc630fad6a= a354446e5 commit e5a0215417930a60ac8fbbc630fad6aa354446e5 Author: Dimitry Andric <dim@FreeBSD.org> AuthorDate: 2023-05-20 15:15:41 +0000 Commit: Dimitry Andric <dim@FreeBSD.org> CommitDate: 2023-05-21 09:12:30 +0000 x11/xorgproto: ensure Xos.h does not define index() as a macro During the exp-run for clang 16, it turned out that Xos.h defines index() and rindex() as preprocessor macros, and this causes many problems with libc++ headers [1]: /usr/include/c++/v1/variant:532:39: error: too few arguments provided= to function-like macro invocation return __at(__fmatrix, __vs.index()...)( ^ /usr/local/include/X11/Xos.h:67:12: note: macro 'index' defined here # define index(s,c) (strchr((s),(c))) ^ /usr/include/c++/v1/variant:813:18: error: too few arguments provided= to function-like macro invocation return index() =3D=3D variant_npos; ^ /usr/local/include/X11/Xos.h:67:12: note: macro 'index' defined here # define index(s,c) (strchr((s),(c))) ^ /usr/include/c++/v1/variant:817:25: error: expected ';' at end of declaration list constexpr size_t index() const noexcept { ^ /usr/include/c++/v1/variant:802:9: error: member initializer '__data' does not name a non-static data member or base class : __data(__tag), __index(__variant_npos<__index_t>) {} ^~~~~~~~~~~~~ /usr/include/c++/v1/variant:808:9: error: member initializer '__data' does not name a non-static data member or base class __data(in_place_index<_Ip>, _VSTD::forward<_Args>(__args)...), ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /usr/include/c++/v1/variant:920:23: error: too few arguments provided= to function-like macro invocation __rhs.index(), ^ /usr/local/include/X11/Xos.h:67:12: note: macro 'index' defined here # define index(s,c) (strchr((s),(c))) ^ Add defined(__FreeBSD__) to Xos.h's #if block to include <strings.h> instead, so it will get our regular index() and rindex() declarations instead. This has also been sent upstream [2]. [1] <https://pkg-status.freebsd.org/gohan05/data/mainamd64PR271047-default/2023= -05-14_19h35m09s/logs/errors/scim-1.4.9_7.log> [2] <https://gitlab.freedesktop.org/xorg/proto/xorgproto/-/merge_requests/73>; PR: 271533 Approved by: manu (maintainer) MFH: 2023Q2 x11/xorgproto/Makefile | 1 + x11/xorgproto/files/patch-include_X11_Xos.h (new) | 11 +++++++++++ 2 files changed, 12 insertions(+) --=20 You are receiving this mail because: You are the assignee for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-271533-7141-90LU0LRrX3>