On Tue, 2011-03-01 at 10:01 -0800, Vasu Dev wrote: > 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. Never mind, you wanted to check both bits set instead checking only one of them, my mistake, anycase these are in different fields as another most recent response mentioned. 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
