Re: [hackers] [dwm] setfullscreen: don't process the property twice

From: Markus Teich <markus.teich_AT_stusta.mhn.de>
Date: Mon, 2 Nov 2015 23:53:02 +0100

Quentin Rameau wrote:
> if(fullscreen) {
> - …
> + if(!c->isfullscreen) {
> + …
> + }
> }
> - else {
> + else if(c->isfullscreen) {

Heyho Quentin,

you should merge the conditions of the first two nested if statements and change
the else if to hold the complete condition `(!fullscreen && c->isfullscreen)`.

--Markus
Received on Mon Nov 02 2015 - 23:53:02 CET