Bug 223577 – [GTK] X11 build fixes

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Philippe Normand]

It is this time of the year when X11 macros leak everywhere again.

[Philippe Normand]

Created attachment 423888 [details]
Patch

[Philippe Normand]

EWS fails:

ThreadedCompositor.cpp:89:51: error: invalid ‘static_cast’ from type ‘uint64_t’ {aka ‘long unsigned int’} to type ‘GLNativeWindowType’ {aka ‘void*’}

[Philippe Normand]

Created attachment 424013 [details]
Patch

[EWS]

Committed r274869: <https://commits.webkit.org/r274869>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 424013 [details].

[Pablo Saavedra]

(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.

[Philippe Normand]

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?

[Pablo Saavedra]

(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.