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

_______________________________________________
devel mailing list
[email protected]
https://lists.open-fcoe.org/mailman/listinfo/devel

Reply via email to