diff options
author | Brett Stottlemyer <bstottle@ford.com> | 2021-07-20 07:44:55 -0400 |
---|---|---|
committer | Brett Stottlemyer <bstottle@ford.com> | 2021-08-04 10:42:18 -0400 |
commit | f43e2918d05df688c19d3164b43418747d4ac2ef (patch) | |
tree | 4ea15423c76fc79db30f0c159d465bbd210fabf5 /src/remoteobjects/qremoteobjectabstractitemmodelreplica.cpp | |
parent | 6892ef6d4577410385e315a4366aa3b6e449c435 (diff) |
Refactor decodeVariant to pass by rvalue ref/std::move
The decodeVariant calls were weird in how their parameters were pass by
reference so they could be changed, but without forcing a copy. It is
clearer to pass by rvalue ref, using std::move on the calling side.
This rippled up to calls that called the method, including setProperties
which is part of the repc generated code.
Because of the repc change, this should *not* be picked to Qt5.
Pick-to: 6.2
Change-Id: Ieb1b0620569ad8eb9797edc57cc189d0b426510c
Reviewed-by: Sona Kurazyan <sona.kurazyan@qt.io>
Diffstat (limited to 'src/remoteobjects/qremoteobjectabstractitemmodelreplica.cpp')
-rw-r--r-- | src/remoteobjects/qremoteobjectabstractitemmodelreplica.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/remoteobjects/qremoteobjectabstractitemmodelreplica.cpp b/src/remoteobjects/qremoteobjectabstractitemmodelreplica.cpp index 8e2a8c0..099415f 100644 --- a/src/remoteobjects/qremoteobjectabstractitemmodelreplica.cpp +++ b/src/remoteobjects/qremoteobjectabstractitemmodelreplica.cpp @@ -105,7 +105,7 @@ void QAbstractItemModelReplicaImplementation::initialize() QVariantList properties; properties << QVariant::fromValue(QList<int>()); properties << QVariant::fromValue(QIntHash()); - setProperties(properties); + setProperties(std::move(properties)); } void QAbstractItemModelReplicaImplementation::registerMetatypes() |