diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2019-05-23 09:38:56 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2019-05-23 10:09:27 +0200 |
commit | 18e7e82d3f23a2a6954308c7a7c49ec23344b593 (patch) | |
tree | 48637fe40f763b90107b21cdffe0eb768cf5db17 /tests/benchmarks/gui | |
parent | ac608b7bd2fc3876d1a521cb3542b6b56133fd0f (diff) |
Remove the last uses of Java-style iterators in QtCore
They are going to be deprecated.
Add a strategic break. This was a pre-existing problem: the comment
claims that h is invalid (though I personally don't see it), but
then goes on to check the loop condition (which, in the mutable
Java iterator case, involves calling h.cend()). We now cache the
end iterator, so if there ever was a problem, it's probably a
lesser one now, but it's still not kosher, and a debug version of
QHash would find it, so break out explicitly.
Saves ~200b in text size on optimized GCC 9.1 Linux AMD64 builds,
ie. ~100b per loop.
Change-Id: I7684485b55fb23a8cf882f89621ebb75a0e607b5
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
Diffstat (limited to 'tests/benchmarks/gui')
0 files changed, 0 insertions, 0 deletions