diff options
author | Arno <arno@disconnect.de> | 2019-11-23 15:53:37 +0100 |
---|---|---|
committer | Arno <arno@disconnect.de> | 2019-11-23 15:53:37 +0100 |
commit | 980f94e493c349c6c86f1b3d05753ce3cfba38d8 (patch) | |
tree | f5267e2fc7ad9c58668ee62834e8e006c3bf32bd /shemovcleaner.h | |
parent | 30d61398cbb8a2694b648d3fceb119fbeaf756a1 (diff) | |
download | ShemovCleaner-980f94e493c349c6c86f1b3d05753ce3cfba38d8.tar.gz ShemovCleaner-980f94e493c349c6c86f1b3d05753ce3cfba38d8.tar.bz2 ShemovCleaner-980f94e493c349c6c86f1b3d05753ce3cfba38d8.zip |
Fix race in updateCopying
If the storage is fast enough, passing a bool to updateCopying does not
suffice. Then the SIGNAL with inProgress == true could arrive *after* the
work is already done, leaving the copy indicator in an inconsitent
state.
So check the QThread directly by ->isRunning() instead of passing a
bool.
Diffstat (limited to 'shemovcleaner.h')
-rw-r--r-- | shemovcleaner.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/shemovcleaner.h b/shemovcleaner.h index db6399f..4553df8 100644 --- a/shemovcleaner.h +++ b/shemovcleaner.h @@ -21,7 +21,7 @@ class ShemovCleaner : public QMainWindow { void setSelectionCount(const QString &msg); void setDuration(const QString &msg); void updateFreeSpace(const QString &path); - void updateCopying(bool inProgress, const QString &count); + void updateCopying(const QString &count); void configure(); void tabChanged(int idx); void aboutThisProgram(); |