diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-08-11 13:54:31 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-08-13 08:34:49 +0200 |
commit | 7a5d2b3cdb448998e6aef1f12f0a70c400a434d2 (patch) | |
tree | 924cfa383240c506613f6b64282bf73632f10b99 | |
parent | 412215fbc5a71afff7ee3b539852ed92c55e11bc (diff) |
loplugin:returnconstant in unoxml,uui
Change-Id: Iac240f59acf8404fe5439a7e450c048df0d7a54e
Reviewed-on: https://gerrit.libreoffice.org/58879
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
-rw-r--r-- | unoxml/inc/eventdispatcher.hxx | 2 | ||||
-rw-r--r-- | unoxml/source/events/eventdispatcher.cxx | 11 | ||||
-rw-r--r-- | uui/source/iahndl-authentication.cxx | 4 | ||||
-rw-r--r-- | uui/source/iahndl.hxx | 2 |
4 files changed, 8 insertions, 11 deletions
diff --git a/unoxml/inc/eventdispatcher.hxx b/unoxml/inc/eventdispatcher.hxx index ee8bec567590..be6a7206e7cb 100644 --- a/unoxml/inc/eventdispatcher.hxx +++ b/unoxml/inc/eventdispatcher.hxx @@ -69,7 +69,7 @@ public: const OUString& aType, const css::uno::Reference< css::xml::dom::events::XEvent >& xEvent); - bool dispatchEvent( + void dispatchEvent( DOM::CDocument & rDocument, ::osl::Mutex & rMutex, xmlNodePtr const pNode, diff --git a/unoxml/source/events/eventdispatcher.cxx b/unoxml/source/events/eventdispatcher.cxx index 85f6df5b9c14..ff5ce7aa8889 100644 --- a/unoxml/source/events/eventdispatcher.cxx +++ b/unoxml/source/events/eventdispatcher.cxx @@ -102,7 +102,7 @@ namespace DOM { namespace events { } } - bool CEventDispatcher::dispatchEvent( + void CEventDispatcher::dispatchEvent( DOM::CDocument & rDocument, ::osl::Mutex & rMutex, xmlNodePtr const pNode, Reference<XNode> const& xNode, Reference< XEvent > const& i_xEvent) const @@ -117,7 +117,7 @@ namespace DOM { namespace events { } if (captureListeners.empty() && targetListeners.empty()) - return true; + return; CEvent *pEvent = nullptr; // pointer to internal event representation @@ -222,7 +222,7 @@ namespace DOM { namespace events { { pEvent->m_currentTarget = rinode->first; callListeners(captureListeners, rinode->second, aType, xEvent); - if (pEvent->m_canceled) return true; + if (pEvent->m_canceled) return; ++rinode; } @@ -232,7 +232,7 @@ namespace DOM { namespace events { pEvent->m_phase = PhaseType_AT_TARGET; pEvent->m_currentTarget = inode->first; callListeners(targetListeners, inode->second, aType, xEvent); - if (pEvent->m_canceled) return true; + if (pEvent->m_canceled) return; // bubbeling phase ++inode; if (i_xEvent->getBubbles()) { @@ -242,12 +242,11 @@ namespace DOM { namespace events { pEvent->m_currentTarget = inode->first; callListeners(targetListeners, inode->second, aType, xEvent); - if (pEvent->m_canceled) return true; + if (pEvent->m_canceled) return; ++inode; } } } - return true; } }} diff --git a/uui/source/iahndl-authentication.cxx b/uui/source/iahndl-authentication.cxx index 5aedeba76a04..d151a36138e7 100644 --- a/uui/source/iahndl-authentication.cxx +++ b/uui/source/iahndl-authentication.cxx @@ -724,7 +724,7 @@ UUIInteractionHelper::handlePasswordRequest( return false; } -bool +void UUIInteractionHelper::handleAuthFallbackRequest( OUString & instructions, OUString & url, uno::Sequence< uno::Reference< task::XInteractionContinuation > > const & rContinuations ) @@ -741,8 +741,6 @@ UUIInteractionHelper::handleAuthFallbackRequest( OUString & instructions, xAuthFallback->setCode(dlg.GetCode()); xAuthFallback->select( ); } - - return true; } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/uui/source/iahndl.hxx b/uui/source/iahndl.hxx index 056b66103ad3..9bb7fda8909b 100644 --- a/uui/source/iahndl.hxx +++ b/uui/source/iahndl.hxx @@ -235,7 +235,7 @@ private: const OUString& i_rServiceName ) const; - bool + void handleAuthFallbackRequest( OUString & instructions, OUString & url, |