loplugin:constantparam in svl

Change-Id: Iefc441262cbdc6f115ea4ca5a673456b59477e13

1 files changed, 10 insertions, 15 deletions

diff --git a/svl/source/undo/undo.cxx b/svl/source/undo/undo.cxx
index 42e96af70527..858c02fde6de 100644
--- a/svl/source/undo/undo.cxx
+++ b/svl/source/undo/undo.cxx
@@ -1254,26 +1254,21 @@ bool SfxUndoManager::HasTopUndoActionMark( UndoStackMark const i_mark )
 }
 
 
-void SfxUndoManager::RemoveOldestUndoActions( size_t const i_count )
+void SfxUndoManager::RemoveOldestUndoAction()
 {
     UndoManagerGuard aGuard( *m_xData );
 
-    size_t nActionsToRemove = i_count;
-    while ( nActionsToRemove )
-    {
-        SfxUndoAction* pActionToRemove = m_xData->pUndoArray->aUndoActions[0].pAction;
-
-        if ( IsInListAction() && ( m_xData->pUndoArray->nCurUndoAction == 1 ) )
-        {
-            assert(!"SfxUndoManager::RemoveOldestUndoActions: cannot remove a not-yet-closed list action!");
-            return;
-        }
+    SfxUndoAction* pActionToRemove = m_xData->pUndoArray->aUndoActions[0].pAction;
 
-        aGuard.markForDeletion( pActionToRemove );
-        m_xData->pUndoArray->aUndoActions.Remove( 0 );
-        --m_xData->pUndoArray->nCurUndoAction;
-        --nActionsToRemove;
+    if ( IsInListAction() && ( m_xData->pUndoArray->nCurUndoAction == 1 ) )
+    {
+        assert(!"SfxUndoManager::RemoveOldestUndoActions: cannot remove a not-yet-closed list action!");
+        return;
     }
+
+    aGuard.markForDeletion( pActionToRemove );
+    m_xData->pUndoArray->aUndoActions.Remove( 0 );
+    --m_xData->pUndoArray->nCurUndoAction;
 }
 
 void SfxUndoManager::dumpAsXml(xmlTextWriterPtr pWriter) const