gpesquero/organicmaps
1
Fork 0
forked from organicmaps/organicmaps
Code Pull requests Activity

memory leak fix

Browse source 
correct deletion ThreadPool::Impl after manual call stop
This commit is contained in:
ExMix 2013-11-01 15:31:20 +03:00 • committed by Alex Zolotarev
parent 8e356b2e73
commit bf5b9df290
2 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
base/thread_pool.cpp
View file

@ -89,6 +89,7 @@ namespace threads
delete m_threads[i].first;
}
m_threads.clear();
m_tasks.ProcessList(bind(&ThreadPool::Impl::FinishTasksOnStop, this, _1));
m_tasks.Clear();
}
@ -115,6 +116,11 @@ namespace threads
ThreadPool::ThreadPool(size_t size, const finish_routine_fn & finishFn)
: m_impl(new Impl(size, finishFn)) {}
ThreadPool::~ThreadPool()
{
delete m_impl;
}
void ThreadPool::AddTask(threads::IRoutine * routine)
{
m_impl->PushBack(routine);

1
base/thread_pool.hpp
View file

@ -12,6 +12,7 @@ namespace threads
{
public:
ThreadPool(size_t size, const finish_routine_fn & finishFn);
~ThreadPool();
// ThreadPool will not delete routine. You can delete it in finish_routine_fn if need
void AddTask(threads::IRoutine * routine);