diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2016-11-08 09:33:15 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2016-11-08 09:21:06 +0000 |
commit | dca61b588a2426cfe9a18e4a1074b380e42884d7 (patch) | |
tree | db4436c600c08086c0da7fb09c290705b33beb4d /package/source/zipapi | |
parent | 6aa8e86f3a80b7cbfb81500a6bb5cfeaca97791b (diff) |
loplugin:expandablemethods in lotuswordpro..package
Change-Id: Ic9827c998f4f78775fdf5c1eaf9d4749d4986102
Reviewed-on: https://gerrit.libreoffice.org/30682
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'package/source/zipapi')
-rw-r--r-- | package/source/zipapi/ZipOutputStream.cxx | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/package/source/zipapi/ZipOutputStream.cxx b/package/source/zipapi/ZipOutputStream.cxx index 38039636b56d..603a61423eb4 100644 --- a/package/source/zipapi/ZipOutputStream.cxx +++ b/package/source/zipapi/ZipOutputStream.cxx @@ -148,16 +148,6 @@ void ZipOutputStream::consumeFinishedScheduledThreadEntries() m_aEntries = aNonFinishedEntries; } -void ZipOutputStream::consumeAllScheduledThreadEntries() -{ - while(!m_aEntries.empty()) - { - ZipOutputEntry* pCandidate = m_aEntries.back(); - m_aEntries.pop_back(); - consumeScheduledThreadEntry(pCandidate); - } -} - void ZipOutputStream::reduceScheduledThreadsToGivenNumberOrLess(sal_Int32 nThreads) { while(static_cast< sal_Int32 >(m_aEntries.size()) > nThreads) @@ -181,7 +171,12 @@ void ZipOutputStream::finish() comphelper::ThreadPool::getSharedOptimalPool().waitUntilDone(mpThreadTaskTag); // consume all processed entries - consumeAllScheduledThreadEntries(); + while(!m_aEntries.empty()) + { + ZipOutputEntry* pCandidate = m_aEntries.back(); + m_aEntries.pop_back(); + consumeScheduledThreadEntry(pCandidate); + } sal_Int32 nOffset= static_cast < sal_Int32 > (m_aChucker.GetPosition()); for (ZipEntry* p : m_aZipList) |