Re: [PATCH 0/10] MR5177: winewayland.drv: part 13.1: Basic OpenGL support

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winewayland.drv/opengl.c:
> - {
> -
>    ERR("Failed to load eglGetProcAddress\n");
> -
>    goto err;
> - }
> -
>
> +#define LOAD_FUNCPTR(func) \
> - do { \
> -
>    if (!(p_##func = load_symbol(egl_handle, #func))) \
> -
>        { ERR("Failed to load symbol %s\n", #func); goto err; } \
> - } while(0)
> - LOAD_FUNCPTR(eglGetDisplay);
> - LOAD_FUNCPTR(eglGetError);
> - LOAD_FUNCPTR(eglInitialize);
>
> +#undef LOAD_FUNCPTR
> -
> - egl_display = p_eglGetDisplay((EGLNativeDisplayType)process_wayland.wl_display);

What if EGL doesn't expect Wayland as its default platform? What about using  EGL_DEFAULT_DISPLAY instead or eglGetPlatformDisplay?