From f19e32b101d56ceb7215a6a9d45d39a2cd264189 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Thu, 14 Jan 2016 16:31:56 +0200 Subject: loplugin:unusedmethods unused return value in rsc Change-Id: I6301aed80083c21e26851817220de6e135eb4da7 --- salhelper/source/timer.cxx | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) (limited to 'salhelper') 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) -- cgit v1.2.3