diff options
| author | Noel Grandin <noel@peralex.com> | 2014-05-02 15:42:25 +0200 |
|---|---|---|
| committer | Noel Grandin <noel@peralex.com> | 2014-05-05 12:47:48 +0200 |
| commit | 4f9b21248ffdf55cef9f3f9d1da76778ee000775 (patch) | |
| tree | b059d288339a68aa4c3901f1100e99b04d05a23d /store | |
| parent | b4107411900f5bb4c215e2d9db09fc2b2b82939b (diff) | |
simplify ternary conditions "xxx ? yyy : false"
Look for code like:
xxx ? yyy : false;
Which can be simplified to:
xxx && yyy
Change-Id: Ia33c0e452aa28af3f0658a5382895aaad0246b4d
Diffstat (limited to 'store')
| -rw-r--r-- | store/source/storbase.hxx | 2 | ||||
| -rw-r--r-- | store/workben/t_page.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/store/source/storbase.hxx b/store/source/storbase.hxx index 30d32605b2b1..1150a6bca570 100644 --- a/store/source/storbase.hxx +++ b/store/source/storbase.hxx @@ -156,7 +156,7 @@ public: bool operator== (long count) const { - return (m_pCount != 0) ? *m_pCount == count : false; + return (m_pCount != 0) && (*m_pCount == count); } }; diff --git a/store/workben/t_page.cxx b/store/workben/t_page.cxx index 54a32e15f2b1..2bf4ebbd53aa 100644 --- a/store/workben/t_page.cxx +++ b/store/workben/t_page.cxx @@ -78,7 +78,7 @@ public: bool operator== (long count) const { - return (m_pCount != 0) ? *m_pCount == count : false; + return (m_pCount != 0) && (*m_pCount == count); } friend void swap<> (SharedCount & lhs, SharedCount & rhs); // nothrow |