Hi, Reviewed-by: Giulio Camuffo <giuliocamu...@gmail.com>
2016-11-03 11:15 GMT+01:00 Vincent Abriou <vincent.abr...@st.com>: > While gl_renderer_attach, query_buffer should be call only if the > query_buffer function exists ie when has_bind_display is true. > > v2: > - Take into account Giulio's remark. Use has_bind_display viariable to test > if > EGL_WL_bind_wayland_display extension is supported. > > Change-Id: I5b5995fec67351cc8e4746f932d7ace838fa73be > Signed-off-by: Vincent Abriou <vincent.abr...@st.com> > --- > libweston/gl-renderer.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libweston/gl-renderer.c b/libweston/gl-renderer.c > index 9747de5..3e8e5ab 100644 > --- a/libweston/gl-renderer.c > +++ b/libweston/gl-renderer.c > @@ -2004,7 +2004,8 @@ gl_renderer_attach(struct weston_surface *es, struct > weston_buffer *buffer) > > if (shm_buffer) > gl_renderer_attach_shm(es, buffer, shm_buffer); > - else if (gr->query_buffer(gr->egl_display, (void *) buffer->resource, > + else if (gr->has_bind_display && > + gr->query_buffer(gr->egl_display, (void *)buffer->resource, > EGL_TEXTURE_FORMAT, &format)) > gl_renderer_attach_egl(es, buffer, format); > else if ((dmabuf = linux_dmabuf_buffer_get(buffer->resource))) > -- > 1.9.1 > > _______________________________________________ > wayland-devel mailing list > wayland-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel