Skip to content

various improvements in performance, safety, features and readability #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
98 changes: 0 additions & 98 deletions ThreadPool.h

This file was deleted.

118 changes: 118 additions & 0 deletions ThreadPool.hpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,118 @@
/*
Copyright (c) 2012 Jakob Progsch, Václav Zeman

This software is provided 'as-is', without any express or implied
warranty. In no event will the authors be held liable for any damages
arising from the use of this software.

Permission is granted to anyone to use this software for any purpose,
including commercial applications, and to alter it and redistribute it
freely, subject to the following restrictions:

1. The origin of this software must not be misrepresented; you must not
claim that you wrote the original software. If you use this software
in a product, an acknowledgment in the product documentation would be
appreciated but is not required.

2. Altered source versions must be plainly marked as such, and must not be
misrepresented as being the original software.

3. This notice may not be removed or altered from any source
distribution.
*/

#ifndef THREAD_POOL_HPP
#define THREAD_POOL_HPP

// containers
#include <vector>
#include <queue>
// threading
#include <thread>
#include <mutex>
#include <condition_variable>
#include <atomic>
#include <future>
// utility wrappers
#include <memory>
#include <functional>
// exceptions
#include <stdexcept>

// std::thread pool for resources recycling
class ThreadPool {
public:
// the constructor just launches some amount of workers
ThreadPool(size_t threads_n = std::thread::hardware_concurrency()) : stop(false)
{
if(!threads_n)
throw std::invalid_argument("more than zero threads expected");

this->workers.reserve(threads_n);
for(; threads_n; --threads_n)
this->workers.emplace_back(
[this]
{
while(true)
{
std::function<void()> task;

{
std::unique_lock<std::mutex> lock(this->queue_mutex);
this->condition.wait(lock,
[this]{ return this->stop || !this->tasks.empty(); });
if(this->stop && this->tasks.empty())
return;
task = std::move(this->tasks.front());
this->tasks.pop();
}

task();
}
}
);
}
// deleted copy&move ctors&assignments
ThreadPool(const ThreadPool&) = delete;
ThreadPool& operator=(const ThreadPool&) = delete;
ThreadPool(ThreadPool&&) = delete;
ThreadPool& operator=(ThreadPool&&) = delete;
// add new work item to the pool
template<class F, class... Args>
std::future<typename std::result_of<F(Args...)>::type> enqueue(F&& f, Args&&... args)
{
using packaged_task_t = std::packaged_task<typename std::result_of<F(Args...)>::type ()>;

std::shared_ptr<packaged_task_t> task(new packaged_task_t(
std::bind(std::forward<F>(f), std::forward<Args>(args)...)
));
auto res = task->get_future();
{
std::unique_lock<std::mutex> lock(this->queue_mutex);
this->tasks.emplace([task](){ (*task)(); });
}
this->condition.notify_one();
return res;
}
// the destructor joins all threads
virtual ~ThreadPool()
{
this->stop = true;
this->condition.notify_all();
for(std::thread& worker : this->workers)
worker.join();
}
private:
// need to keep track of threads so we can join them
std::vector< std::thread > workers;
// the task queue
std::queue< std::function<void()> > tasks;

// synchronization
std::mutex queue_mutex;
std::condition_variable condition;
// workers finalization flag
std::atomic_bool stop;
};

#endif // THREAD_POOL_HPP
2 changes: 1 addition & 1 deletion example.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
#include <vector>
#include <chrono>

#include "ThreadPool.h"
#include "ThreadPool.hpp"

int main()
{
Expand Down