On Tue, Jan 10, 2012 at 02:28:15PM +0100, ib wrote: > + flags |= (enable ? MF_ENABLED : MF_GRAYED); I think those () are rather pointless. Not that I really mind if you think they make it more readable or something like that.