From 562a368fbd301d00489cf0d525921a643fc12f06 Mon Sep 17 00:00:00 2001 From: Daniel Stone Date: Mon, 13 Oct 2025 18:06:15 +0100 Subject: [PATCH] core: Remove flush_damage_for_plane return Nothing uses this, and it was also being set incorrectly, as it would return true when there was no damage to actually be flushed. Signed-off-by: Daniel Stone --- libweston/backend.h | 2 +- libweston/compositor.c | 6 +----- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/libweston/backend.h b/libweston/backend.h index 304002104..3c3194a8c 100644 --- a/libweston/backend.h +++ b/libweston/backend.h @@ -343,7 +343,7 @@ void notify_tablet_tool_frame(struct weston_tablet_tool *tool, const struct timespec *time); -bool +void weston_output_flush_damage_for_plane(struct weston_output *output, struct weston_plane *plane, pixman_region32_t *damage); diff --git a/libweston/compositor.c b/libweston/compositor.c index 35290d422..6d1892ee0 100644 --- a/libweston/compositor.c +++ b/libweston/compositor.c @@ -3526,13 +3526,12 @@ weston_output_put_back_feedback_list(struct weston_output *output) wl_list_init(&output->feedback_list); } -WL_EXPORT bool +WL_EXPORT void weston_output_flush_damage_for_plane(struct weston_output *output, struct weston_plane *plane, pixman_region32_t *damage) { struct weston_paint_node *pnode; - bool changed = false; wl_list_for_each(pnode, &output->paint_node_z_order_list, z_order_link) { @@ -3543,8 +3542,6 @@ weston_output_flush_damage_for_plane(struct weston_output *output, if (!pnode->need_hole) continue; } - changed = true; - /* We can safely clip paint node damage to visible region * here, as we're only dealing with nodes on this output, * and the visibility regions for paint nodes on this @@ -3555,7 +3552,6 @@ weston_output_flush_damage_for_plane(struct weston_output *output, pixman_region32_clear(&pnode->damage); } pixman_region32_intersect(damage, damage, &output->region); - return changed; } WL_EXPORT void