summaryrefslogtreecommitdiff
path: root/ucb
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2018-10-18 08:27:05 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2018-10-18 09:46:28 +0200
commit5fb66ae5595b7435e8954df31473fad15a74b8c2 (patch)
tree3b3f0ce3eafa10557a7e78b10851c97ee16c7ebf /ucb
parent181a1b36ac728e3a43e054496ceb53fd3315abdb (diff)
clang-tidy readability-simplify-boolean-expr
Change-Id: I78fa01a6c803dec782488490b730af3a11814d64 Reviewed-on: https://gerrit.libreoffice.org/61902 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'ucb')
-rw-r--r--ucb/source/ucp/cmis/cmis_datasupplier.cxx5
-rw-r--r--ucb/source/ucp/ext/ucpext_datasupplier.cxx7
2 files changed, 3 insertions, 9 deletions
diff --git a/ucb/source/ucp/cmis/cmis_datasupplier.cxx b/ucb/source/ucp/cmis/cmis_datasupplier.cxx
index 42c5c10157d2..83baa3ba37aa 100644
--- a/ucb/source/ucp/cmis/cmis_datasupplier.cxx
+++ b/ucb/source/ucp/cmis/cmis_datasupplier.cxx
@@ -83,10 +83,7 @@ namespace cmis
return true;
getData();
- if ( maResults.size() > nIndex )
- return true;
-
- return false;
+ return maResults.size() > nIndex;
}
sal_uInt32 DataSupplier::totalCount()
diff --git a/ucb/source/ucp/ext/ucpext_datasupplier.cxx b/ucb/source/ucp/ext/ucpext_datasupplier.cxx
index cc8ac22b9f04..9d6f962119c0 100644
--- a/ucb/source/ucp/ext/ucpext_datasupplier.cxx
+++ b/ucb/source/ucp/ext/ucpext_datasupplier.cxx
@@ -253,11 +253,8 @@ namespace ucb { namespace ucp { namespace ext
{
::osl::ClearableGuard< ::osl::Mutex > aGuard( m_pImpl->m_aMutex );
- if ( m_pImpl->m_aResults.size() > i_nIndex )
- // result already present.
- return true;
-
- return false;
+ // true if result already present.
+ return m_pImpl->m_aResults.size() > i_nIndex;
}