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 /vbahelper/source/msforms | |
| 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 'vbahelper/source/msforms')
| -rw-r--r-- | vbahelper/source/msforms/vbalistcontrolhelper.cxx | 12 | ||||
| -rw-r--r-- | vbahelper/source/msforms/vbalistcontrolhelper.hxx | 12 |
2 files changed, 12 insertions, 12 deletions
diff --git a/vbahelper/source/msforms/vbalistcontrolhelper.cxx b/vbahelper/source/msforms/vbalistcontrolhelper.cxx index 462e39aae95e..308c0934ece5 100644 --- a/vbahelper/source/msforms/vbalistcontrolhelper.cxx +++ b/vbahelper/source/msforms/vbalistcontrolhelper.cxx @@ -80,7 +80,7 @@ uno::Any ListPropListener::getValueEvent() return aRet; } -void SAL_CALL +void ListControlHelper::AddItem( const uno::Any& pvargItem, const uno::Any& pvargIndex ) { if ( pvargItem.hasValue() ) @@ -135,7 +135,7 @@ ListControlHelper::AddItem( const uno::Any& pvargItem, const uno::Any& pvargInde } } -void SAL_CALL +void ListControlHelper::removeItem( const uno::Any& index ) { sal_Int32 nIndex = 0; @@ -164,7 +164,7 @@ ListControlHelper::removeItem( const uno::Any& index ) } } -void SAL_CALL +void ListControlHelper::Clear( ) { // urk, setValue doesn't seem to work !! @@ -172,14 +172,14 @@ ListControlHelper::Clear( ) m_xProps->setPropertyValue( "StringItemList", uno::makeAny( uno::Sequence< OUString >() ) ); } -void SAL_CALL +void ListControlHelper::setRowSource( const OUString& _rowsource ) { if ( _rowsource.isEmpty() ) Clear(); } -sal_Int32 SAL_CALL +sal_Int32 ListControlHelper::getListCount() { uno::Sequence< OUString > sList; @@ -187,7 +187,7 @@ ListControlHelper::getListCount() return sList.getLength(); } -uno::Any SAL_CALL +uno::Any ListControlHelper::List( const ::uno::Any& pvargIndex, const uno::Any& pvarColumn ) { return uno::makeAny( uno::Reference< XPropValue > ( new ScVbaPropValue( new ListPropListener( m_xProps, pvargIndex, pvarColumn ) ) ) ); diff --git a/vbahelper/source/msforms/vbalistcontrolhelper.hxx b/vbahelper/source/msforms/vbalistcontrolhelper.hxx index 334f7347603f..dff3e1c33541 100644 --- a/vbahelper/source/msforms/vbalistcontrolhelper.hxx +++ b/vbahelper/source/msforms/vbalistcontrolhelper.hxx @@ -29,17 +29,17 @@ class ListControlHelper final public: explicit ListControlHelper( const css::uno::Reference< css::beans::XPropertySet >& rxControl ) : m_xProps( rxControl ){} /// @throws css::uno::RuntimeException - void SAL_CALL AddItem( const css::uno::Any& pvargItem, const css::uno::Any& pvargIndex ); + void AddItem( const css::uno::Any& pvargItem, const css::uno::Any& pvargIndex ); /// @throws css::uno::RuntimeException - void SAL_CALL removeItem( const css::uno::Any& index ); + void removeItem( const css::uno::Any& index ); /// @throws css::uno::RuntimeException - void SAL_CALL setRowSource( const OUString& _rowsource ); + void setRowSource( const OUString& _rowsource ); /// @throws css::uno::RuntimeException - ::sal_Int32 SAL_CALL getListCount(); + ::sal_Int32 getListCount(); /// @throws css::uno::RuntimeException - css::uno::Any SAL_CALL List( const css::uno::Any& pvargIndex, const css::uno::Any& pvarColumn ); + css::uno::Any List( const css::uno::Any& pvargIndex, const css::uno::Any& pvarColumn ); /// @throws css::uno::RuntimeException - void SAL_CALL Clear( ); + void Clear( ); }; #endif |