diff options
author | Ulf Hermann <ulf.hermann@qt.io> | 2022-09-30 16:34:28 +0200 |
---|---|---|
committer | Ulf Hermann <ulf.hermann@qt.io> | 2022-10-07 15:17:06 +0200 |
commit | e8e03215654ca730243336a80453cf9396cbdf58 (patch) | |
tree | 011ffaecb0d016ee288891e12351f83a95eee25a /src/qmlworkerscript | |
parent | 81faddec9c6607834da4fdb931f81f29e1f7ac69 (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/qmlworkerscript')
-rw-r--r-- | src/qmlworkerscript/qv4serialize.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/qmlworkerscript/qv4serialize.cpp b/src/qmlworkerscript/qv4serialize.cpp index fda96cf763..c0f34ab1fc 100644 --- a/src/qmlworkerscript/qv4serialize.cpp +++ b/src/qmlworkerscript/qv4serialize.cpp @@ -384,7 +384,6 @@ ReturnedValue Serialize::deserialize(const char *&data, ExecutionEngine *engine) case WorkerSequence: { ScopedValue value(scope); - bool succeeded = false; quint32 length = headersize(header); quint32 seqLength = length - 1; value = deserialize(data, engine); @@ -396,8 +395,8 @@ ReturnedValue Serialize::deserialize(const char *&data, ExecutionEngine *engine) array->arrayPut(ii, value); } array->setArrayLengthUnchecked(seqLength); - QVariant seqVariant = QV4::SequencePrototype::toVariant(array, QMetaType(sequenceType), &succeeded); - return QV4::SequencePrototype::fromVariant(engine, seqVariant, &succeeded); + QVariant seqVariant = QV4::SequencePrototype::toVariant(array, QMetaType(sequenceType)); + return QV4::SequencePrototype::fromVariant(engine, seqVariant); } } Q_ASSERT(!"Unreachable"); |