diff options
| author | Jan Ekström <jeebjp@gmail.com> | 2022-10-26 23:37:48 +0300 |
|---|---|---|
| committer | Jan Ekström <jeebjp@gmail.com> | 2022-10-27 19:07:05 +0300 |
| commit | bc3e96098dbe3efbdffdadc44ebc44b532709deb (patch) | |
| tree | 8a8426ddcac835051ae9c7c1f370e4830c0cb890 /video/out/wayland_common.c | |
| parent | e0c4193ee5c0ccc9811947f881c3e647544f81df (diff) | |
| download | mpv-bc3e96098dbe3efbdffdadc44ebc44b532709deb.tar.bz2 mpv-bc3e96098dbe3efbdffdadc44ebc44b532709deb.tar.xz | |
video/out/wayland_common: adjust decoration disagreement log level
Originally, I considered warning once to be useful for figuring
out whether the change in logic regarding resetting requested
mode actually fixed the reported issue or not, but alas not everyone
was happy with that decision. Thus the log level will always be
debug. This enables us to lose one level of indent as well as a
variable, which is always positive.
Additionally, make the message more explicit regarding what could
possibly be implied by the mismatch, as it seems like this was
not always clear.
Diffstat (limited to 'video/out/wayland_common.c')
| -rw-r--r-- | video/out/wayland_common.c | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/video/out/wayland_common.c b/video/out/wayland_common.c index 0bc48e07d7..fb03c08021 100644 --- a/video/out/wayland_common.c +++ b/video/out/wayland_common.c @@ -967,19 +967,16 @@ static void configure_decorations(void *data, struct vo_wayland_state *wl = data; struct mp_vo_opts *opts = wl->vo_opts; - if (wl->requested_decoration) { - if (mode != wl->requested_decoration) { - MP_MSG(wl, wl->warned_of_mismatch ? MSGL_DEBUG : MSGL_WARN, - "Requested %s decorations but compositor responded with %s\n", - zxdg_decoration_mode_to_str(wl->requested_decoration), - zxdg_decoration_mode_to_str(mode)); - - wl->warned_of_mismatch = true; - } - - wl->requested_decoration = 0; + if (wl->requested_decoration && mode != wl->requested_decoration) { + MP_DBG(wl, + "Requested %s decorations but compositor responded with %s. " + "It is likely that compositor wants us to stay in a given mode.\n", + zxdg_decoration_mode_to_str(wl->requested_decoration), + zxdg_decoration_mode_to_str(mode)); } + wl->requested_decoration = 0; + if (mode == ZXDG_TOPLEVEL_DECORATION_V1_MODE_SERVER_SIDE) { MP_VERBOSE(wl, "Enabling server decorations\n"); } else { |