diff options
author | Jarek Kobus <jaroslaw.kobus@qt.io> | 2021-05-12 16:55:32 +0200 |
---|---|---|
committer | Jarek Kobus <jaroslaw.kobus@qt.io> | 2021-05-12 22:09:03 +0000 |
commit | da27ac457f524bc96788228461e62e1ef2eaaab7 (patch) | |
tree | f49040fa034a3bc02cf675ca9294575a91adea66 /src/plugins/ios | |
parent | 91f136ef3ab75471cabcaed9dc16dad9f504add8 (diff) |
Do QFutureWatcher::setFuture() after connecting to watcher's signals
Otherwise we may have race condition.
Reference: documentation for QFutureWatcher::setFuture()
and the implementation of
QFutureInterfaceBasePrivate::connectOutputInterface(
QFutureCallOutInterface *interface).
Change-Id: I5b483baaf1b844871a162f47ce7683e9ff495acb
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
Diffstat (limited to 'src/plugins/ios')
-rw-r--r-- | src/plugins/ios/simulatoroperationdialog.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/ios/simulatoroperationdialog.cpp b/src/plugins/ios/simulatoroperationdialog.cpp index 88ce5d48a69..6327ca68879 100644 --- a/src/plugins/ios/simulatoroperationdialog.cpp +++ b/src/plugins/ios/simulatoroperationdialog.cpp @@ -75,9 +75,9 @@ void SimulatorOperationDialog::addFutures(const QList<QFuture<void> > &futureLis foreach (auto future, futureList) { if (!future.isFinished() || !future.isCanceled()) { auto watcher = new QFutureWatcher<void>; - watcher->setFuture(future); connect(watcher, &QFutureWatcher<void>::finished, this, &SimulatorOperationDialog::futureFinished); + watcher->setFuture(future); m_futureWatchList << watcher; } } |