actions/itgmania212121
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Just in case these are ever implemented, fix. sem_* return -1 for all error conditions. Only sem_destroy sets errno to EBUSY anway.

Browse Source 
Also, ASSERT so we get a backtrace.
This commit is contained in:
Steve Checkoway
2006-02-05 13:24:44 +00:00
parent 79218ae6ad
commit 3cd26abb99
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
stepmania/src/arch/Threads/Threads_Pthreads.cpp
+4 -3
View File
@@ -301,10 +301,11 @@ bool SemaImpl_Pthreads::Wait()
bool SemaImpl_Pthreads::TryWait() bool SemaImpl_Pthreads::TryWait()
{ {
int ret = sem_trywait( &sem ); int ret = sem_trywait( &sem );
if( ret == EBUSY ) if( ret == -1 && errno == EAGAIN )
return false; return false;
if( ret )
RageException::Throw( "TryWait: sem_trywait failed: %s", strerror(errno) ); ASSERT_M( ret == 0, ssprintf("TryWait: sem_trywait failed: %s", strerror(errno)) );
return true; return true;
} }
#else #else