Fix warnings about unused expression results in tests
Change-Id: I401ec8a3742a5bb51a1b8b291c53395c3880117c Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
This commit is contained in:
parent
6f17509ad9
commit
9023791a45
@ -858,14 +858,14 @@ void tst_QMimeDatabase::fromThreads()
|
||||
QThreadPool tp;
|
||||
tp.setMaxThreadCount(20);
|
||||
// Note that data-based tests cannot be used here (QTest::fetchData asserts).
|
||||
QtConcurrent::run(&tp, &tst_QMimeDatabase::mimeTypeForName, this);
|
||||
QtConcurrent::run(&tp, &tst_QMimeDatabase::aliases, this);
|
||||
QtConcurrent::run(&tp, &tst_QMimeDatabase::allMimeTypes, this);
|
||||
QtConcurrent::run(&tp, &tst_QMimeDatabase::icons, this);
|
||||
QtConcurrent::run(&tp, &tst_QMimeDatabase::inheritance, this);
|
||||
QtConcurrent::run(&tp, &tst_QMimeDatabase::knownSuffix, this);
|
||||
QtConcurrent::run(&tp, &tst_QMimeDatabase::mimeTypeForFileWithContent, this);
|
||||
QtConcurrent::run(&tp, &tst_QMimeDatabase::allMimeTypes, this); // a second time
|
||||
Q_UNUSED(QtConcurrent::run(&tp, &tst_QMimeDatabase::mimeTypeForName, this));
|
||||
Q_UNUSED(QtConcurrent::run(&tp, &tst_QMimeDatabase::aliases, this));
|
||||
Q_UNUSED(QtConcurrent::run(&tp, &tst_QMimeDatabase::allMimeTypes, this));
|
||||
Q_UNUSED(QtConcurrent::run(&tp, &tst_QMimeDatabase::icons, this));
|
||||
Q_UNUSED(QtConcurrent::run(&tp, &tst_QMimeDatabase::inheritance, this));
|
||||
Q_UNUSED(QtConcurrent::run(&tp, &tst_QMimeDatabase::knownSuffix, this));
|
||||
Q_UNUSED(QtConcurrent::run(&tp, &tst_QMimeDatabase::mimeTypeForFileWithContent, this));
|
||||
Q_UNUSED(QtConcurrent::run(&tp, &tst_QMimeDatabase::allMimeTypes, this)); // a second time
|
||||
QVERIFY(tp.waitForDone(60000));
|
||||
}
|
||||
|
||||
|
@ -169,7 +169,7 @@ void tst_QExplicitlySharedDataPointer::data() const
|
||||
pointer.data();
|
||||
|
||||
/* Check that this cast is possible. */
|
||||
static_cast<const MyClass *>(pointer.data());
|
||||
Q_UNUSED(static_cast<const MyClass *>(pointer.data()));
|
||||
|
||||
QVERIFY(! (pointer == nullptr));
|
||||
QVERIFY(! (nullptr == pointer));
|
||||
@ -181,7 +181,7 @@ void tst_QExplicitlySharedDataPointer::data() const
|
||||
pointer.data();
|
||||
|
||||
/* Check that this cast is possible. */
|
||||
static_cast<const MyClass *>(pointer.data());
|
||||
Q_UNUSED(static_cast<const MyClass *>(pointer.data()));
|
||||
}
|
||||
|
||||
/* Must not mutate the pointer. */
|
||||
@ -190,8 +190,8 @@ void tst_QExplicitlySharedDataPointer::data() const
|
||||
pointer.data();
|
||||
|
||||
/* Check that these casts are possible. */
|
||||
static_cast<MyClass *>(pointer.data());
|
||||
static_cast<const MyClass *>(pointer.data());
|
||||
Q_UNUSED(static_cast<MyClass *>(pointer.data()));
|
||||
Q_UNUSED(static_cast<const MyClass *>(pointer.data()));
|
||||
}
|
||||
|
||||
/* Must not mutate the pointer. */
|
||||
@ -200,8 +200,8 @@ void tst_QExplicitlySharedDataPointer::data() const
|
||||
pointer.data();
|
||||
|
||||
/* Check that these casts are possible. */
|
||||
static_cast<MyClass *>(pointer.data());
|
||||
static_cast<const MyClass *>(pointer.data());
|
||||
Q_UNUSED(static_cast<MyClass *>(pointer.data()));
|
||||
Q_UNUSED(static_cast<const MyClass *>(pointer.data()));
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -252,7 +252,7 @@ void tst_QScopedPointer::negationOperatorSignature()
|
||||
!p;
|
||||
|
||||
/* The return value should be bool. */
|
||||
static_cast<bool>(!p);
|
||||
Q_UNUSED(static_cast<bool>(!p));
|
||||
}
|
||||
|
||||
void tst_QScopedPointer::operatorBool()
|
||||
@ -302,7 +302,7 @@ void tst_QScopedPointer::isNullSignature()
|
||||
const QScopedPointer<int> p(new int(69));
|
||||
|
||||
/* The signature should be const and return bool. */
|
||||
static_cast<bool>(p.isNull());
|
||||
Q_UNUSED(static_cast<bool>(p.isNull()));
|
||||
}
|
||||
|
||||
void tst_QScopedPointer::objectSize()
|
||||
|
Loading…
Reference in New Issue
Block a user