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

Reply via email to