FlapJackApps/subsurface
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

mobile UI: don't call into the event loop

Browse Source 
While this has worked fine for a long time, it now suddenly seems to trigger
crashes. So let's not do that (there are a number of discussions online that
talk about how this could indeed lead to BadThings(tm) happening.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2020-02-21 17:09:17 -08:00
parent c8b55ddb76
commit 3bcc562150
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
mobile-widgets/qmlmanager.cpp
View File

@ -103,9 +103,9 @@ extern "C" int gitProgressCB(const char *text)
qint64 elapsed = timer.elapsed();
self->appendTextToLog(text);
self->setNotificationText(text);
if (elapsed - lastTime > 50) { // 20 Hz refresh
qApp->processEvents(QEventLoop::ExcludeUserInputEvents);
}
//if (elapsed - lastTime > 50) { // 20 Hz refresh
// qApp->processEvents(QEventLoop::ExcludeUserInputEvents);
//}
lastTime = elapsed;
}
// return 0 so that we don't end the download