aboutsummaryrefslogtreecommitdiffstats
path: root/src/qml/jsruntime/qv4qobjectwrapper.cpp
diff options
context:
space:
mode:
authorUlf Hermann <ulf.hermann@qt.io>2022-09-30 16:34:28 +0200
committerUlf Hermann <ulf.hermann@qt.io>2022-10-07 15:17:06 +0200
commite8e03215654ca730243336a80453cf9396cbdf58 (patch)
tree011ffaecb0d016ee288891e12351f83a95eee25a /src/qml/jsruntime/qv4qobjectwrapper.cpp
parent81faddec9c6607834da4fdb931f81f29e1f7ac69 (diff)
QML: Drop the "succeeded" out parameters from SequencePrototype
The success of the operation is visible from the return value in all cases. Change-Id: I93177785f76b8078ddd8eeb7d77143993fe80739 Reviewed-by: Semih Yavuz <semih.yavuz@qt.io> Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'src/qml/jsruntime/qv4qobjectwrapper.cpp')
-rw-r--r--src/qml/jsruntime/qv4qobjectwrapper.cpp5
1 files changed, 2 insertions, 3 deletions
diff --git a/src/qml/jsruntime/qv4qobjectwrapper.cpp b/src/qml/jsruntime/qv4qobjectwrapper.cpp
index 376332179c..ef8c8c76d6 100644
--- a/src/qml/jsruntime/qv4qobjectwrapper.cpp
+++ b/src/qml/jsruntime/qv4qobjectwrapper.cpp
@@ -168,11 +168,10 @@ static ReturnedValue loadProperty(ExecutionEngine *v4, QObject *object,
}
// see if it's a sequence type
- bool succeeded = false;
QV4::ScopedValue retn(scope, QV4::SequencePrototype::newSequence(
v4, propMetaType, object, property.coreIndex(),
- !property.isWritable(), &succeeded));
- if (succeeded)
+ !property.isWritable()));
+ if (!retn->isUndefined())
return retn->asReturnedValue();
if (!propMetaType.isValid()) {