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 /cppu | |
| parent | loplugin:countusersofdefaultparams in editeng..package (diff) | |
| download | core-9a06b99d2f53bd8d0a9ab0936efed9924a2abb88.tar.gz core-9a06b99d2f53bd8d0a9ab0936efed9924a2abb88.zip | |
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 'cppu')
| -rw-r--r-- | cppu/source/uno/lbmap.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/cppu/source/uno/lbmap.cxx b/cppu/source/uno/lbmap.cxx index 4bdf0a6d6d8e..74e167eb336c 100644 --- a/cppu/source/uno/lbmap.cxx +++ b/cppu/source/uno/lbmap.cxx @@ -64,8 +64,8 @@ public: Mapping(Mapping && other): _pMapping(other._pMapping) { other._pMapping = nullptr; } inline ~Mapping(); - inline Mapping & SAL_CALL operator = ( uno_Mapping * pMapping ); - Mapping & SAL_CALL operator = ( const Mapping & rMapping ) + inline Mapping & operator = ( uno_Mapping * pMapping ); + Mapping & operator = ( const Mapping & rMapping ) { return operator = ( rMapping._pMapping ); } Mapping & operator =(Mapping && other) { if (_pMapping != nullptr) { @@ -75,9 +75,9 @@ public: other._pMapping = nullptr; return *this; } - uno_Mapping * SAL_CALL get() const + uno_Mapping * get() const { return _pMapping; } - bool SAL_CALL is() const + bool is() const { return (_pMapping != nullptr); } }; |