[interfacekit] BLocker's implementation issue(?)

From: "burton666@xxxxxxxxx" <burton666@xxxxxxxxx>
To: "interfacekit" <interfacekit@xxxxxxxxxxxxx>
Date: Mon, 1 Sep 2003 16:43:19 +0200

I was looking through BLocker's source code, and I noticed this:

int32 oldBenaphoreCount = atomic_add(&fBenaphoreCount, 1);
if (oldBenaphoreCount > 0) {
 etc. etc.


Should this be 
if (atomic_add(&fBenaphoreCount, 1))
  etc. etc. ?

Since the value returned from atomic_add could change before executing the if.

Follow-Ups:

[interfacekit] Re: BLocker's implementation issue(?)
From: Axel Dörfler

[interfacekit] Font Face Remapping
From: DarkWyrm

Other related posts:

» [interfacekit] BLocker's implementation issue(?)
» [interfacekit] Re: BLocker's implementation issue(?)
» [interfacekit] Re: BLocker's implementation issue(?)

Browse interfacekit