From: Rhialto (rhialto@azenomei.knuffel.net)
Date: Thu May 05 2005 - 14:08:01 CEST
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!
This archive was generated by hypermail 2.1.4 : Fri May 06 2005 - 06:45:03 CEST