compositor: Avoid using weston_log() in weston_view_is_opaque()
authorMarius Vlad <marius.vlad@collabora.com>
Fri, 21 Aug 2020 10:57:54 +0000 (13:57 +0300)
committerMarius Vlad <marius.vlad@collabora.com>
Mon, 24 Aug 2020 11:19:02 +0000 (14:19 +0300)
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 <marius.vlad@collabora.com>
libweston/compositor.c

index 57b220302b39ffc00235d354c56c67f9bbcba8f2..7fd4cc1ef9d5364ee203f619fdbf88b791792cd3 100644 (file)
@@ -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);