It is this time of the year when X11 macros leak everywhere again.
Created attachment 423888 [details] Patch
EWS fails: ThreadedCompositor.cpp:89:51: error: invalid ‘static_cast’ from type ‘uint64_t’ {aka ‘long unsigned int’} to type ‘GLNativeWindowType’ {aka ‘void*’}
Created attachment 424013 [details] Patch
Committed r274869: <https://commits.webkit.org/r274869> All reviewed patches have been landed. Closing bug and clearing flags on attachment 424013 [details].
(In reply to Philippe Normand from comment #2) > EWS fails: > > ThreadedCompositor.cpp:89:51: error: invalid ‘static_cast’ from type > ‘uint64_t’ {aka ‘long unsigned int’} to type ‘GLNativeWindowType’ {aka > ‘void*’} I'm getting a similar issue building wpewebkit (main) for ARMv7 (raspberry-pi using mesa driver). Based on https://bug-178090-attachments.webkit.org/attachment.cgi?id=323356 I have created a patch for this in https://bugs.webkit.org/show_bug.cgi?id=223684.
Comment on attachment 424013 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=424013&action=review > Source/WebKit/Shared/CoordinatedGraphics/threadedcompositor/ThreadedCompositor.cpp:94 > +#if CPU(ADDRESS64) > + auto windowType = reinterpret_cast<GLNativeWindowType>(m_nativeSurfaceHandle); > +#else > + // On 32-bit platforms GLNativeWindowType is an integer type, which cannot be casted with reinterpret_cast. > + auto windowType = static_cast<GLNativeWindowType>(m_nativeSurfaceHandle); > +#endif So this doesn't work for you?
(In reply to Philippe Normand from comment #6) > Comment on attachment 424013 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=424013&action=review > > > Source/WebKit/Shared/CoordinatedGraphics/threadedcompositor/ThreadedCompositor.cpp:94 > > +#if CPU(ADDRESS64) > > + auto windowType = reinterpret_cast<GLNativeWindowType>(m_nativeSurfaceHandle); > > +#else > > + // On 32-bit platforms GLNativeWindowType is an integer type, which cannot be casted with reinterpret_cast. > > + auto windowType = static_cast<GLNativeWindowType>(m_nativeSurfaceHandle); > > +#endif > > So this doesn't work for you? No. it doesn't works. you can see my proposal based on a previous solution did by Adrián in a previous patch.