diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-04-24 09:23:15 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-04-24 10:05:54 +0200 |
| commit | a975225678c00272fc6e2ee2c85e6fe00a2204f1 (patch) | |
| tree | 47aa7fdcd49221fd3084b9d77418b6690b7bc0e0 /lotuswordpro/source/filter/lwplayout.cxx | |
| parent | 78a3a304871eb3eb861a49ed00345b54fba01114 (diff) | |
clang-tidy readability-simplify-boolean-expr in hwpfilter..lotuswordpro
Change-Id: I945d3fe6af5f88937b341dfc3696bf1d36344862
Reviewed-on: https://gerrit.libreoffice.org/36874
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'lotuswordpro/source/filter/lwplayout.cxx')
| -rw-r--r-- | lotuswordpro/source/filter/lwplayout.cxx | 30 |
1 files changed, 6 insertions, 24 deletions
diff --git a/lotuswordpro/source/filter/lwplayout.cxx b/lotuswordpro/source/filter/lwplayout.cxx index b5f5056ebf50..ca90224f7e69 100644 --- a/lotuswordpro/source/filter/lwplayout.cxx +++ b/lotuswordpro/source/filter/lwplayout.cxx @@ -974,12 +974,8 @@ bool LwpMiddleLayout::CanSizeRight() { sal_uInt8 RelType = GetRelativeType(); - if (RelType == LwpLayoutRelativityGuts::LAY_INLINE || RelType == LwpLayoutRelativityGuts::LAY_PARA_RELATIVE - || RelType == LwpLayoutRelativityGuts::LAY_INLINE_VERTICAL) - { - return false; - } - return true; + return !(RelType == LwpLayoutRelativityGuts::LAY_INLINE || RelType == LwpLayoutRelativityGuts::LAY_PARA_RELATIVE + || RelType == LwpLayoutRelativityGuts::LAY_INLINE_VERTICAL); } sal_Int32 LwpMiddleLayout::GetMinimumWidth() { @@ -1453,9 +1449,7 @@ bool LwpMiddleLayout::GetUsePrinterSettings() bool LwpMiddleLayout::HasContent() { rtl::Reference<LwpObject> content = m_Content.obj(); - if(content.is()) - return true; - return false; + return content.is(); } LwpLayout::LwpLayout( LwpObjectHeader &objHdr, LwpSvStream* pStrm ) : @@ -2084,11 +2078,7 @@ bool LwpPlacableLayout::IsAnchorPage() return false; rtl::Reference<LwpVirtualLayout> xLayout = GetParentLayout(); - if (xLayout.is() && (xLayout->IsPage() || xLayout->IsHeader() || xLayout->IsFooter())) - { - return true; - } - return false; + return xLayout.is() && (xLayout->IsPage() || xLayout->IsHeader() || xLayout->IsFooter()); } /** * @descr: whether the parent layout is frame layout @@ -2100,11 +2090,7 @@ bool LwpPlacableLayout::IsAnchorFrame() return false; rtl::Reference<LwpVirtualLayout> xLayout = GetParentLayout(); - if (xLayout.is() && (xLayout->IsFrame() || xLayout->IsGroupHead())) - { - return true; - } - return false; + return xLayout.is() && (xLayout->IsFrame() || xLayout->IsGroupHead()); } /** * @descr: whether the parent layout is cell layout @@ -2116,11 +2102,7 @@ bool LwpPlacableLayout::IsAnchorCell() return false; rtl::Reference<LwpVirtualLayout> xLayout = GetParentLayout(); - if (xLayout.is() && xLayout->IsCell()) - { - return true; - } - return false; + return xLayout.is() && xLayout->IsCell(); } /** |