[PATCH weston 3/6] compositor: remove dead code from weston_output_move()

Pekka Paalanen ppaalanen at gmail.com
Wed Feb 25 05:48:58 PST 2015


From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>

Also fixes a theoretical memory leak as the region was never fini'd.

Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
---
 src/compositor.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/compositor.c b/src/compositor.c
index b1970cd..abc42e8 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -3754,7 +3754,6 @@ weston_output_init_geometry(struct weston_output *output, int x, int y)
 WL_EXPORT void
 weston_output_move(struct weston_output *output, int x, int y)
 {
-	pixman_region32_t old_region;
 	struct wl_resource *resource;
 
 	output->move_x = x - output->x;
@@ -3763,9 +3762,6 @@ weston_output_move(struct weston_output *output, int x, int y)
 	if (output->move_x == 0 && output->move_y == 0)
 		return;
 
-	pixman_region32_init(&old_region);
-	pixman_region32_copy(&old_region, &output->region);
-
 	weston_output_init_geometry(output, x, y);
 
 	output->dirty = 1;
-- 
2.0.5



More information about the wayland-devel mailing list