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) {

Reply via email to