[Libreoffice-commits] core.git: basctl/source

[gokulakrishnan-shankar (via logerrit)]

 basctl/source/basicide/baside2.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 20a5e38ee279022b7e6fbc32d2d259738244c0fd
Author:     gokulakrishnan-shankar <gokullan1...@gmail.com>
AuthorDate: Thu Feb 16 12:58:31 2023 +0530
Commit:     Andreas Heinisch <andreas.heini...@yahoo.de>
CommitDate: Sat Mar 11 11:25:19 2023 +0000

    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.heini...@yahoo.de>
    Tested-by: Andreas Heinisch <andreas.heini...@yahoo.de>

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;