summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrei Golubev <andrei.golubev@qt.io>2021-02-12 09:23:33 +0100
committerAndrei Golubev <andrei.golubev@qt.io>2021-02-12 11:59:25 +0100
commit9c282fe2e90eec05e160241069c219c7f09f4078 (patch)
tree0ed75df8be9e4802dea5b3d5a451548cfa53f6e7
parent4660f51f25a6c56182dff0ea24f7c1ad2bac9cea (diff)
Transform JSCallData args setting into a function
Change-Id: I4154a0b5c7115375292794e0564d2f3657e6b4dd Reviewed-by: Ulf Hermann <ulf.hermann@qt.io> Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
-rw-r--r--src/qml/CMakeLists.txt2
-rw-r--r--src/qml/jsruntime/qv4jscall.cpp81
-rw-r--r--src/qml/jsruntime/qv4jscall_p.h2
-rw-r--r--src/qml/qml/qqmlboundsignal.cpp22
4 files changed, 85 insertions, 22 deletions
diff --git a/src/qml/CMakeLists.txt b/src/qml/CMakeLists.txt
index 39c50e5050..b4156dd1a9 100644
--- a/src/qml/CMakeLists.txt
+++ b/src/qml/CMakeLists.txt
@@ -148,7 +148,7 @@ qt_internal_add_module(Qml
jsruntime/qv4include.cpp jsruntime/qv4include_p.h
jsruntime/qv4internalclass.cpp jsruntime/qv4internalclass_p.h
jsruntime/qv4iterator.cpp jsruntime/qv4iterator_p.h
- jsruntime/qv4jscall_p.h
+ jsruntime/qv4jscall_p.h jsruntime/qv4jscall.cpp
jsruntime/qv4jsonobject.cpp jsruntime/qv4jsonobject_p.h
jsruntime/qv4lookup.cpp jsruntime/qv4lookup_p.h
jsruntime/qv4managed.cpp jsruntime/qv4managed_p.h
diff --git a/src/qml/jsruntime/qv4jscall.cpp b/src/qml/jsruntime/qv4jscall.cpp
new file mode 100644
index 0000000000..ce91248f80
--- /dev/null
+++ b/src/qml/jsruntime/qv4jscall.cpp
@@ -0,0 +1,81 @@
+/****************************************************************************
+**
+** Copyright (C) 2021 The Qt Company Ltd.
+** Contact: https://www.qt.io/licensing/
+**
+** This file is part of the QtQml module of the Qt Toolkit.
+**
+** $QT_BEGIN_LICENSE:LGPL$
+** Commercial License Usage
+** Licensees holding valid commercial Qt licenses may use this file in
+** accordance with the commercial license agreement provided with the
+** Software or, alternatively, in accordance with the terms contained in
+** a written agreement between you and The Qt Company. For licensing terms
+** and conditions see https://www.qt.io/terms-conditions. For further
+** information use the contact form at https://www.qt.io/contact-us.
+**
+** GNU Lesser General Public License Usage
+** Alternatively, this file may be used under the terms of the GNU Lesser
+** General Public License version 3 as published by the Free Software
+** Foundation and appearing in the file LICENSE.LGPL3 included in the
+** packaging of this file. Please review the following information to
+** ensure the GNU Lesser General Public License version 3 requirements
+** will be met: https://www.gnu.org/licenses/lgpl-3.0.html.
+**
+** GNU General Public License Usage
+** Alternatively, this file may be used under the terms of the GNU
+** General Public License version 2.0 or (at your option) the GNU General
+** Public license version 3 or any later version approved by the KDE Free
+** Qt Foundation. The licenses are as published by the Free Software
+** Foundation and appearing in the file LICENSE.GPL2 and LICENSE.GPL3
+** included in the packaging of this file. Please review the following
+** information to ensure the GNU General Public License requirements will
+** be met: https://www.gnu.org/licenses/gpl-2.0.html and
+** https://www.gnu.org/licenses/gpl-3.0.html.
+**
+** $QT_END_LICENSE$
+**
+****************************************************************************/
+
+#include "qv4jscall_p.h"
+
+#include <private/qqmlengine_p.h>
+#include <private/qv4qobjectwrapper_p.h>
+
+QT_BEGIN_NAMESPACE
+
+/*! \internal
+
+ Sets the arguments of JSCallData from type erased \a args based on type
+ information provided by \a types
+ */
+void QV4::populateJSCallArguments(QQmlEnginePrivate *ep, ExecutionEngine *v4, JSCallData &jsCall,
+ void **args, int *types)
+{
+ const int argCount = types ? types[0] : 0;
+ for (int ii = 0; ii < argCount; ++ii) {
+ int type = types[ii + 1];
+ //### ideally we would use metaTypeToJS, however it currently gives different results
+ // for several cases (such as QVariant type and QObject-derived types)
+ // args[ii] = v4->metaTypeToJS(type, args[ii + 1]);
+ if (type == qMetaTypeId<QJSValue>()) {
+ jsCall.args[ii] = QJSValuePrivate::convertToReturnedValue(
+ v4, *reinterpret_cast<QJSValue *>(args[ii + 1]));
+ } else if (type == QMetaType::QVariant) {
+ jsCall.args[ii] = v4->fromVariant(*((QVariant *)args[ii + 1]));
+ } else if (type == QMetaType::Int) {
+ //### optimization. Can go away if we switch to metaTypeToJS, or be expanded otherwise
+ jsCall.args[ii] = QV4::Value::fromInt32(*reinterpret_cast<const int *>(args[ii + 1]));
+ } else if (ep->isQObject(type)) {
+ if (!*reinterpret_cast<void *const *>(args[ii + 1]))
+ jsCall.args[ii] = QV4::Value::nullValue();
+ else
+ jsCall.args[ii] = QV4::QObjectWrapper::wrap(
+ v4, *reinterpret_cast<QObject *const *>(args[ii + 1]));
+ } else {
+ jsCall.args[ii] = v4->fromVariant(QVariant(QMetaType(type), args[ii + 1]));
+ }
+ }
+}
+
+QT_END_NAMESPACE
diff --git a/src/qml/jsruntime/qv4jscall_p.h b/src/qml/jsruntime/qv4jscall_p.h
index 31689b1ba1..ffc666d1e5 100644
--- a/src/qml/jsruntime/qv4jscall_p.h
+++ b/src/qml/jsruntime/qv4jscall_p.h
@@ -112,6 +112,8 @@ ReturnedValue FunctionObject::call(const JSCallData &data) const
return call(data.thisObject, data.args, data.argc);
}
+void populateJSCallArguments(QQmlEnginePrivate *ep, ExecutionEngine *v4, JSCallData &jsCall,
+ void **args, int *types);
struct ScopedStackFrame {
Scope &scope;
diff --git a/src/qml/qml/qqmlboundsignal.cpp b/src/qml/qml/qqmlboundsignal.cpp
index 8fceb94f45..1bc0efc5d9 100644
--- a/src/qml/qml/qqmlboundsignal.cpp
+++ b/src/qml/qml/qqmlboundsignal.cpp
@@ -196,27 +196,7 @@ void QQmlBoundSignalExpression::evaluate(void **a)
int argCount = argsTypes ? *argsTypes : 0;
QV4::JSCallData jsCall(scope, argCount);
- for (int ii = 0; ii < argCount; ++ii) {
- int type = argsTypes[ii + 1];
- //### ideally we would use metaTypeToJS, however it currently gives different results
- // for several cases (such as QVariant type and QObject-derived types)
- //args[ii] = engine->metaTypeToJS(type, a[ii + 1]);
- if (type == qMetaTypeId<QJSValue>()) {
- jsCall->args[ii] = QJSValuePrivate::convertToReturnedValue(v4, *reinterpret_cast<QJSValue *>(a[ii + 1]));
- } else if (type == QMetaType::QVariant) {
- jsCall->args[ii] = scope.engine->fromVariant(*((QVariant *)a[ii + 1]));
- } else if (type == QMetaType::Int) {
- //### optimization. Can go away if we switch to metaTypeToJS, or be expanded otherwise
- jsCall->args[ii] = QV4::Value::fromInt32(*reinterpret_cast<const int*>(a[ii + 1]));
- } else if (ep->isQObject(type)) {
- if (!*reinterpret_cast<void* const *>(a[ii + 1]))
- jsCall->args[ii] = QV4::Value::nullValue();
- else
- jsCall->args[ii] = QV4::QObjectWrapper::wrap(v4, *reinterpret_cast<QObject* const *>(a[ii + 1]));
- } else {
- jsCall->args[ii] = scope.engine->fromVariant(QVariant(QMetaType(type), a[ii + 1]));
- }
- }
+ populateJSCallArguments(ep, v4, jsCall, a, argsTypes);
QQmlJavaScriptExpression::evaluate(jsCall.callData(), nullptr);