2 files changed, 8 insertions, 8 deletions
diff --git a/basctl/source/basicide/baside2b.cxx b/basctl/source/basicide/baside2b.cxx
index a82d57c5266f..c02bac26d0f2 100644
--- a/basctl/source/basicide/baside2b.cxx
+++ b/basctl/source/basicide/baside2b.cxx
@@ -2891,12 +2891,12 @@ UnoTypeCodeCompletetor::UnoTypeCodeCompletetor( const std::vector< OUString >& a
         return;
     }
 
-    unsigned int j = 1;//start from aVect[1]: aVect[0] is the variable name
+    auto j = aVect.begin() + 1;//start from aVect[1]: aVect[0] is the variable name
     OUString sMethName;
 
-    while( j != aVect.size() )
+    while( j != aVect.end() )
     {
-        sMethName = aVect[j];
+        sMethName = *j;
 
         if( CodeCompleteOptions::IsExtendedTypeDeclaration() )
         {
diff --git a/basctl/source/basicide/layout.cxx b/basctl/source/basicide/layout.cxx
index b93e50ae3395..6d87bf687ccb 100644
--- a/basctl/source/basicide/layout.cxx
+++ b/basctl/source/basicide/layout.cxx
@@ -216,7 +216,7 @@ void Layout::SplittedSide::Add (DockingWindow* pWin, Size const& rSize)
 void Layout::SplittedSide::Remove (DockingWindow* pWin)
 {
     // contains?
-    unsigned iWin;
+    std::vector<Item>::size_type iWin;
     for (iWin = 0; iWin != vItems.size(); ++iWin)
         if (vItems[iWin].pWin == pWin)
             break;
@@ -251,8 +251,8 @@ bool Layout::SplittedSide::IsDocking (DockingWindow const& rWin)
 // IsEmpty() -- are there no windows docked in this strip?
 bool Layout::SplittedSide::IsEmpty () const
 {
-    for (unsigned i = 0; i != vItems.size(); ++i)
-        if (IsDocking(*vItems[i].pWin))
+    for (auto const & i: vItems)
+        if (IsDocking(*i.pWin))
             return false;
     return true;
 }
@@ -301,9 +301,9 @@ void Layout::SplittedSide::ArrangeIn (Rectangle const& rRect)
     // positioning separator lines and windows
     bool bPrevDocking = false; // is the previous window docked?
     long nStartPos = 0; // window position in the strip
-    unsigned iLastWin = vItems.size(); // index of last docking window in the strip
+    std::vector<Item>::size_type iLastWin = vItems.size(); // index of last docking window in the strip
 
-    for (unsigned i = 0; i != vItems.size(); ++i)
+    for (std::vector<Item>::size_type i = 0; i != vItems.size(); ++i)
     {
         // window
         DockingWindow& rWin = *vItems[i].pWin;