This variable is used to signal a thread to exit. It should be volatile.

It's a long stretch, but it could happen that the compiler optimises
out the checking of the variable because it could not change
synchronously. In any case, it's best to set it to volatile to ensure
that all architectures reload properly (memory ordering semantics)

Reviewed-by: Alexis Menard

1 files changed, 1 insertions, 1 deletions

diff --git a/src/gui/dialogs/qfileinfogatherer_p.h b/src/gui/dialogs/qfileinfogatherer_p.h
index cb980d2..bab4a29 100644
--- a/src/gui/dialogs/qfileinfogatherer_p.h
+++ b/src/gui/dialogs/qfileinfogatherer_p.h
@@ -184,7 +184,7 @@ private:
 
     QMutex mutex;
     QWaitCondition condition;
-    bool abort;
+    volatile bool abort;
 
     QStack<QString> path;
     QStack<QStringList> files;