diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-11-22 09:33:32 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-11-23 06:59:31 +0100 |
| commit | 7a1c21e53fc4733a4bb52282ce0098fcc085ab0e (patch) | |
| tree | c1b90f74e1ce9e9f3a852f398890899459189cab /store/source/storpage.cxx | |
| parent | c24c32bf71b8e64bd0d36e511f554e1f6c015842 (diff) | |
loplugin:simplifybool for negation of comparison operator
Change-Id: Ie56daf560185274754afbc7a09c432b5c2793791
Reviewed-on: https://gerrit.libreoffice.org/45068
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'store/source/storpage.cxx')
| -rw-r--r-- | store/source/storpage.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/store/source/storpage.cxx b/store/source/storpage.cxx index a4b573219a95..d441121e0ac4 100644 --- a/store/source/storpage.cxx +++ b/store/source/storpage.cxx @@ -160,7 +160,7 @@ storeError OStorePageManager::remove_Impl (entry & rEntry) // Check current page index. PageHolderObject< page > xPage (aNode.get()); sal_uInt16 i = xPage->find (rEntry), n = xPage->usageCount(); - if (!(i < n)) + if (i >= n) { // Path to entry not exists (Must not happen(?)). return store_E_NotExists; @@ -191,7 +191,7 @@ storeError OStorePageManager::remove_Impl (entry & rEntry) // Check index. i = xPage->find (rEntry); n = xPage->usageCount(); - if (!(i < n)) + if (i >= n) { // Path to entry not exists (Must not happen(?)). return store_E_NotExists; @@ -229,7 +229,7 @@ storeError OStorePageManager::namei ( return store_E_InvalidParameter; // Check name length. - if (!(pName->length < STORE_MAXIMUM_NAMESIZE)) + if (pName->length >= STORE_MAXIMUM_NAMESIZE) return store_E_NameTooLong; // Transform pathname into key. |