[haiku-commits] Re: haiku: hrev51344 - src/apps/networkstatus

  • From: kallisti5 <kallisti5@xxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 03 Aug 2017 13:20:51 -0500

On 2017-08-03 13:17, Axel Dörfler wrote:

Am 03/08/2017 um 20:08 schrieb kallisti5:
+    if ((flags & (IFF_UP | IFF_LINK | IFF_CONFIGURING)) == IFF_LINK)
+        return kStatusLinkNoConfig;
Wait... am I crazy or does this not make sense?

this: X & (IFF_UP | IFF_LINK | IFF_CONFIGURING) == IFF_LINK
   reduces to this: X & (IFF_LINK) == IFF_LINK

Masking for IFF_UP and IFF_CONFIGURING doesn't really do anything.

Maybe you meant:
   X & (IFF_UP | IFF_LINK | IFF_CONFIGURING) != 0   ?

No, I'm afraid you are crazy, indeed ;-)

The above check makes sure that out of the given flags, only IFF_LINK
is set, but not the other two.

Ah, duh. Ok. :-)

Other related posts: