loplugin:unusedmethods unused return value in rsc

Change-Id: I6301aed80083c21e26851817220de6e135eb4da7

1 files changed, 7 insertions, 11 deletions

diff --git a/salhelper/source/timer.cxx b/salhelper/source/timer.cxx
index ee42e583441d..c62ed062a14b 100644
--- a/salhelper/source/timer.cxx
+++ b/salhelper/source/timer.cxx
@@ -39,10 +39,10 @@ public:
     virtual ~TimerManager();
 
     /// register timer
-    bool SAL_CALL registerTimer(salhelper::Timer* pTimer);
+    void SAL_CALL registerTimer(salhelper::Timer* pTimer);
 
     /// unregister timer
-    bool SAL_CALL unregisterTimer(salhelper::Timer* pTimer);
+    void SAL_CALL unregisterTimer(salhelper::Timer* pTimer);
 
     /// lookup timer
     bool SAL_CALL lookupTimer(const salhelper::Timer* pTimer);
@@ -283,13 +283,13 @@ TimerManager* TimerManager::getTimerManager()
     return m_pManager;
 }
 
-bool TimerManager::registerTimer(Timer* pTimer)
+void TimerManager::registerTimer(Timer* pTimer)
 {
     OSL_ASSERT(pTimer);
 
     if ( pTimer == nullptr )
     {
-        return false;
+        return;
     }
 
     osl::MutexGuard Guard(m_Lock);
@@ -320,17 +320,15 @@ bool TimerManager::registerTimer(Timer* pTimer)
         // signal it to TimerManager Thread
         m_notEmpty.set();
     }
-
-    return true;
 }
 
-bool TimerManager::unregisterTimer(Timer* pTimer)
+void TimerManager::unregisterTimer(Timer* pTimer)
 {
     OSL_ASSERT(pTimer);
 
     if ( pTimer == nullptr )
     {
-        return false;
+        return;
     }
 
     // lock access
@@ -344,12 +342,10 @@ bool TimerManager::unregisterTimer(Timer* pTimer)
         {
             // remove timer from list
             *ppIter = (*ppIter)->m_pNext;
-            return true;
+            return;
         }
         ppIter= &((*ppIter)->m_pNext);
     }
-
-    return false;
 }
 
 bool TimerManager::lookupTimer(const Timer* pTimer)