From: Michael Stapelberg Date: Tue, 25 Aug 2015 18:09:56 +0000 (+0200) Subject: Revert "Check if output is disabled in handle_output()" X-Git-Tag: 4.10.4~1 X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;ds=sidebyside;h=ce7d256d05adea3d17eb16b336a3237fb5529ea2;p=i3%2Fi3 Revert "Check if output is disabled in handle_output()" This reverts commit e71c304444dd3070877887d2bb5407cd64033946. It turns out that several users have workflows in which they turn off their monitors without using e.g. `xrandr --output DP-1 --off`. The result is that the monitors are disconnected, but not disabled. With commit e71c304444dd3070877887d2bb5407cd64033946, i3 started to see these two states as one and the same state, but that causes more harm than it does good. For example, for some users with only one monitor, i3 would just exit when these users turned off their monitor. related to #1858, #1839 fixes #1845 --- diff --git a/src/randr.c b/src/randr.c index 9549c9d5..a4a0f6fd 100644 --- a/src/randr.c +++ b/src/randr.c @@ -555,12 +555,6 @@ static void handle_output(xcb_connection_t *conn, xcb_randr_output_t id, return; } - if (output->connection == XCB_RANDR_CONNECTION_DISCONNECTED) { - DLOG("Disabling output %s: it is disconnected\n", new->name); - new->to_be_disabled = true; - return; - } - bool updated = update_if_necessary(&(new->rect.x), crtc->x) | update_if_necessary(&(new->rect.y), crtc->y) | update_if_necessary(&(new->rect.width), crtc->width) |