[qt/qt/qtremoteobjects]: Summary of bulk changes made

KDE Git Services - Bulk Change <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git repository change summary for qt/qt/qtremoteobjects
Pushed by mirror-service into branch '6.11'.
Changed from a91e34bf589e856010ed2a95d538b1994f2d114f to 2e0ef099d920fae892ffdf646167d9ca1e583d6e
Acknowledgement was received that this change introduces only existing code that has been pushed to another public open source repository.

This change contains the following new commits:

Git commit 0b1e59603e1e9185cf6778cc5e7d860c73d1dc53 by Qt Cherry-pick Bot (on behalf of Ivan Solovev) on 27/07/2026 at 15:03..
Remove unused variables in qremoteobjectnode.cpp

Found by Axivion static analysis.

Commit e77c98352c584bac7b752dde21c93ff09f78b00d started deserializing
the data directly into a ClassSlot instance, but didn't remove the
now unused local variables. Do it now.

The original commit was merged into 6.9, so picking only down to 6.11.

Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV95
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV96
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV97
Task-number: QTBUG-144772
Change-Id: I093605cc0bf83d59aba637a256bfb50156d4397a
Reviewed-by: Tatiana Borisova <[email protected]>
(cherry picked from commit 4c8f01c2740c97dc53f6baad9fb34ecb88051047)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
(cherry picked from commit 875d57e3b13390787292cd6495158a3365b0df52)
https://invent.kde.org/qt/qt/qtremoteobjects/-/commit/0b1e59603e1e9185cf6778cc5e7d860c73d1dc53

Git commit b720f5b93560d386e8618e0a2e436cfdfb2fc031 by Qt Cherry-pick Bot (on behalf of Ivan Solovev) on 27/07/2026 at 15:03..
qremoteobjectsource.cpp: be more explicit on what we delete

In the ranged-for loops over m_children, each 'it' variable is
a QPointer. Calling delete on it is perfectly fine, because it goes
via operator T*().
However, the follow-up change will take 'it' by const-ref instead of
by-value, and then the code will be a bit confusing (i.e. calling
delete on a const ref).

To make it more readable, explicitly call data(), to signal that
we're deleting the underlying data.

Amends d9ee9fff1b9291395ba7ab75d7e46ddf4a9ff3a4.

Pick-to: 6.8
Change-Id: I2a9f7d8eff55a7101d7b067a6454e3517b4cc013
Reviewed-by: Tatiana Borisova <[email protected]>
(cherry picked from commit 5ae43a67a5a6ebc5e7fd33dea0a1870c8692ad8a)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
(cherry picked from commit b7f0da83a1cff48ca3421e404e1d4cc9836bae3c)
https://invent.kde.org/qt/qt/qtremoteobjects/-/commit/b720f5b93560d386e8618e0a2e436cfdfb2fc031

Git commit c9c9a686db99e2f2c4f548bc8692f86ea306e89d by Qt Cherry-pick Bot (on behalf of Ivan Solovev) on 27/07/2026 at 15:03..
Fix Qt-RangeLoopReference warnings

All cases use ranged-for loop to iterate over non-trivially-copyable
data (QVariant, QByteArray, QPointer).
In all cases it's better to take the values by const-ref, rather than
do actual copies.

Amends multiple patches, but all of them preceed Qt 6.8, so it's safe
to pick down to that branch.

Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV78
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV88
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV89
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV90
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV91
Task-number: QTBUG-144772
Pick-to: 6.8
Change-Id: I24dea8db3dc562dd819e6add1bd4094f0caf73db
Reviewed-by: Tatiana Borisova <[email protected]>
(cherry picked from commit fa8a8a18627c38b99a2fdaadfce72f3637e73ba3)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
(cherry picked from commit a128c2a7923aac05baec34ba07e2978a25c6c764)
https://invent.kde.org/qt/qt/qtremoteobjects/-/commit/c9c9a686db99e2f2c4f548bc8692f86ea306e89d

Git commit 4804b3b5523a22f27c1b68b393a0cb7497ffa9c4 by Qt Cherry-pick Bot (on behalf of Ivan Solovev) on 27/07/2026 at 15:03..
Fix Qt-RangeLoopDetach warnings: trivial cases

To prevent detaching, wrap the containers into std::as_const().
If the container is an rvalue, it's impossible to use std::as_const().
In such cases introduce a local const variable.

Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV248
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV249
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV250
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV255
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV256
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV257
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV291
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV328
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV329
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV330
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV331
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV332
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV334
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV335
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV336
Task-number: QTBUG-144772
Pick-to: 6.8
Change-Id: I87d1fd52aaa219c0c75671765d421531dca68ca3
Reviewed-by: Tatiana Borisova <[email protected]>
(cherry picked from commit b9e9ccba9a7ec83fbcf95bfd07c1afde08a8ad09)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
(cherry picked from commit fc0b892c95c0aa31dd6962413772351624d1cf94)
https://invent.kde.org/qt/qt/qtremoteobjects/-/commit/4804b3b5523a22f27c1b68b393a0cb7497ffa9c4

Git commit f53102243225abb7ab9bedc4de3f7818144c9b0f by Qt Cherry-pick Bot (on behalf of Ivan Solovev) on 27/07/2026 at 15:04..
Fix Qt-RangeLoopDetach: iteration over QMap::keys()

A loop that iterates a map over QMap::keys() and then accesses the
elements via operator[] can be more efficiently replaced with
QMap::asKeyValueRange().

Amends e8cadf8f88128758725ef7498ed7164751095832.
QMap::asKeyValueRange() is available since Qt 6.4, so we can pick the
change to all active branches.

Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV333
Task-number: QTBUG-144772
Pick-to: 6.8
Change-Id: Idf19ceb35784cd062dc591264b03c9097c189744
Reviewed-by: Tatiana Borisova <[email protected]>
(cherry picked from commit 8028972dc58e7ad776407c6136568125703cca4c)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
(cherry picked from commit c0c08d80817fa9ebc2e5291209af10bb54f487c9)
https://invent.kde.org/qt/qt/qtremoteobjects/-/commit/f53102243225abb7ab9bedc4de3f7818144c9b0f

Git commit 395b96dc9b5168c1820ecb4f00716ae6611e87ea by Qt Cherry-pick Bot (on behalf of Ivan Solovev) on 27/07/2026 at 15:04..
Fix Qt-RangeLoopDetach warning in structured bindings

Several range-for loops are iterating over variables obtained via
structured bindings. The warning was triggered because the variables
are non-const.

Could be fixed in a usual way by wrapping the usages in std::as_const(),
but a better solution would be to initially declare them const.

Amends e77c98352c584bac7b752dde21c93ff09f78b00d
and c03d630e6d9631791d0668b64025bd82796a55ce.
Both of them do not exist in 6.8, so picking only down to 6.11.

Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV251
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV252
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV253
Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV254
Task-number: QTBUG-144772
Change-Id: Ia0fb7da547798d3a05a5c928f423692dc7706d28
Reviewed-by: Tatiana Borisova <[email protected]>
Reviewed-by: Dennis Oberst <[email protected]>
(cherry picked from commit b4d8c52362e498622b3627bd501b65103ae6ad5d)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
(cherry picked from commit 14c6771b7235949d8e16b27dfc5e4f16a6ba891e)
https://invent.kde.org/qt/qt/qtremoteobjects/-/commit/395b96dc9b5168c1820ecb4f00716ae6611e87ea

Git commit 2e0ef099d920fae892ffdf646167d9ca1e583d6e by Qt Cherry-pick Bot (on behalf of Ivan Solovev) on 27/07/2026 at 15:04..
qremoteobjectnode.cpp: do not use auto when constructing QBA

It deduces the return type to QStringBuilder instead of QByteArray.

Originally pointed out by Axivion, which complains that it can cause
a crash. The complaint is not valid since
qtbase/af8f9a2a6e3dbc865470f8d7f09b533ccea4f353.

However, it is still better to explicitly use QByteArray here.
QStringBuilder does the concatenation every time the variable is used,
instead of doing it once. In this case the variable is used twice, so
using QStringBuilder is less efficient then immediately converting
the result to QByteArray.

Amends c03d630e6d9631791d0668b64025bd82796a55ce (picked to 6.9).

Axivion-Id: qt_Add-ons_qtremoteobjects_dev_linux:SV100
Fixes: QTBUG-144772
Change-Id: I9f807bce7bfa921b02dc07f49d86e303eb38f9b6
Reviewed-by: Marc Mutz <[email protected]>
(cherry picked from commit f2331eb16d9bd565a8f72ca0e6ee06187899cfa1)
Reviewed-by: Qt Cherry-pick Bot <[email protected]>
(cherry picked from commit 74d8479ba0405e51bfd4315812f39cdf8d83f973)
https://invent.kde.org/qt/qt/qtremoteobjects/-/commit/2e0ef099d920fae892ffdf646167d9ca1e583d6e
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.