From 06a3f96fe162e5aadb032c8b00b37815abbfec1d Mon Sep 17 00:00:00 2001 From: Jeff Abrahamson Date: Mon, 7 Jul 2014 19:59:05 +0200 Subject: [PATCH 1/2] Refactor duplicate branches of if() into a single block with leading ?:. --- src/window.c | 34 +++++++++------------------------- 1 file changed, 9 insertions(+), 25 deletions(-) diff --git a/src/window.c b/src/window.c index de9032a..9d784ea 100644 --- a/src/window.c +++ b/src/window.c @@ -467,34 +467,18 @@ set_active_window_body (rp_window *win, int force) /* With Xinerama, we can move a window over to the current screen; otherwise * we have to switch to the screen that the window belongs to. */ - if (rp_have_xinerama) + rp_screen *screen = (rp_have_xinerama ? current_screen() : win->scr); + /* use the intended frame if we can. */ + if (win->intended_frame_number >= 0) { - /* use the intended frame if we can. */ - if (win->intended_frame_number >= 0) - { - frame = screen_get_frame (current_screen(), win->intended_frame_number); - win->intended_frame_number = -1; - if (frame != current_frame()) - last_frame = current_frame(); - } - - if (!frame) - frame = screen_get_frame (current_screen(), current_screen()->current_frame); + frame = screen_get_frame (screen, win->intended_frame_number); + win->intended_frame_number = -1; + if (frame != current_frame()) + last_frame = current_frame(); } - else - { - /* use the intended frame if we can. */ - if (win->intended_frame_number >= 0) - { - frame = screen_get_frame (win->scr, win->intended_frame_number); - win->intended_frame_number = -1; - if (frame != current_frame()) - last_frame = current_frame(); - } - if (!frame) - frame = screen_get_frame (win->scr, win->scr->current_frame); - } + if (!frame) + frame = screen_get_frame (screen, screen->current_frame); if (frame->dedicated && !force) { -- 1.9.1