Title: [225051] trunk/Source/WebCore
- Revision
- 225051
- Author
- clo...@igalia.com
- Date
- 2017-11-20 07:57:57 -0800 (Mon, 20 Nov 2017)
Log Message
[WPE] GLContextEGLWPE.cpp:44:96: error: invalid cast from type ‘GLNativeWindowType {aka long long unsigned int}’ to type ‘EGLNativeWindowType {aka unsigned int}
https://bugs.webkit.org/show_bug.cgi?id=179511
Reviewed by Žan Doberšek.
r217208 changed the definition of GLNativeWindowType from EGLNativeWindowType to uint64_t for platform WPE.
This built fine on a 64-bit platform since uint64_t matches the size of a pointer, but fails on 32-bits.
This switches back the definition to EGLNativeWindowType.
And we need to include <EGL/eglplatform.h> first in order to get a definition for EGLNativeWindowType itself.
This commit pretty much reverts the changes r217208 did on platform/graphics/GLContext.h
No new tests, it is a build fix.
* platform/graphics/GLContext.h:
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (225050 => 225051)
--- trunk/Source/WebCore/ChangeLog 2017-11-20 15:46:17 UTC (rev 225050)
+++ trunk/Source/WebCore/ChangeLog 2017-11-20 15:57:57 UTC (rev 225051)
@@ -1,3 +1,22 @@
+2017-11-20 Carlos Alberto Lopez Perez <clo...@igalia.com>
+
+ [WPE] GLContextEGLWPE.cpp:44:96: error: invalid cast from type ‘GLNativeWindowType {aka long long unsigned int}’ to type ‘EGLNativeWindowType {aka unsigned int}
+ https://bugs.webkit.org/show_bug.cgi?id=179511
+
+ Reviewed by Žan Doberšek.
+
+ r217208 changed the definition of GLNativeWindowType from EGLNativeWindowType to uint64_t for platform WPE.
+ This built fine on a 64-bit platform since uint64_t matches the size of a pointer, but fails on 32-bits.
+
+ This switches back the definition to EGLNativeWindowType.
+ And we need to include <EGL/eglplatform.h> first in order to get a definition for EGLNativeWindowType itself.
+
+ This commit pretty much reverts the changes r217208 did on platform/graphics/GLContext.h
+
+ No new tests, it is a build fix.
+
+ * platform/graphics/GLContext.h:
+
2017-11-20 Michael Catanzaro <mcatanz...@igalia.com>
Remove ENABLE_ALLINONE_BUILD option
Modified: trunk/Source/WebCore/platform/graphics/GLContext.h (225050 => 225051)
--- trunk/Source/WebCore/platform/graphics/GLContext.h 2017-11-20 15:46:17 UTC (rev 225050)
+++ trunk/Source/WebCore/platform/graphics/GLContext.h 2017-11-20 15:57:57 UTC (rev 225051)
@@ -24,10 +24,17 @@
#include "PlatformDisplay.h"
#include <wtf/Noncopyable.h>
-#if USE(EGL) && !PLATFORM(GTK) && !PLATFORM(WPE)
+#if USE(EGL) && !PLATFORM(GTK)
+#if PLATFORM(WPE)
+// FIXME: For now default to the GBM EGL platform, but this should really be
+// somehow deducible from the build configuration.
+#define __GBM__ 1
+#include <EGL/eglplatform.h>
+#else
#include "eglplatform.h"
+#endif // PLATFORM(WPE)
typedef EGLNativeWindowType GLNativeWindowType;
-#else
+#else // !USE(EGL) || PLATFORM(GTK)
typedef uint64_t GLNativeWindowType;
#endif
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes