Fix task_future::wait_until, remove manual check for predicate

Author: gilzoide

include/task.hpp

@@ -115,10 +115,10 @@ class task {
 	 * If the task is pending (`get_state() == task_state::pending`), blocks until task finishes or until the specified `timeout_time` has been reached.
 	 * 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 Clock, class Duration>
-	std::future_status wait_until(const std::chrono::time_point<Clock, Duration>& timeout_time) const {
+	bool wait_until(const std::chrono::time_point<Clock, Duration>& timeout_time) const {
 		return future->wait_until(timeout_time);
 	}
 

include/task_future.hpp

@@ -55,28 +55,19 @@ class task_future_base {
 
 	void wait() {
 		std::unique_lock<std::mutex> lock(mutex);
-		if (state != task_state::pending) {
-			return;
-		}
 		condition_variable.wait(lock, wait_predicate());
 	}
 
 	template<class Rep, class Period>
 	std::future_status wait_for(const std::chrono::duration<Rep, Period>& timeout_duration) {
 		std::unique_lock<std::mutex> lock(mutex);
-		if (state != task_state::pending) {
-			return std::future_status::ready;
-		}
 		return condition_variable.wait_for(lock, timeout_duration, wait_predicate());
 	}
 
 	template<class Clock, class Duration>
-	std::future_status wait_until(const std::chrono::time_point<Clock, Duration>& timeout_time) {
+	bool wait_until(const std::chrono::time_point<Clock, Duration>& timeout_time) {
 		std::unique_lock<std::mutex> lock(mutex);
-		if (state != task_state::pending) {
-			return std::future_status::ready;
-		}
-		return condition_variable.wait_for(lock, timeout_time, wait_predicate());
+		return condition_variable.wait_until(lock, timeout_time, wait_predicate());
 	}
 
 protected: