diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-19 13:23:27 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-20 07:41:29 +0100 |
| commit | e725836ec19858be83bf6f7f251dd52dbcdf31d6 (patch) | |
| tree | ec8aafe6e546a35dff6c3349e9644a6d6b94eea9 /tools | |
| parent | 73139fe600fc1399ae828077981a2498cb0a0b0c (diff) | |
loplugin:unused-returns in filter,tools,xmloff
Change-Id: I7a57be7e241883adac4417baa699a97d6304a631
Reviewed-on: https://gerrit.libreoffice.org/48188
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/source/datetime/tdate.cxx | 6 | ||||
| -rw-r--r-- | tools/source/memtools/multisel.cxx | 11 |
2 files changed, 8 insertions, 9 deletions
diff --git a/tools/source/datetime/tdate.cxx b/tools/source/datetime/tdate.cxx index f6073b85451e..8b6076c912ce 100644 --- a/tools/source/datetime/tdate.cxx +++ b/tools/source/datetime/tdate.cxx @@ -478,18 +478,16 @@ bool Date::IsValidDate( sal_uInt16 nDay, sal_uInt16 nMonth, sal_Int16 nYear ) return true; } -bool Date::Normalize() +void Date::Normalize() { sal_uInt16 nDay = GetDay(); sal_uInt16 nMonth = GetMonth(); sal_Int16 nYear = GetYear(); if (!Normalize( nDay, nMonth, nYear)) - return false; + return; setDateFromDMY( nDay, nMonth, nYear ); - - return true; } //static diff --git a/tools/source/memtools/multisel.cxx b/tools/source/memtools/multisel.cxx index cfb4137039ca..1928a6913e5a 100644 --- a/tools/source/memtools/multisel.cxx +++ b/tools/source/memtools/multisel.cxx @@ -539,15 +539,18 @@ bool StringRangeEnumerator::insertRange( sal_Int32 i_nFirst, sal_Int32 i_nLast, return bSuccess; } -bool StringRangeEnumerator::insertJoinedRanges( +void StringRangeEnumerator::insertJoinedRanges( const std::vector< sal_Int32 >& rNumbers ) { size_t nCount = rNumbers.size(); if( nCount == 0 ) - return true; + return; if( nCount == 1 ) - return insertRange( rNumbers[0], -1, false ); + { + insertRange( rNumbers[0], -1, false ); + return; + } for( size_t i = 0; i < nCount - 1; i++ ) { @@ -561,8 +564,6 @@ bool StringRangeEnumerator::insertJoinedRanges( insertRange( nFirst, nLast, nFirst != nLast ); } - - return true; } bool StringRangeEnumerator::setRange( const OUString& i_rNewRange ) |