AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove the comparison operators of QFuture

Browse Source 
These operators don't do what the user might expect and may lead to
confusing results.

[ChangeLog][Source-Incompatible Changes][QFuture] The comparison
operators of QFuture have been removed. They were comparing the
underlying d-ptrs instead of comparing the results (as the users
might expect), which is not very helpful for the users point of view.

Change-Id: I80a887610eac38b60329128cca52cdb5fb515207
Reviewed-by: Sona Kurazyan <sona.kurazyan@qt.io>
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
This commit is contained in:
Sona Kurazyan 2020-10-29 12:41:11 +01:00 committed by Volker Hilsheimer
parent 387f3e00a8
commit 30a1683f65
5 changed files with 2 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/corelib/thread/qfuture.h
View File

@ -101,9 +101,6 @@ public:
operator T() const;
#endif
bool operator==(const QFuture &other) const { return (d == other.d); }
bool operator!=(const QFuture &other) const { return (d != other.d); }
void cancel() { d.cancel(); }
bool isCanceled() const { return d.isCanceled(); }

11
src/corelib/thread/qfuture.qdoc
View File

@ -167,17 +167,6 @@
Assigns \a other to this future and returns a reference to this future.
*/
/*! \fn template <typename T> bool QFuture<T>::operator==(const QFuture &other) const
Returns \c true if \a other is a copy of this future; otherwise returns \c false.
*/
/*! \fn template <typename T> bool QFuture<T>::operator!=(const QFuture &other) const
Returns \c true if \a other is \e not a copy of this future; otherwise returns
false.
*/
/*! \fn template <typename T> void QFuture<T>::cancel()
Cancels the asynchronous computation represented by this future. Note that

2
src/corelib/thread/qfuturewatcher.h
View File

@ -210,7 +210,7 @@ private:
template <typename T>
Q_INLINE_TEMPLATE void QFutureWatcher<T>::setFuture(const QFuture<T> &_future)
{
if (_future == m_future)
if (_future.d == m_future.d)
return;
disconnectOutputInterface(true);

2
tests/auto/corelib/thread/qfuture/tst_qfuture.cpp
View File

@ -1504,8 +1504,6 @@ void tst_QFuture::voidConversions()
QFuture<void> voidFuture(intFuture);
voidFuture = intFuture;
QVERIFY(voidFuture == intFuture);
}
{

3
tests/auto/corelib/thread/qfuturesynchronizer/tst_qfuturesynchronizer.cpp
View File

@ -107,7 +107,7 @@ void tst_QFutureSynchronizer::futures()
synchronizer.addFuture(future);
}
QCOMPARE(futures, synchronizer.futures());
QCOMPARE(futures.count(), synchronizer.futures().count());
}
void tst_QFutureSynchronizer::setFuture()
@ -122,7 +122,6 @@ void tst_QFutureSynchronizer::setFuture()
QFuture<void> future;
synchronizer.setFuture(future);
QCOMPARE(synchronizer.futures().size(), 1);
QCOMPARE(synchronizer.futures().first(), future);
}
void tst_QFutureSynchronizer::waitForFinished()