Skip to content
This repository
Browse code

changed: Sleep(0) is defined as sched_yield() in XTimeUtils.cpp (than…

…ks ace20022).
  • Loading branch information...
commit 7ff4e0fe9d004ab93980ab9da199092ea5485f73 1 parent 70735fe
wsoltys authored February 19, 2013
4  xbmc/cores/AudioEngine/Sinks/AESinkNULL.cpp
@@ -135,11 +135,7 @@ void CAESinkNULL::Process()
135 135
   // The object has been created and waiting to play,
136 136
   m_inited.Set();
137 137
   // yield to give other threads a chance to do some work.
138  
-#ifdef TARGET_WINDOWS
139 138
   Sleep(0);
140  
-#else
141  
-  sched_yield();
142  
-#endif
143 139
 
144 140
   SetPriority(THREAD_PRIORITY_ABOVE_NORMAL);
145 141
   while (!m_bStop)

3 notes on commit 7ff4e0f

Tobias Arrskog
Collaborator

Well we don't want to use boost threading out in the code, it should be hidden within the thread implementation. That being said, yield in the thread interfaces would be good :+1: as afaik we don't have that

Cory Fields
Owner

Boost threads are not header-only. As it stands, due to porting headaches, that makes them a no-go.

IIRC our current implementation was written specifically to avoid using boost.

Please sign in to comment.
Something went wrong with that request. Please try again.