diff options
| author | Giuseppe D'Angelo <giuseppe.dangelo@kdab.com> | 2021-09-06 20:40:17 +0200 |
|---|---|---|
| committer | Giuseppe D'Angelo <giuseppe.dangelo@kdab.com> | 2021-09-18 09:41:49 +0200 |
| commit | 51e28dc54fab3caa9e2b3bea4fd7a2fb5724d7c7 (patch) | |
| tree | ebf9f48b5ad026220949fd69b7b0ceb582aa541e /src/corelib/io/qprocess_win.cpp | |
| parent | 33238ea2c63b372ee8795eaafbfc5a859a778f8d (diff) | |
QIconLoader: code tidies
Turn QThemeIconEntries into an owning container (std::vector of
unique_ptr), so that code using QThemeIconInfo doesn't have to
manage ownership (and forget to do so, and cause bugs like
QTBUG-93050).
The fallout is mostly on isEmpty() vs empty(); as drive-by fixes:
* use auto;
* use make_unique (no raw news);
* turn a few indexed loops into range-based ones;
* streamline an if-else-if chain;
* turn a !(a == b) condition into a != b.
Change-Id: Ie3ac9de57c80ed3184ec0d15c847f81306ef48ca
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Diffstat (limited to 'src/corelib/io/qprocess_win.cpp')
0 files changed, 0 insertions, 0 deletions