diff options
author | gokulakrishnan-shankar <gokullan1101@gmail.com> | 2023-02-16 12:58:31 +0530 |
---|---|---|
committer | Andreas Heinisch <andreas.heinisch@yahoo.de> | 2023-03-11 11:25:19 +0000 |
commit | 20a5e38ee279022b7e6fbc32d2d259738244c0fd (patch) | |
tree | b745103ad12ac037faffc01dbc42ae8f9021a3a8 /basctl | |
parent | 60827355e483351886f7a96b5de12d810f35f0e6 (diff) |
tdf#145538 Refactor to use range-based for-loops
Replaced an existing for-loop (that used indexes to iterate over an array) with a range-based for-loop for better readability
Change-Id: I0cb112e6a41d38158deb99fb27f2b8b7f8852339
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/147145
Reviewed-by: Andreas Heinisch <andreas.heinisch@yahoo.de>
Tested-by: Andreas Heinisch <andreas.heinisch@yahoo.de>
Diffstat (limited to 'basctl')
-rw-r--r-- | basctl/source/basicide/baside2.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/basctl/source/basicide/baside2.cxx b/basctl/source/basicide/baside2.cxx index 1df230f9bd92..12a78d9b65f4 100644 --- a/basctl/source/basicide/baside2.cxx +++ b/basctl/source/basicide/baside2.cxx @@ -1591,10 +1591,10 @@ void ModulWindowLayout::SyntaxColors::NewConfig (bool bFirst) } bool bChanged = false; - for (unsigned i = 0; i != std::size(vIds); ++i) + for (const auto& vId: vIds) { - Color const aColor = aConfig.GetColorValue(vIds[i].eEntry).nColor; - Color& rMyColor = aColors[vIds[i].eTokenType]; + Color const aColor = aConfig.GetColorValue(vId.eEntry).nColor; + Color& rMyColor = aColors[vId.eTokenType]; if (bFirst || aColor != rMyColor) { rMyColor = aColor; |