diff options
author | Kent Hansen <kent.hansen@nokia.com> | 2012-03-20 19:08:39 +0100 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-03-21 13:38:03 +0100 |
commit | e2502e1a06fbb15a8d0abe8ac1be5e6dec1e1152 (patch) | |
tree | 4bd2bed45ab4983dcf0f3db8730f4cd4e0e91eee /src/testlib/qtestcase.cpp | |
parent | d4258bc2385f8ad3284b009b768c16f12e357a3b (diff) |
Use the new QMetaMethod API in testlib
Use QMetaMethod::name() instead of parsing the signature.
Use QMetaMethod::returnType() instead of checking the length of
typeName().
Use QMetaMethod::parameterCount() instead of checking the
size of parameterTypes().
Change-Id: I424370b19b5b150865377666dca0fba5f29ad30f
Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@nokia.com>
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>
Diffstat (limited to 'src/testlib/qtestcase.cpp')
-rw-r--r-- | src/testlib/qtestcase.cpp | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/src/testlib/qtestcase.cpp b/src/testlib/qtestcase.cpp index 962814085f..08dd4a9f80 100644 --- a/src/testlib/qtestcase.cpp +++ b/src/testlib/qtestcase.cpp @@ -1095,20 +1095,16 @@ int Q_TESTLIB_EXPORT defaultKeyDelay() static bool isValidSlot(const QMetaMethod &sl) { - if (sl.access() != QMetaMethod::Private || !sl.parameterTypes().isEmpty() - || qstrlen(sl.typeName()) || sl.methodType() != QMetaMethod::Slot) + if (sl.access() != QMetaMethod::Private || sl.parameterCount() != 0 + || sl.returnType() != QMetaType::Void || sl.methodType() != QMetaMethod::Slot) return false; - QByteArray signature = sl.methodSignature(); - const char *sig = signature.constData(); - int len = qstrlen(sig); - if (len < 2) + QByteArray name = sl.name(); + if (name.isEmpty()) return false; - if (sig[len - 2] != '(' || sig[len - 1] != ')') + if (name.endsWith("_data")) return false; - if (len > 7 && strcmp(sig + (len - 7), "_data()") == 0) - return false; - if (strcmp(sig, "initTestCase()") == 0 || strcmp(sig, "cleanupTestCase()") == 0 - || strcmp(sig, "cleanup()") == 0 || strcmp(sig, "init()") == 0) + if (name == "initTestCase" || name == "cleanupTestCase" + || name == "cleanup" || name == "init") return false; return true; } |