setfullscreen: don't process the property twice
authorQuentin Rameau <quinq+hackers@fifth.space>
Sat, 7 Nov 2015 13:09:08 +0000 (14:09 +0100)
committerHiltjo Posthuma <hiltjo@codemadness.org>
Sat, 7 Nov 2015 13:31:21 +0000 (14:31 +0100)
commitdce4fb373757727374d00c857ec0dfd225bbeafd
tree90d4a9f92b62603cf2bd85fc749db57608b903ab
parent646b351cc79845f4cc77415dfff474b9ae0053d9
setfullscreen: don't process the property twice

Some clients try to set _NET_WM_STATE_FULLSCREEN even when the window is
already in fullscreen.
For example, c->oldstate was set two times in a raw and window would
then always be floating.
We must check that it's not the case before processing it.
(original patch modified with suggestion from Markus Teich
<markus.teich@stusta.mhn.de>)
dwm.c