Fix return value for wait_for

Author: gilzoide

include/dispatch_queue.hpp

@@ -113,31 +113,28 @@ class dispatch_queue {
 
 	/**
 	 * Wait until all pending tasks finish processing.
-	 * @see std::future<T>::wait
 	 */
 	void wait();
 
 	/**
 	 * Wait until all pending tasks finish processing.
-	 * Blocks until specified `timeout_duration` has elapsed or the result becomes available, whichever comes first.
-	 * The return value indicates why `wait_for` returned.
-	 * @see std::future<T>::wait_for
+	 * Blocks until specified `timeout_duration` has elapsed or all queued tasks complete, whichever comes first.
+	 * @returns `false` if the timeout has expired, otherwise `true`.
 	 */
 	template<class Rep, class Period>
-	std::future_status wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) {
+	bool wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) {
 		if (worker_pool) {
 			return worker_pool->wait_for(timeout_duration);
 		}
 		else {
-			return std::future_status::ready;
+			return true;
 		}
 	}
 
 	/**
 	 * Wait until all pending tasks finish processing.
-	 * Blocks until the specified `timeout_time` has been reached or the result becomes available, whichever comes first.
-	 * The return value indicates why `wait_until` returned.
-	 * @see std::future<T>::wait_until
+	 * Blocks until the specified `timeout_time` has been reached or all queued tasks complete, whichever comes first.
+	 * @returns `false` if the timeout has expired, otherwise `true`.
 	 */
 	template<class Clock, class Duration>
 	bool wait_until(const std::chrono::time_point<Clock, Duration>& timeout_time) {

include/task.hpp

@@ -102,10 +102,10 @@ class task {
 	 * If the task is pending (`get_state() == task_state::pending`), blocks until task finishes or until the specified `timeout_duration` has elapsed.
 	 * Otherwise returns immediately without blocking.
 	 *
-	 * @returns `std::future_status::timeout` if the timeout has expired, otherwise `std::future_status::ready`.
+	 * @returns `false` if the timeout has expired, otherwise `true`.
 	 */
 	template<class Rep, class Period>
-	std::future_status wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) const {
+	bool wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) const {
 		return future->wait_for(timeout_duration);
 	}
 

include/task_future.hpp

@@ -59,7 +59,7 @@ class task_future_base {
 	}
 
 	template<class Rep, class Period>
-	std::future_status wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) {
+	bool wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) {
 		std::unique_lock<std::mutex> lock(mutex);
 		return condition_variable.wait_for(lock, timeout_duration, wait_predicate());
 	}

include/worker_pool.hpp

@@ -45,7 +45,7 @@ class worker_pool {
 	void wait();
 
 	template<class Rep, class Period>
-	std::future_status wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) {
+	bool wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) {
 		std::unique_lock<std::mutex> lock(mutex);
 		return all_done_condition_variable.wait_for(lock, timeout_duration, wait_predicate());
 	}