diff options
| author | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2016-04-11 09:09:44 +0200 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2016-04-11 08:19:20 +0000 |
| commit | b15ee18a5e81b7c3908fb5343930638cac4918d9 (patch) | |
| tree | 3c48958dc1adc5ab7a57177670b2afa772e3b83e /sot | |
| parent | d91f38bcff7014564c71cb8782fee28b3090caab (diff) | |
cppcheck: silence warnings assertWithSideEffect
Mark some class methods as pure.
warnings look like this:
> dbaccess/source/core/api/RowSetBase.cxx
> 593 assertWithSideEffect 398 warning Assert statement calls
> a function which may have desired side effects: 'isAfterLast'.
Change-Id: I1b69340c3714be4678b599fa9a8ca933122aa857
Reviewed-on: https://gerrit.libreoffice.org/23981
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'sot')
| -rw-r--r-- | sot/source/sdstor/stgcache.hxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sot/source/sdstor/stgcache.hxx b/sot/source/sdstor/stgcache.hxx index 3c0d866524a0..d0ac5c89494e 100644 --- a/sot/source/sdstor/stgcache.hxx +++ b/sot/source/sdstor/stgcache.hxx @@ -66,13 +66,13 @@ public: void IncRef() { m_nRef++; } sal_uInt16 DecRef() { return --m_nRef; } void SetPhysPageSize( short ); - sal_Int32 GetPhysPages() { return m_nPages; } - short GetPhysPageSize() { return m_nPageSize; } + sal_Int32 GetPhysPages() const { return m_nPages; } + short GetPhysPageSize() const { return m_nPageSize; } SvStream* GetStrm() { return m_pStrm; } void SetStrm( SvStream*, bool ); void SetStrm( UCBStorageStream* ); - bool IsWritable() { return ( m_pStrm && m_pStrm->IsWritable() ); } - bool Good() { return m_nError == SVSTREAM_OK; } + bool IsWritable() const { return ( m_pStrm && m_pStrm->IsWritable() ); } + bool Good() const { return m_nError == SVSTREAM_OK; } ErrCode GetError() { return m_nError; } void MoveError( StorageBase& ); void SetError( ErrCode ); |