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 /sw/source/core/SwNumberTree | |
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 'sw/source/core/SwNumberTree')
-rw-r--r-- | sw/source/core/SwNumberTree/SwNodeNum.cxx | 2 | ||||
-rw-r--r-- | sw/source/core/SwNumberTree/SwNumberTree.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/sw/source/core/SwNumberTree/SwNodeNum.cxx b/sw/source/core/SwNumberTree/SwNodeNum.cxx index 09fae2081264..61c6410024c5 100644 --- a/sw/source/core/SwNumberTree/SwNodeNum.cxx +++ b/sw/source/core/SwNumberTree/SwNodeNum.cxx @@ -243,7 +243,7 @@ bool SwNodeNum::LessThan(const SwNumberTreeNode & rNode) const { // #i83479# - refactoring // simplify comparison by comparing the indexes of the text nodes - bResult = ( mpTxtNode->GetIndex() < rTmpNode.mpTxtNode->GetIndex() ) ? true : false; + bResult = ( mpTxtNode->GetIndex() < rTmpNode.mpTxtNode->GetIndex() ); } return bResult; diff --git a/sw/source/core/SwNumberTree/SwNumberTree.cxx b/sw/source/core/SwNumberTree/SwNumberTree.cxx index 60e5f99e9eb6..b35a53d8eb90 100644 --- a/sw/source/core/SwNumberTree/SwNumberTree.cxx +++ b/sw/source/core/SwNumberTree/SwNumberTree.cxx @@ -690,7 +690,7 @@ void SwNumberTreeNode::RemoveMe() bool SwNumberTreeNode::IsValid() const { - return mpParent ? mpParent->IsValid(this) : false; + return mpParent && mpParent->IsValid(this); } SwNumberTree::tSwNumTreeNumber SwNumberTreeNode::GetNumber(bool bValidate) |