diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-06-03 10:52:46 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-06-04 08:23:43 +0200 |
| commit | b83ac35bf46ecbebf7f806235eca38a71e521c32 (patch) | |
| tree | 660cb00ba4045c7238e9d43176196339ecf901f0 /store | |
| parent | 7b905c8f1d520ae1aaf4bd2fbb6a3325b404ea95 (diff) | |
loplugin:simplifypointertobool improve (2)
to look for the
x.get() == null
pattern, which can be simplified to
!x
Change-Id: I0eddf93257ab53ab31949961d7c33ac2dd7288ea
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95400
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'store')
| -rw-r--r-- | store/source/storbase.hxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/store/source/storbase.hxx b/store/source/storbase.hxx index 7829ae1c4a79..f4745d7c6885 100644 --- a/store/source/storbase.hxx +++ b/store/source/storbase.hxx @@ -418,7 +418,7 @@ class PageHolderObject public: bool construct (rtl::Reference< PageData::Allocator > const & rxAllocator) { - if ((m_xPage.get() == nullptr) && rxAllocator.is()) + if (!m_xPage && rxAllocator.is()) { std::shared_ptr<PageData> tmp (rxAllocator->construct<T>(), PageData::Deallocate(rxAllocator)); m_xPage.swap (tmp); |