On Thu 05 May 2005 at 13:54:41 +0200, Rhialto wrote:
>      tmp_win->iconify_by_unmapping |= 
> -     (short)(int) LookInList(Scr->IconifyByUn, tmp_win->full_name,
> -         &tmp_win->class);
> +     LookInList(Scr->IconifyByUn, tmp_win->full_name,
> +         &tmp_win->class) != NULL;

Perhaps this third one should even better be

    tmp_win->iconify_by_unmapping = tmp_win->iconify_by_unmapping ||
        LookInList(Scr->IconifyByUn, tmp_win->full_name,
            &tmp_win->class);

i.e. foo |= bar != NULL becomes foo = foo || bar, for lack of a ||=
operator. . It's more efficient too in case foo is set already.

-Olaf.
-- 
___ Olaf 'Rhialto' Seibert                            --  rhialto/at/falu.nl
\X/ Hi! I'm a signature virus! Copy me to your .signature to help me spread!

Reply via email to