Let's use the iterator return by erase method (final)

Change-Id: I1934b0c2a9ae9fa089d7e9ded4a289908fb2709a
Reviewed-on: https://gerrit.libreoffice.org/23173Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>

cppuhelper/source/servicemanager.cxx

@@ -1776,7 +1776,7 @@ void cppuhelper::ServiceManager::removeRdbFiles(
                     removeFromImplementationMap(
                         &data_.singletons, j->second->info->singletons,
                         j->second);
-                    data_.namedImplementations.erase(j++);
+                    j = data_.namedImplementations.erase(j);
                 } else {
                     ++j;
                 }

dbaccess/source/ui/browser/genericcontroller.cxx

@@ -736,7 +736,7 @@ void OGenericUnoController::removeStatusListener(const Reference< XStatusListene
                 )
             )
         {
-            m_arrStatusListener.erase( iterSearch++ );
+            iterSearch = m_arrStatusListener.erase(iterSearch);
             if (!bRemoveForAll)
                 // remove the listener only for the given URL, so we can exit the loop after deletion
                 break;

sc/source/core/data/conditio.cxx

@@ -2269,7 +2269,7 @@ bool ScConditionalFormatList::CheckAllEntries()
         if ((*itr)->GetRange().empty())
         {
             bValid = false;
-            m_ConditionalFormats.erase(itr++);
+            itr = m_ConditionalFormats.erase(itr);
         }
         else
             ++itr;