diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-03 16:19:55 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-04 08:35:13 +0200 |
| commit | 2ddddc46fdc3cf18cefcca29934eaab0544d2198 (patch) | |
| tree | e79a7be6c614ed564ab7ebd13f948d28f738f015 /package/qa | |
| parent | solenv/bin/pack_images.py no longer needs executable bit (diff) | |
| download | core-2ddddc46fdc3cf18cefcca29934eaab0544d2198.tar.gz core-2ddddc46fdc3cf18cefcca29934eaab0544d2198.zip | |
Make ThreadPool::pushTask take param by std::unique_ptr
And fix leak in XclExpRowBuffer::Finalize, was not freeing the
synchronous task it creates
Change-Id: Id1e9ddb5d968e6b95d9d2b5ca0c9e50774580182
Reviewed-on: https://gerrit.libreoffice.org/56874
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'package/qa')
| -rw-r--r-- | package/qa/cppunit/test_package.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/package/qa/cppunit/test_package.cxx b/package/qa/cppunit/test_package.cxx index 04e6b0643e96..0c49e55dca11 100644 --- a/package/qa/cppunit/test_package.cxx +++ b/package/qa/cppunit/test_package.cxx @@ -13,6 +13,7 @@ #include <comphelper/threadpool.hxx> #include <com/sun/star/packages/zip/ZipFileAccess.hpp> #include <com/sun/star/beans/NamedValue.hpp> +#include <o3tl/make_unique.hxx> #include <iterator> @@ -159,7 +160,7 @@ namespace mxNA->getByName(aName) >>= xStrm; CPPUNIT_ASSERT(xStrm.is()); - aPool.pushTask(new Worker(pTag, xStrm, *itBuf)); + aPool.pushTask(o3tl::make_unique<Worker>(pTag, xStrm, *itBuf)); } aPool.waitUntilDone(pTag); |