replace never-true condition with a working one

state->initial is set to false before calling x_push_node() since we
began pushing the window stack before pushing changes. Therefore, the
condition could never be true.
This commit is contained in:
Michael Stapelberg 2013-12-15 14:57:04 +01:00
parent 318b089e6d
commit 28a993e656
1 changed files with 3 additions and 4 deletions

View File

@ -731,10 +731,9 @@ void x_push_node(Con *con) {
} }
/* Map if map state changed, also ensure that the child window /* Map if map state changed, also ensure that the child window
* is changed if we are mapped *and* in initial state (meaning the * is changed if we are mapped and there is a new, unmapped child window.
* container was empty before, but now got a child). Unmaps are handled in * Unmaps are handled in x_push_node_unmaps(). */
* x_push_node_unmaps(). */ if ((state->mapped != con->mapped || (con->window != NULL && !state->child_mapped)) &&
if ((state->mapped != con->mapped || (con->mapped && state->initial)) &&
con->mapped) { con->mapped) {
xcb_void_cookie_t cookie; xcb_void_cookie_t cookie;