Modernize a bit accessibility and basctl

by using for range loops
+ compare end iterator with "!=" instead of "<"

Change-Id: I615a8ca417a58fde4976bb6d8d9112dc5d993a88
Reviewed-on: https://gerrit.libreoffice.org/50277
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

1 files changed, 3 insertions, 3 deletions

diff --git a/basctl/source/basicide/layout.cxx b/basctl/source/basicide/layout.cxx
index ae19f7695d4d..54195c564898 100644
--- a/basctl/source/basicide/layout.cxx
+++ b/basctl/source/basicide/layout.cxx
@@ -175,10 +175,10 @@ Layout::SplittedSide::SplittedSide (Layout* pParent, Side eSide) :
 void Layout::SplittedSide::dispose()
 {
     aSplitter.disposeAndClear();
-    for (auto i = vItems.begin(); i != vItems.end(); ++i)
+    for (auto & item : vItems)
     {
-        i->pSplit.disposeAndClear();
-        i->pWin.clear();
+        item.pSplit.disposeAndClear();
+        item.pWin.clear();
     }
 }