>> Flags flags = ((this->flags()&(~(STATE|PUSHED|HIGHLIGHT)))|OUTPUT); >> >> ... to make it _absolutely_ clear. ((((useless spaces removed >> intentionally)))) ;-P
> I like the spaces, though... A little whitespace[1] breaks up the > line nicely and aids readability I always think. Maybe it isn't fast and light enough, but a lot of code would be improved by simply introducing a temporary local variable with an informative name. A good compiler should just optimize it away. Flags descriptive_name = this->flags() & ~(STATE|PUSHED|HIGHLIGHT); Flags flags = descriptive_name | OUTPUT; Of course the difficulty is finding the right descriptive name that is short, readable, unambiguous, aesthetically pleasing... :-) D _______________________________________________ fltk mailing list [email protected] http://lists.easysw.com/mailman/listinfo/fltk
