[haiku-commits] Re: r34823 - haiku/trunk/src/libs/compat/freebsd_wlan/net80211

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 30 Dec 2009 09:50:30 +0100

coling@xxxxxx wrote:
> +     // atomic_add returns old value
> +     return 1 == atomic_add((vint32*)&ni->ni_refcnt, -1);

Quoting the coding style guide:
"When comparing a function call result/variable with a constant, don't 
place the constant on the left side of the comparison..."

Bye,
   Axel.


Other related posts: