On 2/12/09, Peter Hartlich <[email protected]> wrote: >> - if((ev->window == root) && (ev->atom = XA_WM_NAME)) >> + if((ev->window == root) && (ev->atom == XA_WM_NAME)) > > Why not remove the inner parens as well? Without them, gcc would have > noticed.
yes, that's how i'd do (but in certain cases parents makes the code
easier to read)
now this is what i found in toggletag():
if(!sel)
return;
mask = sel->tags ^ (arg->ui & TAGMASK);
- if(sel && mask) {
+ if(mask) {
