diff options
author | Jarek Kobus <jaroslaw.kobus@qt.io> | 2020-12-08 15:41:46 +0100 |
---|---|---|
committer | Jarek Kobus <jaroslaw.kobus@qt.io> | 2020-12-14 12:35:47 +0000 |
commit | cf010911f7681b23aff6a150c50252374447addd (patch) | |
tree | f1774eac8d9b3a8c66a00fe3b73163507660d20d /src/libs/utils/settingsaccessor.cpp | |
parent | 93dd966ce29720a295d01190f9b3f65e7d9043a8 (diff) |
Don't allocate unneeded temporary containers
Fix clazy warnings: allocating an unneeded temporary container
[clazy-container-anti-pattern]
Change-Id: I4b4c2c634eea650bbdf3c12d982a17f899fc94ec
Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
Reviewed-by: David Schulz <david.schulz@qt.io>
Reviewed-by: hjk <hjk@qt.io>
Diffstat (limited to 'src/libs/utils/settingsaccessor.cpp')
-rw-r--r-- | src/libs/utils/settingsaccessor.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/libs/utils/settingsaccessor.cpp b/src/libs/utils/settingsaccessor.cpp index ea967caeb3..5bcee4211c 100644 --- a/src/libs/utils/settingsaccessor.cpp +++ b/src/libs/utils/settingsaccessor.cpp @@ -49,8 +49,8 @@ namespace Utils { QMessageBox::StandardButtons SettingsAccessor::Issue::allButtons() const { QMessageBox::StandardButtons result = QMessageBox::NoButton; - for (const QMessageBox::StandardButton &b : buttons.keys()) - result |= b; + for (auto it = buttons.cbegin(); it != buttons.cend(); ++it) + result |= it.key(); return result; } |