[Webkit-unassigned] [Bug 229337] GLContextGLX.cpp may sometimes fail build because of X11 include files

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 29 13:53:17 PDT 2021


https://bugs.webkit.org/show_bug.cgi?id=229337

--- Comment #12 from Fujii Hironori <Hironori.Fujii at sony.com> ---
WebKit has a lot of #undef in WTF and WebCore layer for this issue
For example, https://github.com/WebKit/WebKit/blob/bb27304754d0bb686fc3ddfccc42fb8ec1f14d62/Source/WTF/wtf/Threading.h#L64,L68
If you can solve the issue completely by @no-unify approach, we no longer need the #undef.

I think @no-unify approach can't solve the issue completely  because #include order is ruled by WebKit coding style.
If a header defining None or True are included after X11 header, it can't compile. We need #undef. @no-unify approach isn't a good idea.
I think we should add X11Headers.h to include all X11 headers we need.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20211029/c1a8c148/attachment-0001.htm>


More information about the webkit-unassigned mailing list