comicsansg/CubicSDR
comicsansg/CubicSDR
1
0
Fork 0
mirror of https://github.com/cjcliffe/CubicSDR.git synced 2024-11-13 23:51:46 -05:00
Code Issues Projects Releases Wiki Activity

Merge pull request #416 from vsonnier/ModemArgInfoList_concurrent_access_crash

Browse Source 
Crash with non-empty ModemArgInfoList (in FM Stereo)
This commit is contained in:
Charles J. Cliffe 2016-07-28 12:19:32 -05:00 committed by GitHub
commit a984ce5b41
3 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/AppFrame.cpp
View File

@ -1496,8 +1496,12 @@ void AppFrame::OnIdle(wxIdleEvent& event) {
demod = wxGetApp().getDemodMgr().getLastActiveDemodulator();
if (modemPropertiesUpdated.load() && demod && demod->isModemInitialized()) {
modemProps->initProperties(demod->getModemArgs());
//reset notification flag
modemPropertiesUpdated.store(false);
modemProps->initProperties(demod->getModemArgs());
demodTray->Layout();
modemProps->fitColumns();
#if ENABLE_DIGITAL_LAB
@ -1873,8 +1877,8 @@ FFTVisualDataThread *AppFrame::getWaterfallDataThread() {
return waterfallDataThread;
}
void AppFrame::updateModemProperties(ModemArgInfoList args) {
newModemArgs = args;
void AppFrame::notifyUpdateModemProperties() {
modemPropertiesUpdated.store(true);
}

3
src/AppFrame.h
View File

@ -83,7 +83,7 @@ public:
FFTVisualDataThread *getWaterfallDataThread();
void updateModemProperties(ModemArgInfoList args);
void notifyUpdateModemProperties();
void setMainWaterfallFFTSize(int fftSize);
void gkNudgeLeft(DemodulatorInstance *demod, int snap);
@ -163,7 +163,6 @@ private:
ModemProperties *modemProps;
std::atomic_bool modemPropertiesUpdated;
ModemArgInfoList newModemArgs;
wxMenuItem *showTipMenuItem;
bool lowPerfMode;

2
src/demod/DemodulatorWorkerThread.cpp
View File

@ -61,7 +61,7 @@ void DemodulatorWorkerThread::run() {
cModem->writeSettings(demodCommand.settings);
}
result.sampleRate = demodCommand.sampleRate;
wxGetApp().getAppFrame()->updateModemProperties(cModem->getSettings());
wxGetApp().getAppFrame()->notifyUpdateModemProperties();
}
result.modem = cModem;