diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-29 11:34:54 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-29 13:46:26 +0200 |
commit | 6f6f1680ec35b1616c2fc54e5ee8c3b85edf333b (patch) | |
tree | 5ec6cfdde2c0c848416155d1d8cab72a3254525e /lotuswordpro | |
parent | 8581d880f8aa8c2be15c875db291cebbb42841c7 (diff) |
loplugin:simplifybool in hwpfilter..lotuswordpro
Change-Id: Iedfd492c963eb89fe75fdd73cae630e7e1dae119
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95100
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'lotuswordpro')
-rw-r--r-- | lotuswordpro/source/filter/LotusWordProImportFilter.cxx | 4 | ||||
-rw-r--r-- | lotuswordpro/source/filter/lwpsdwfileloader.cxx | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/lotuswordpro/source/filter/LotusWordProImportFilter.cxx b/lotuswordpro/source/filter/LotusWordProImportFilter.cxx index 16650de3369a..2c4f92c3e706 100644 --- a/lotuswordpro/source/filter/LotusWordProImportFilter.cxx +++ b/lotuswordpro/source/filter/LotusWordProImportFilter.cxx @@ -127,8 +127,8 @@ OUString SAL_CALL LotusWordProImportFilter::detect( css::uno::Sequence< Property Sequence< ::sal_Int8 > aData; sal_Int32 nLen = SAL_N_ELEMENTS( header ); - if ( !( ( nLen == xInputStream->readBytes( aData, nLen ) ) - && ( memcmp( static_cast<void const *>(header), static_cast<void const *>(aData.getConstArray()), nLen ) == 0 ) ) ) + if ( ( nLen != xInputStream->readBytes( aData, nLen ) ) + || ( memcmp( static_cast<void const *>(header), static_cast<void const *>(aData.getConstArray()), nLen ) != 0 ) ) sTypeName.clear(); return sTypeName; diff --git a/lotuswordpro/source/filter/lwpsdwfileloader.cxx b/lotuswordpro/source/filter/lwpsdwfileloader.cxx index 0c19b3569a5d..68dc4d39a3c6 100644 --- a/lotuswordpro/source/filter/lwpsdwfileloader.cxx +++ b/lotuswordpro/source/filter/lwpsdwfileloader.cxx @@ -81,7 +81,7 @@ void LwpSdwFileLoader::CreateDrawObjects(std::vector< rtl::Reference<XFFrame> >* unsigned char BinSignature[2]; m_pStream->ReadBytes(BinSignature, 2); - if (!(BinSignature[0] == 'S' && BinSignature[1] == 'M')) + if (BinSignature[0] != 'S' || BinSignature[1] != 'M') return; unsigned short nVersion; |