diff options
| author | Daniel Stone <daniels@collabora.com> | 2016-12-01 12:33:54 +0000 |
|---|---|---|
| committer | Daniel Stone <daniels@collabora.com> | 2018-02-27 17:01:18 +0000 |
| commit | c3fcb5bed521c36413570bf9e77d6d5e0a3d743d (patch) | |
| tree | 7c0ddb64adc839ddb37d92af2ac4fc3733f84504 | |
| parent | e1c69be6720c1c27c020e0868370852376c72034 (diff) | |
| download | weston-c3fcb5bed521c36413570bf9e77d6d5e0a3d743d.tar.gz | |
compositor-drm: Don't need safe view-list traversal
Nothing in this loop reorders views within the compositor's view_list.
Signed-off-by: Daniel Stone <daniels@collabora.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
| -rw-r--r-- | libweston/compositor-drm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libweston/compositor-drm.c b/libweston/compositor-drm.c index 9594425f..7cd0e0dd 100644 --- a/libweston/compositor-drm.c +++ b/libweston/compositor-drm.c @@ -3061,7 +3061,7 @@ drm_assign_planes(struct weston_output *output_base, void *repaint_data) struct drm_output *output = to_drm_output(output_base); struct drm_output_state *state; struct drm_plane_state *plane_state; - struct weston_view *ev, *next; + struct weston_view *ev; pixman_region32_t overlap, surface_overlap; struct weston_plane *primary, *next_plane; bool picked_scanout = false; @@ -3087,7 +3087,7 @@ drm_assign_planes(struct weston_output *output_base, void *repaint_data) pixman_region32_init(&overlap); primary = &output_base->compositor->primary_plane; - wl_list_for_each_safe(ev, next, &output_base->compositor->view_list, link) { + wl_list_for_each(ev, &output_base->compositor->view_list, link) { struct weston_surface *es = ev->surface; /* Test whether this buffer can ever go into a plane: |