From 9aff503a55e0725b68d8f6ccf2fbf444246b744d Mon Sep 17 00:00:00 2001 From: darkraven Date: Thu, 6 Sep 2012 23:27:23 +0800 Subject: [PATCH] remove unnecessary code in render_con(). The removed code was add by commit 61b8a62 to fix #564. That bug is cause by rendering the mplayer window again as a floating window (even if it has been rendered before as a fullscreen window, at line 202). So simply checking for fullscreen window is enough to solve this problem. Treating floating/tiling fullscreen window differently is not needed. --- src/render.c | 17 ++--------------- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/src/render.c b/src/render.c index da993a57..0eda1a97 100644 --- a/src/render.c +++ b/src/render.c @@ -246,26 +246,13 @@ void render_con(Con *con, bool render_fullscreen) { Con *content = output_get_content(output); Con *workspace = TAILQ_FIRST(&(content->focus_head)); - /* Check for (floating!) fullscreen nodes */ + /* Check for fullscreen nodes */ /* XXX: This code duplication is unfortunate. Keep in mind to fix * this when we clean up the whole render.c */ Con *fullscreen = NULL; fullscreen = con_get_fullscreen_con(workspace, CF_OUTPUT); - if (fullscreen) { - /* Either the fullscreen window is inside the floating - * container, then we need to render and raise it now… */ - if (con_inside_floating(fullscreen)) { - fullscreen->rect = output->rect; - x_raise_con(fullscreen); - render_con(fullscreen, true); + if (fullscreen) continue; - } else { - /* …or it’s a tiling window, in which case the floating - * windows should not overlap it, so we skip rendering this - * output. */ - continue; - } - } Con *child; TAILQ_FOREACH(child, &(workspace->floating_head), floating_windows) {