From 67b382ccdc42ceb2e52e4de59b4de76e7a68aa09 Mon Sep 17 00:00:00 2001 From: Marius Vlad Date: Fri, 21 Aug 2020 13:57:54 +0300 Subject: [PATCH] compositor: Avoid using weston_log() in weston_view_is_opaque() As from commit b7e5f10bf47, weston_view_is_opaque() is called from debug_scene_graph_cb(), which on its own represents a (different) scope. By default, we already have a subscriber for the 'log' scope, which will cause a harmless, yet spurious, message. Signed-off-by: Marius Vlad --- libweston/compositor.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libweston/compositor.c b/libweston/compositor.c index 57b22030..7fd4cc1e 100644 --- a/libweston/compositor.c +++ b/libweston/compositor.c @@ -1888,10 +1888,8 @@ weston_view_is_opaque(struct weston_view *ev, pixman_region32_t *region) if (ev->surface->is_opaque) return true; - if (ev->transform.dirty) { - weston_log("%s: transform dirty", __func__); + if (ev->transform.dirty) return false; - } pixman_region32_init(&r); pixman_region32_subtract(&r, region, &ev->transform.opaque);