diff options
author | Fabian Kosmale <fabian.kosmale@qt.io> | 2021-02-03 22:13:51 +0100 |
---|---|---|
committer | Fabian Kosmale <fabian.kosmale@qt.io> | 2021-03-12 18:03:30 +0000 |
commit | 022782d371965c9455af35915eaf87536bd6b759 (patch) | |
tree | 235eb5f3c2e3f9cf50b26db8eeb5d5c9aa79f36f | |
parent | 341654213bfd02a448490212e3b9b0d2027dfd2f (diff) |
Fix QMetaType benchmark
The benchmark used to crash because QMetaType::typeName would return an
empty string, which is not a legal value for newRow.
Change-Id: I9e6c6c1cf153943bfa21181cd2cca596a7943ea0
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
(cherry picked from commit 1316af2f3ed7861d0955793261a18f8ebf237f6f)
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
-rw-r--r-- | tests/benchmarks/corelib/kernel/qmetatype/tst_qmetatype.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/tests/benchmarks/corelib/kernel/qmetatype/tst_qmetatype.cpp b/tests/benchmarks/corelib/kernel/qmetatype/tst_qmetatype.cpp index abeaa79f26..126af8adde 100644 --- a/tests/benchmarks/corelib/kernel/qmetatype/tst_qmetatype.cpp +++ b/tests/benchmarks/corelib/kernel/qmetatype/tst_qmetatype.cpp @@ -248,9 +248,11 @@ void tst_QMetaType::isRegisteredNotRegistered() void tst_QMetaType::constructInPlace_data() { QTest::addColumn<int>("typeId"); - for (int i = QMetaType::FirstCoreType; i <= QMetaType::LastCoreType; ++i) - if (i != QMetaType::Void) - QTest::newRow(QMetaType::typeName(i)) << i; + for (int i = QMetaType::FirstCoreType; i <= QMetaType::LastCoreType; ++i) { + auto name = QMetaType::typeName(i); + if (name && i != QMetaType::Void) + QTest::newRow(name) << i; + } QTest::newRow("custom") << qMetaTypeId<BigClass>(); // GUI types are tested in tst_QGuiMetaType. |