Function wait_and_pop_with_timeout() is not needed any more.

Author: joto

include/osmium/thread/pool.hpp

@@ -123,7 +123,7 @@ namespace osmium {
                 osmium::thread::set_thread_name("_osmium_worker");
                 while (true) {
                     function_wrapper task;
-                    m_work_queue.wait_and_pop_with_timeout(task);
+                    m_work_queue.wait_and_pop(task);
                     if (task) {
                         if (task()) {
                             // The called tasks returns true only when the

include/osmium/thread/queue.hpp

@@ -172,27 +172,6 @@ namespace osmium {
                 }
             }
 
-            void wait_and_pop_with_timeout(T& value) {
-#ifdef OSMIUM_DEBUG_QUEUE_SIZE
-                ++m_pop_counter;
-#endif
-                std::unique_lock<std::mutex> lock(m_mutex);
-#ifdef OSMIUM_DEBUG_QUEUE_SIZE
-                if (m_queue.empty()) {
-                    ++m_empty_counter;
-                }
-#endif
-                if (!m_data_available.wait_for(lock, std::chrono::seconds(1), [this] {
-                    return !m_queue.empty();
-                })) {
-                    return;
-                }
-                if (!m_queue.empty()) {
-                    value = std::move(m_queue.front());
-                    m_queue.pop();
-                }
-            }
-
             bool try_pop(T& value) {
 #ifdef OSMIUM_DEBUG_QUEUE_SIZE
                 ++m_pop_counter;