path: root/examples
diff options
authorMarc Mutz <>2019-08-15 14:42:12 +0200
committerMarc Mutz <>2019-08-17 09:51:29 +0200
commitcb3e1e551f340ce1e6280123d8b5411b3c1c96d8 (patch)
tree01e5a381ed47ee55702055a25ed0f62b641c5c0e /examples
parent3e79151fa2552edd2e58c246d136f1fd0b2a4c86 (diff)
QWidgetBackingStore: clean up around dirtyOnScreenWidgets
The QVector dirtyOnScreenWidgets was aggregated by pointer, which makes no sense, as a QVector is just as large as a pointer (and even in Qt 6, when it will be larger, it's not going to be horrible). But this complicated the code quite a bit. Aggregate by value instead (it's just one of three such vectors now). Drive-by fixes: - use QVector::removeAll() instead of rolling your own - port two indexed loops to ranged ones. In the first case, it's safe, as the loop body clearly doesn't touch the iteratee (it's just a std::accumulate). In the second, the question no longer applies, as we're now using a consume loop. Change-Id: Icd4ac13bb4a6f9a783f0adf2fb6a5bdfacd1f91a Reviewed-by: Friedemann Kleint <> Reviewed-by: Volker Hilsheimer <>
Diffstat (limited to 'examples')
0 files changed, 0 insertions, 0 deletions