[PATCH 2/3] simple-egl: Set the right size for the opaque region when fullscreen

Kristian Høgsberg hoegsberg at gmail.com
Fri Nov 16 16:25:14 PST 2012


On Fri, Nov 16, 2012 at 05:42:26PM +0200, Ander Conselvan de Oliveira wrote:
> Ping? Just noticed (again) today that we set the wrong opaque region
> with simple-egl when fullscreen.

Oh, ok, I think I lost track of that one in the simple-egl opaque
discussion.  Thanks for the reminder.

Kristian

> On 09/07/2012 05:32 PM, Ander Conselvan de Oliveira wrote:
> >window.window_size holds the size of the window in windowed mode. Use
> >window.geometry for setting the opaque region since that holds the
> >current size.
> >---
> >  clients/simple-egl.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> >diff --git a/clients/simple-egl.c b/clients/simple-egl.c
> >index a67ce2f..0278817 100644
> >--- a/clients/simple-egl.c
> >+++ b/clients/simple-egl.c
> >@@ -404,8 +404,8 @@ redraw(void *data, struct wl_callback *callback, uint32_t time)
> >  	if (window->opaque || window->fullscreen) {
> >  		region = wl_compositor_create_region(window->display->compositor);
> >  		wl_region_add(region, 0, 0,
> >-			      window->window_size.width,
> >-			      window->window_size.height);
> >+			      window->geometry.width,
> >+			      window->geometry.height);
> >  		wl_surface_set_opaque_region(window->surface, region);
> >  		wl_region_destroy(region);
> >  	}
> >
> 


More information about the wayland-devel mailing list