Thanks for these patches! Kind regards, Anselm
2009/2/12 Szabolcs Nagy <nszabo...@gmail.com>: > On 2/12/09, Peter Hartlich <sg...@hartlich.com> 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) {