Make it a bit more clear what the purpose of the variable is.
Signed-off-by: Daniel Stone <dani...@collabora.com>
Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk>
---
libweston/compositor-drm.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c
index 1a705ccda..2dbbf7b6e 100644
--- a/libweston/compositor-drm.c
+++ b/libweston/compositor-drm.c
@@ -3074,7 +3074,7 @@ drm_assign_planes(struct weston_output *output_base, void
*repaint_data)
struct drm_output_state *state;
struct drm_plane_state *plane_state;
struct weston_view *ev;
- pixman_region32_t overlap, surface_overlap;
+ pixman_region32_t surface_overlap, renderer_region;
struct weston_plane *primary, *next_plane;
assert(!output->state_last);
@@ -3095,7 +3095,7 @@ drm_assign_planes(struct weston_output *output_base, void
*repaint_data)
* the client buffer can be used directly for the sprite surface
* as we do for flipping full screen surfaces.
*/
- pixman_region32_init(&overlap);
+ pixman_region32_init(&renderer_region);
primary = &output_base->compositor->primary_plane;
wl_list_for_each(ev, &output_base->compositor->view_list, link) {
@@ -3119,7 +3119,7 @@ drm_assign_planes(struct weston_output *output_base, void
*repaint_data)
es->keep_buffer = false;
pixman_region32_init(&surface_overlap);
- pixman_region32_intersect(&surface_overlap, &overlap,
+ pixman_region32_intersect(&surface_overlap, &renderer_region,
&ev->transform.boundingbox);
next_plane = NULL;
@@ -3140,7 +3140,8 @@ drm_assign_planes(struct weston_output *output_base, void
*repaint_data)
weston_view_move_to_plane(ev, next_plane);
if (next_plane == primary)
- pixman_region32_union(&overlap, &overlap,
+ pixman_region32_union(&renderer_region,
+ &renderer_region,
&ev->transform.boundingbox);
if (next_plane == primary ||
@@ -3157,7 +3158,7 @@ drm_assign_planes(struct weston_output *output_base, void
*repaint_data)
pixman_region32_fini(&surface_overlap);
}
- pixman_region32_fini(&overlap);
+ pixman_region32_fini(&renderer_region);
/* We rely on output->cursor_view being both an accurate reflection of
* the cursor plane's state, but also being maintained across repaints
--
2.14.3
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel
