Merge pull request #8972 from sethalves/fix-for-2159

lock around audio-client quit flag to avoid shutdown race
This commit is contained in:
Brad Hefta-Gaub 2016-11-02 00:14:33 -07:00 committed by GitHub
commit 56b3ac8c89

View file

@ -85,18 +85,26 @@ public:
}
void beforeAboutToQuit() {
Lock lock(_checkDevicesMutex);
_quit = true;
}
void run() override {
while (!_quit) {
while (true) {
{
Lock lock(_checkDevicesMutex);
if (_quit) {
break;
}
_audioClient->checkDevices();
}
QThread::msleep(DEVICE_CHECK_INTERVAL_MSECS);
_audioClient->checkDevices();
}
}
private:
AudioClient* _audioClient { nullptr };
Mutex _checkDevicesMutex;
bool _quit { false };
};