diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-11-06 13:27:23 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-11-07 07:56:39 +0100 |
| commit | 2e2d736ccbb21daae91603732c87d5375ede1b47 (patch) | |
| tree | 6621e351d38dff77dc16c6413767f4baa3c76326 /package | |
| parent | 1647bc1309dcd313dff2614f4844b71b1c558dee (diff) | |
loplugin:collapseif in framework..sal
Change-Id: I3068b18f5cff024a48a8f8c68d69cadad30fe4d5
Reviewed-on: https://gerrit.libreoffice.org/62953
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'package')
| -rw-r--r-- | package/source/zipapi/ZipFile.cxx | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/package/source/zipapi/ZipFile.cxx b/package/source/zipapi/ZipFile.cxx index 2e2a6216cb4f..60bdced74b5a 100644 --- a/package/source/zipapi/ZipFile.cxx +++ b/package/source/zipapi/ZipFile.cxx @@ -83,13 +83,10 @@ ZipFile::ZipFile( const rtl::Reference<comphelper::RefCountedMutex>& aMutexHolde , m_xContext ( rxContext ) , bRecoveryMode( false ) { - if (bInitialise) + if (bInitialise && readCEN() == -1 ) { - if ( readCEN() == -1 ) - { - aEntries.clear(); - throw ZipException( "stream data looks to be broken" ); - } + aEntries.clear(); + throw ZipException( "stream data looks to be broken" ); } } |