On Tue, 2011-03-01 at 10:01 -0800, Vasu Dev wrote:

On Tue, 2011-03-01 at 09:49 -0800, Joe Eykholt wrote:

quoted

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