summaryrefslogtreecommitdiff
path: root/xmloff
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2020-08-06 13:32:43 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2020-08-14 18:03:06 +0200
commite67657d5211f6e95ddf8bd621108608573b00d5d (patch)
tree66724101dbd95721714bd40fcb4861663432774c /xmloff
parent186def8f48e273c3a3b4d23b3ab2efd0d8664731 (diff)
loplugin:simplifybool more
look for expressions like !(a && !b) which can be expanded out Change-Id: I72515a9638762b050f9a258c08da39ebfa2ef8e7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100579 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'xmloff')
-rw-r--r--xmloff/source/draw/ximppage.cxx2
-rw-r--r--xmloff/source/text/XMLTextFrameContext.cxx2
2 files changed, 2 insertions, 2 deletions
diff --git a/xmloff/source/draw/ximppage.cxx b/xmloff/source/draw/ximppage.cxx
index cd4ae5ec892e..dacf059a5182 100644
--- a/xmloff/source/draw/ximppage.cxx
+++ b/xmloff/source/draw/ximppage.cxx
@@ -448,7 +448,7 @@ void SdXMLGenericPageContext::SetStyle( OUString const & rStyleName )
void SdXMLGenericPageContext::SetLayout()
{
// set PresentationPageLayout?
- if(!(GetSdImport().IsImpress() && !maPageLayoutName.isEmpty()))
+ if(!GetSdImport().IsImpress() || maPageLayoutName.isEmpty())
return;
sal_Int32 nType = -1;
diff --git a/xmloff/source/text/XMLTextFrameContext.cxx b/xmloff/source/text/XMLTextFrameContext.cxx
index f5c75bfc7d81..58988bae259c 100644
--- a/xmloff/source/text/XMLTextFrameContext.cxx
+++ b/xmloff/source/text/XMLTextFrameContext.cxx
@@ -1297,7 +1297,7 @@ void XMLTextFrameContext_Impl::SetHyperlink( const OUString& rHRef,
void XMLTextFrameContext_Impl::SetName()
{
Reference<XNamed> xNamed(xPropSet, UNO_QUERY);
- if (!(!m_sOrigName.isEmpty() && xNamed.is()))
+ if (m_sOrigName.isEmpty() || !xNamed.is())
return;
OUString const name(xNamed->getName());