In message <[EMAIL PROTECTED]> on Thu, 5 May 2005 14:08:01 +0200, Rhialto 
<[EMAIL PROTECTED]> said:

rhialto> Perhaps this third one should even better be
rhialto> 
rhialto>     tmp_win->iconify_by_unmapping = tmp_win->iconify_by_unmapping ||
rhialto>        LookInList(Scr->IconifyByUn, tmp_win->full_name,
rhialto>            &tmp_win->class);
rhialto> 
rhialto> i.e. foo |= bar != NULL becomes foo = foo || bar, for lack of a ||=
rhialto> operator. . It's more efficient too in case foo is set already.

Good point.  The only thing that worried me was the possibility that
LookInList() does some kind side-effect with &tmp_win->class.
Fortunately, it doesn't.

I think I'll make it part of 3.8 to constify quite a bit...

-----
Please consider sponsoring my work on free software.
See http://www.free.lp.se/sponsoring.html for details.

-- 
Richard Levitte                         [EMAIL PROTECTED]
                                        http://richard.levitte.org/

"When I became a man I put away childish things, including
 the fear of childishness and the desire to be very grown up."
                                                -- C.S. Lewis

Reply via email to