[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xorg-devel
Subject:    Re: [PATCH xserver] xwayland: Avoid using epoxy_has_egl()
From:       Mario Kleiner <mario.kleiner.de () gmail ! com>
Date:       2018-04-30 19:15:15
Message-ID: CAEsyxyg2dL66zJHoTfTACVQ1YysXdUeyW18nZLB_aPtqgpXqAA () mail ! gmail ! com
[Download RAW message or body]

On Mon, Apr 30, 2018 at 8:10 PM, Adam Jackson <ajax@redhat.com> wrote:
> There's no real point - if we don't have EGL then the extension check is
> also going to fail - and the entrypoint is new in 1.5.0, which we don't
> need to require yet.
>
> Signed-off-by: Adam Jackson <ajax@redhat.com>
> ---
>  hw/xwayland/xwayland-glamor.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/hw/xwayland/xwayland-glamor.c b/hw/xwayland/xwayland-glamor.c
> index 2bd16e7ac2..cdca072ed3 100644
> --- a/hw/xwayland/xwayland-glamor.c
> +++ b/hw/xwayland/xwayland-glamor.c
> @@ -55,8 +55,7 @@ xwl_glamor_egl_make_current(struct xwl_screen *xwl_screen)
>  Bool
>  xwl_glamor_egl_supports_device_probing(void)
>  {
> -    return epoxy_has_egl() &&
> -        epoxy_has_egl_extension(NULL, "EGL_EXT_device_base");
> +    return epoxy_has_egl_extension(NULL, "EGL_EXT_device_base");
>  }
>

Reviewed-by: Mario Kleiner <mario.kleiner.de@gmail.com>
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel
[prev in list] [next in list] [prev in thread] [next in thread]