From 4d0214c62b4363c9a86546978dbc1cc042087a40 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kristian=20H=C3=B8gsberg?= Date: Thu, 8 Nov 2012 11:36:02 -0500 Subject: [PATCH] compositor: Use pixman_region32_intersect_rect() instead of temp region We avoid reusing 'opaque' and don't leak the region. --- src/compositor.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index 855203a8..55d20606 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -1262,13 +1262,12 @@ surface_commit(struct wl_client *client, struct wl_resource *resource) surface->pending.sy = 0; /* wl_surface.damage */ - pixman_region32_init_rect(&opaque, 0, 0, - surface->geometry.width, - surface->geometry.height); pixman_region32_union(&surface->damage, &surface->damage, &surface->pending.damage); - pixman_region32_intersect(&surface->damage, &surface->damage, - &opaque); + pixman_region32_intersect_rect(&surface->damage, &surface->damage, + 0, 0, + surface->geometry.width, + surface->geometry.height); empty_region(&surface->pending.damage); /* wl_surface.set_opaque_region */