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

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:40:34 +00:00
parent c22c264123
commit 84c93bbbe5
1 changed files with 11 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/socket.cpp
View file

@ -70,8 +70,19 @@ void CSocket::Init ( const quint16 iPortNumber )
}
// connect the "activated" signal
#ifdef ENABLE_RECEIVE_SOCKET_IN_SEPARATE_THREAD
// We have to use a blocked queued connection since in case we use a
// separate socket thread, the "readyRead" signal would occur and our
// "OnDataReceived" function would be run in another thread. This could
// lead to a situation that a new "readRead" occurs while the processing
// of the previous signal was not finished -> the error: "Multiple
// socket notifiers for same socket" may occur.
QObject::connect ( &SocketDevice, SIGNAL ( readyRead() ),
this, SLOT ( OnDataReceived() ), Qt::BlockingQueuedConnection );
#else
QObject::connect ( &SocketDevice, SIGNAL ( readyRead() ),
this, SLOT ( OnDataReceived() ) );
#endif
}
void CSocket::SendPacket ( const CVector<uint8_t>& vecbySendBuf,