eeva/jamulus
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

enable the experimental "threaded socket" code since one outstanding issue is now fixed :"fix for the "Multiple socket notifiers for same socket" error if threaded socket is used"

Browse source
This commit is contained in:
Volker Fischer 2013-08-18 07:41:30 +00:00
parent 84c93bbbe5
commit 9d05436964
1 changed files with 4 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/global.h
View file

@ -72,14 +72,10 @@ LED bar: lbr
// does not effect the stability of the audio stream (e.g. if the GUI is on // does not effect the stability of the audio stream (e.g. if the GUI is on
// high load because of a table update, the incoming network packets must still // high load because of a table update, the incoming network packets must still
// be put in the jitter buffer with highest priority). // be put in the jitter buffer with highest priority).
// Unfortunately, the current implementation of the separate thread does not // Since the new code is still in "experimental state", we introduced this macro
// work correctly. Therefore we introduced this macro to enable/disable the new // to enable/disable the new code.
// code. If the new code is enabled, under Windows Qt warnings like #define ENABLE_RECEIVE_SOCKET_IN_SEPARATE_THREAD
// "QSocketNotifier: Multiple socket notifiers for same socket" appear and after //#undef ENABLE_RECEIVE_SOCKET_IN_SEPARATE_THREAD
// a while the receive socket does not receive any new network packets anymore.
// This has to be solved!
//#define ENABLE_RECEIVE_SOCKET_IN_SEPARATE_THREAD
#undef ENABLE_RECEIVE_SOCKET_IN_SEPARATE_THREAD