Use QMetaType API directly instead of going through QVariant.
authorJędrzej Nowacki <jedrzej.nowacki@nokia.com>
Tue, 15 May 2012 13:42:26 +0000 (15:42 +0200)
committerQt by Nokia <qt-info@nokia.com>
Fri, 18 May 2012 18:25:11 +0000 (20:25 +0200)
Change-Id: I418ccca7cb6e2aa1ba678e24dd36b39ebecadcbe
Reviewed-by: Stephen Kelly <stephen.kelly@kdab.com>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
src/dbus/qdbusmarshaller.cpp

index 145d991..b7968a2 100644 (file)
@@ -176,7 +176,7 @@ inline bool QDBusMarshaller::append(const QDBusVariant &arg)
     }
 
     const QVariant &value = arg.variant();
-    QVariant::Type id = QVariant::Type(value.userType());
+    int id = value.userType();
     if (id == QVariant::Invalid) {
         qWarning("QDBusMarshaller: cannot add a null QDBusVariant");
         error(QLatin1String("Variant containing QVariant::Invalid passed in arguments"));
@@ -185,7 +185,7 @@ inline bool QDBusMarshaller::append(const QDBusVariant &arg)
 
     QByteArray tmpSignature;
     const char *signature = 0;
-    if (int(id) == QDBusMetaTypeId::argument) {
+    if (id == QDBusMetaTypeId::argument) {
         // take the signature from the QDBusArgument object we're marshalling
         tmpSignature =
             qvariant_cast<QDBusArgument>(value).currentSignature().toLatin1();
@@ -197,9 +197,9 @@ inline bool QDBusMarshaller::append(const QDBusVariant &arg)
     if (!signature) {
         qWarning("QDBusMarshaller: type `%s' (%d) is not registered with D-BUS. "
                  "Use qDBusRegisterMetaType to register it",
-                 QVariant::typeToName( id ), id);
+                 QMetaType::typeName(id), id);
         error(QString::fromLatin1("Unregistered type %1 passed in arguments")
-              .arg(QLatin1String(QVariant::typeToName(id))));
+              .arg(QLatin1String(QMetaType::typeName(id))));
         return false;
     }