Eliminate QQmlValueTypeProvider::createVariantFromString()

This can be expressed as constructing the variant from a QJSValue.

Change-Id: I3140958469423acdc498e26129e349bcfb601198
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>

3 files changed, 1 insertions, 23 deletions

diff --git a/src/qml/qml/qqmlglobal.cpp b/src/qml/qml/qqmlglobal.cpp
index 1fbb9889c2..73cc9abc93 100644
--- a/src/qml/qml/qqmlglobal.cpp
+++ b/src/qml/qml/qqmlglobal.cpp
@@ -93,22 +93,6 @@ bool QQmlValueTypeProvider::createValueFromString(int type, const QJSValue &s, Q
     return false;
 }
 
-QVariant QQmlValueTypeProvider::createVariantFromString(int type, const QString &s, bool *ok)
-{
-    QVariant v;
-
-    QQmlValueTypeProvider *p = this;
-    do {
-        if (p->variantFromString(type, s, &v)) {
-            if (ok) *ok = true;
-            return v;
-        }
-    } while ((p = p->next));
-
-    if (ok) *ok = false;
-    return QVariant();
-}
-
 QVariant QQmlValueTypeProvider::createVariantFromJsObject(int type, const QJSValue &obj, bool *ok)
 {
     QVariant v;
@@ -169,7 +153,6 @@ bool QQmlValueTypeProvider::writeValueType(int type, const void *src, QVariant&
 }
 
 bool QQmlValueTypeProvider::create(int, const QJSValue &, QVariant *) { return false; }
-bool QQmlValueTypeProvider::variantFromString(int, const QString &, QVariant *) { return false; }
 bool QQmlValueTypeProvider::store(int, const void *, void *, size_t) { return false; }
 
 struct ValueTypeProviderList {
diff --git a/src/qml/qml/qqmlglobal_p.h b/src/qml/qml/qqmlglobal_p.h
index 39ef990bd5..722a3f4989 100644
--- a/src/qml/qml/qqmlglobal_p.h
+++ b/src/qml/qml/qqmlglobal_p.h
@@ -226,8 +226,6 @@ public:
 
     QVariant createValueType(int, const QJSValue &params);
     bool createValueFromString(int, const QJSValue &, QVariant *);
-
-    QVariant createVariantFromString(int, const QString &, bool *);
     QVariant createVariantFromJsObject(int, const QJSValue &, bool *);
 
     bool equalValueType(int, const void *, const QVariant&);
@@ -237,9 +235,6 @@ public:
 
 private:
     virtual bool create(int, const QJSValue &params, QVariant *);
-
-    virtual bool variantFromString(int, const QString &, QVariant *);
-
     virtual bool store(int, const void *, void *, size_t);
 
     friend Q_QML_PRIVATE_EXPORT void QQml_addValueTypeProvider(QQmlValueTypeProvider *);
diff --git a/src/qml/qml/qqmlstringconverters.cpp b/src/qml/qml/qqmlstringconverters.cpp
index 07d6f4846d..74dd5f431c 100644
--- a/src/qml/qml/qqmlstringconverters.cpp
+++ b/src/qml/qml/qqmlstringconverters.cpp
@@ -76,7 +76,7 @@ QVariant QQmlStringConverters::variantFromString(const QString &s, int preferred
     case QMetaType::QRect:
         return QVariant::fromValue(rectFFromString(s, ok).toRect());
     default:
-        return QQml_valueTypeProvider()->createVariantFromString(preferredType, s, ok);
+        return QQml_valueTypeProvider()->createVariantFromJsObject(preferredType, QJSValue(s), ok);
     }
 }