On Tue, 2011-03-01 at 09:49 -0800, Joe Eykholt wrote: > > Otherwise I'd add some parens or I might code this as: > > if ((netdev->priv_flags & (IFF_BONDING | IFF_MASTER)) == > (IFF_BONDING | IFF_MASTER)) or also skip == in code as:- if (netdev->priv_flags & (IFF_BONDING | IFF_MASTER)) Would least save few chars in source code. Vasu -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html