diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-04 14:20:26 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-05 07:32:46 +0100 |
| commit | 9a06b99d2f53bd8d0a9ab0936efed9924a2abb88 (patch) | |
| tree | 544f3e51a3978bd234a1c9fcdbf12d9b84352da4 /unotools | |
| parent | eaf89e477af94bd3977aca17d72dd442c7604e63 (diff) | |
loplugin:salcall fix non-virtual methods
first, since those are safer to change than virtual methods
Change-Id: Ie3b624019d75ee2b793cee33b3c5f64e994e8bfe
Reviewed-on: https://gerrit.libreoffice.org/45798
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'unotools')
| -rw-r--r-- | unotools/source/config/eventcfg.cxx | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/unotools/source/config/eventcfg.cxx b/unotools/source/config/eventcfg.cxx index 12b99a2db9cc..6f26ea335dcf 100644 --- a/unotools/source/config/eventcfg.cxx +++ b/unotools/source/config/eventcfg.cxx @@ -101,19 +101,19 @@ public: /// @throws css::container::NoSuchElementException /// @throws css::lang::WrappedTargetException /// @throws css::uno::RuntimeException - void SAL_CALL replaceByName( const OUString& aName, const css::uno::Any& aElement ); + void replaceByName( const OUString& aName, const css::uno::Any& aElement ); /// @throws css::container::NoSuchElementException /// @throws css::lang::WrappedTargetException /// @throws css::uno::RuntimeException - css::uno::Any SAL_CALL getByName( const OUString& aName ); + css::uno::Any getByName( const OUString& aName ); /// @throws css::uno::RuntimeException - css::uno::Sequence< OUString > SAL_CALL getElementNames( ); + css::uno::Sequence< OUString > getElementNames( ); /// @throws css::uno::RuntimeException - bool SAL_CALL hasByName( const OUString& aName ); + bool hasByName( const OUString& aName ); /// @throws css::uno::RuntimeException static css::uno::Type SAL_CALL getElementType( ); /// @throws css::uno::RuntimeException - bool SAL_CALL hasElements() const; + bool hasElements() const; OUString const & GetEventName( GlobalEventId nID ) const; }; @@ -238,7 +238,7 @@ void GlobalEventConfig_Impl::initBindingInfo() } } -void SAL_CALL GlobalEventConfig_Impl::replaceByName( const OUString& aName, const Any& aElement ) +void GlobalEventConfig_Impl::replaceByName( const OUString& aName, const Any& aElement ) { Sequence< beans::PropertyValue > props; //DF should we prepopulate the hash with a list of valid event Names? @@ -258,7 +258,7 @@ void SAL_CALL GlobalEventConfig_Impl::replaceByName( const OUString& aName, cons SetModified(); } -Any SAL_CALL GlobalEventConfig_Impl::getByName( const OUString& aName ) +Any GlobalEventConfig_Impl::getByName( const OUString& aName ) { Any aRet; Sequence< beans::PropertyValue > props(2); @@ -284,12 +284,12 @@ Any SAL_CALL GlobalEventConfig_Impl::getByName( const OUString& aName ) return aRet; } -Sequence< OUString > SAL_CALL GlobalEventConfig_Impl::getElementNames( ) +Sequence< OUString > GlobalEventConfig_Impl::getElementNames( ) { return uno::Sequence< OUString >(m_supportedEvents.data(), SupportedEventsVector::size()); } -bool SAL_CALL GlobalEventConfig_Impl::hasByName( const OUString& aName ) +bool GlobalEventConfig_Impl::hasByName( const OUString& aName ) { if ( m_eventBindingHash.find( aName ) != m_eventBindingHash.end() ) return true; @@ -306,7 +306,7 @@ Type SAL_CALL GlobalEventConfig_Impl::getElementType( ) return cppu::UnoType<Sequence<beans::PropertyValue>>::get(); } -bool SAL_CALL GlobalEventConfig_Impl::hasElements() const +bool GlobalEventConfig_Impl::hasElements() const { return m_eventBindingHash.empty(); } |