diff options
| author | Fabian Kosmale <fabian.kosmale@qt.io> | 2021-02-03 22:13:51 +0100 |
|---|---|---|
| committer | Fabian Kosmale <fabian.kosmale@qt.io> | 2021-02-04 13:29:46 +0100 |
| commit | 1316af2f3ed7861d0955793261a18f8ebf237f6f (patch) | |
| tree | 1a6a4886689f585f5eadd1cedfc2a522d5b2581a /tests/benchmarks/corelib/kernel/qmetatype/tst_qmetatype.cpp | |
| parent | f30a4f0be7c98a93964caa329b3bfe036cefe566 (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>
Diffstat (limited to 'tests/benchmarks/corelib/kernel/qmetatype/tst_qmetatype.cpp')
| -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. |