C++11 migration:

- use lambda functions instead of binding member functions
- replace boost::mutex by STLs <mutex>
- reformat according to new guidelines
This commit is contained in:
Dennis Luxen 2014-05-08 15:47:48 +02:00
parent 88a4bb4d12
commit 37b8f97d60

View File

@ -30,11 +30,10 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
#include "../typedefs.h" #include "../typedefs.h"
#include <boost/bind.hpp>
#include <boost/circular_buffer.hpp> #include <boost/circular_buffer.hpp>
#include <boost/thread/condition.hpp> #include <condition_variable>
#include <boost/thread/mutex.hpp> #include <functional>
#include <boost/thread/thread.hpp> #include <mutex>
template <typename Data> class ConcurrentQueue template <typename Data> class ConcurrentQueue
{ {
@ -44,10 +43,12 @@ template <typename Data> class ConcurrentQueue
inline void push(const Data &data) inline void push(const Data &data)
{ {
boost::mutex::scoped_lock lock(m_mutex); std::unique_lock<std::mutex> lock(m_mutex);
m_not_full.wait(lock, boost::bind(&ConcurrentQueue<Data>::is_not_full, this)); m_not_full.wait(lock,
[this]
{ return m_internal_queue.size() < m_internal_queue.capacity(); });
m_internal_queue.push_back(data); m_internal_queue.push_back(data);
lock.unlock(); m_mutex.unlock();
m_not_empty.notify_one(); m_not_empty.notify_one();
} }
@ -55,40 +56,35 @@ template <typename Data> class ConcurrentQueue
inline void wait_and_pop(Data &popped_value) inline void wait_and_pop(Data &popped_value)
{ {
boost::mutex::scoped_lock lock(m_mutex); std::unique_lock<std::mutex> lock(m_mutex);
m_not_empty.wait(lock, boost::bind(&ConcurrentQueue<Data>::is_not_empty, this)); m_not_empty.wait(lock,
[this]
{ return !m_internal_queue.empty(); });
popped_value = m_internal_queue.front(); popped_value = m_internal_queue.front();
m_internal_queue.pop_front(); m_internal_queue.pop_front();
lock.unlock(); m_mutex.unlock();
m_not_full.notify_one(); m_not_full.notify_one();
} }
inline bool try_pop(Data &popped_value) inline bool try_pop(Data &popped_value)
{ {
boost::mutex::scoped_lock lock(m_mutex); std::unique_lock<std::mutex> lock(m_mutex);
if (m_internal_queue.empty()) if (m_internal_queue.empty())
{ {
return false; return false;
} }
popped_value = m_internal_queue.front(); popped_value = m_internal_queue.front();
m_internal_queue.pop_front(); m_internal_queue.pop_front();
lock.unlock(); m_mutex.unlock();
m_not_full.notify_one(); m_not_full.notify_one();
return true; return true;
} }
private: private:
inline bool is_not_empty() const { return !m_internal_queue.empty(); }
inline bool is_not_full() const
{
return m_internal_queue.size() < m_internal_queue.capacity();
}
boost::circular_buffer<Data> m_internal_queue; boost::circular_buffer<Data> m_internal_queue;
boost::mutex m_mutex; std::mutex m_mutex;
boost::condition m_not_empty; std::condition_variable m_not_empty;
boost::condition m_not_full; std::condition_variable m_not_full;
}; };
#endif // CONCURRENT_QUEUE_H #endif // CONCURRENT_QUEUE_H