De-inline QVariant(float)
authorHarald Fernengel <harald.fernengel@nokia.com>
Wed, 22 Aug 2012 11:34:21 +0000 (13:34 +0200)
committerQt by Nokia <qt-info@nokia.com>
Thu, 23 Aug 2012 00:55:17 +0000 (02:55 +0200)
All QVariant constructors are not inline, so change QVariant(float) as
well. This allows for refactoring of QVariant during lifetime of Qt 5
without breaking QVariant(float) case.

Change-Id: Icd1fc331a1ef8a2872be811aa7d1160406038f06
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@nokia.com>
src/corelib/kernel/qvariant.cpp
src/corelib/kernel/qvariant.h

index fb1292c..b899cd0 100644 (file)
@@ -1448,6 +1448,9 @@ QVariant::QVariant(bool val)
 QVariant::QVariant(double val)
     : d(Double)
 { d.data.d = val; }
+QVariant::QVariant(float val)
+    : d(QMetaType::Float)
+{ d.data.f = val; }
 
 QVariant::QVariant(const QByteArray &val)
     : d(ByteArray)
index 6d7d56e..bde67ed 100644 (file)
@@ -210,7 +210,7 @@ class Q_CORE_EXPORT QVariant
     QVariant(qulonglong ull);
     QVariant(bool b);
     QVariant(double d);
-    QVariant(float f) { d.is_null = false; d.type = QMetaType::Float; d.data.f = f; }
+    QVariant(float f);
 #ifndef QT_NO_CAST_FROM_ASCII
     QT_ASCII_CAST_WARN QVariant(const char *str);
 #endif