diff options
author | Marc Mutz <marc.mutz@qt.io> | 2023-08-08 07:56:32 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@qt.io> | 2023-08-08 17:44:56 +0000 |
commit | f493b72ee7e6c525159a6df5df13536021ed6b91 (patch) | |
tree | ab41becbd7186fce2ff5da91b73b9c8642fdb64c /tests/auto/gui/util/qdoublevalidator/tst_qdoublevalidator.cpp | |
parent | ba4b88d9d2f1dd03906f445c55f8959e0165505f (diff) |
DynamicTreeModel test helper: port away from Q_FOREACH
This source file is included in four other test projects, so it makes
more sense to port it away from Q_FOREACH than to white-list it
everywhere it's used.
The change in ModelMoveCommand::doCommand() is trivial. I only dropped
the pointless top-level const of the loop variable as a drive-by.
The change regarding `parents` in ModelChangeChildrenLayoutsCommand's
doCommand() is also trivial, I just ported to braced initialization to
get the QList to be const. We're forced by the Qt API
(layoutChanged()) to use a QList here, therefore no array.
Ibid., the change regarding `persistent` is simple. The container
cannot be marked as const without a lot of churn to its initialization
(applying IILE, basically), but other than that, it's a local
variable, and the loop body clearly doesn't modify it.
Task-number: QTBUG-115839
Pick-to: 6.6 6.5
Change-Id: I7a0e85804626a3cc612921b49e72e4b9f30b676d
Reviewed-by: Ivan Solovev <ivan.solovev@qt.io>
Diffstat (limited to 'tests/auto/gui/util/qdoublevalidator/tst_qdoublevalidator.cpp')
0 files changed, 0 insertions, 0 deletions