[PATCH weston 2/3] gl-renderer: check EGL_EXT_platform_base in supports()

[Manuel Bachmann]

Looks good to me !

2015-04-08 16:02 GMT+02:00 Pekka Paalanen <ppaalanen at gmail.com>:

> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
>
> An EGL implementation may support client extensions without supporting
> EGL_EXT_platform_base. In such a case, we should return 0 to fall back
> to the old eglGetDisplay() way.
>
> Cc: Manuel Bachmann <manuel.bachmann at open.eurogiciel.org>
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> ---
>  src/gl-renderer.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/src/gl-renderer.c b/src/gl-renderer.c
> index 5a2ed9f..35cd7e7 100644
> --- a/src/gl-renderer.c
> +++ b/src/gl-renderer.c
> @@ -2213,6 +2213,9 @@ gl_renderer_supports(struct weston_compositor *ec,
>                                extensions);
>         }
>
> +       if (!strstr(extensions, "EGL_EXT_platform_base"))
> +               return 0;
> +
>         snprintf(s, sizeof s, "EGL_KHR_platform_%s", extension_suffix);
>         if (strstr(extensions, s))
>                 return 1;
> @@ -2225,8 +2228,8 @@ gl_renderer_supports(struct weston_compositor *ec,
>         if (strstr(extensions, s))
>                 return 1;
>
> -       /* at this point we definitely have some client extensions but
> -        * haven't found the supplied client extension, so chances are it's
> +       /* at this point we definitely have some platform extensions but
> +        * haven't found the supplied platform, so chances are it's
>          * not supported. */
>
>         return -1;
> --
> 2.0.5
>
>


-- 
Regards,



*Manuel BACHMANN Tizen Project VANNES-FR*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20150408/85ab51a3/attachment.html>