From: Richard Levitte - VMS Whacker (richard@levitte.org)
Date: Thu May 05 2005 - 14:59:37 CEST
In message <20050505120801.GF9138@azenomei.knuffel.net> on Thu, 5 May 2005 14:08:01 +0200, Rhialto <rhialto@azenomei.knuffel.net> 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 richard@levitte.org
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
This archive was generated by hypermail 2.1.4 : Fri May 06 2005 - 06:45:03 CEST