[PATCH weston 1/3] gl-renderer: fix EGL initialization steps

Daniel Stone daniel at fooishbar.org
Wed Apr 8 09:21:21 PDT 2015


On 8 April 2015 at 17:17, Manuel Bachmann
<manuel.bachmann at open.eurogiciel.org> wrote:
> Looks good to me !

All three R-b and pushed; thanks.
   b4deec6..8b69d03  master -> master

Cheers,
Daniel

> 2015-04-08 16:02 GMT+02:00 Pekka Paalanen <ppaalanen at gmail.com>:
>>
>> From: Manuel Bachmann <manuel.bachmann at open.eurogiciel.org>
>>
>> Some DRI drivers, including VMware vmwgfx, do not support
>> calling eglQueryString() with a EGL_NO_DISPLAY parameter.
>> Just as we do in gl_renderer_supports(), which returns 0
>> but does not fail in this case, do not fail in
>> gl_renderer_setup_egl_extensions().
>>
>> Signed-off-by: Manuel Bachmann <manuel.bachmann at open.eurogiciel.org>
>> [Pekka: split the patch]
>> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
>> ---
>>  src/gl-renderer.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/gl-renderer.c b/src/gl-renderer.c
>> index b3b2364..5a2ed9f 100644
>> --- a/src/gl-renderer.c
>> +++ b/src/gl-renderer.c
>> @@ -2136,7 +2136,7 @@ gl_renderer_setup_egl_extensions(struct
>> weston_compositor *ec)
>>                 (const char *) eglQueryString(EGL_NO_DISPLAY,
>> EGL_EXTENSIONS);
>>         if (!extensions) {
>>                 weston_log("Retrieving EGL client extension string
>> failed.\n");
>> -               return -1;
>> +               return 0;
>>         }
>>
>>         if (strstr(extensions, "EGL_EXT_platform_base"))
>> --
>> 2.0.5
>>
>
>
>
> --
> Regards,
>
> Manuel BACHMANN
> Tizen Project
> VANNES-FR
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>


More information about the wayland-devel mailing list