Fixed bug for old versions of Windows

CMakeLists.txt

@@ -98,6 +98,8 @@ set_target_properties(concurrencpp PROPERTIES
         SOVERSION "${PROJECT_VERSION_MAJOR}"
 )
 
+set_target_properties(${PROJECT_NAME} PROPERTIES DEBUG_POSTFIX "d")
+
 target_coroutine_options(concurrencpp)
 
 target_compile_definitions(concurrencpp

include/concurrencpp/results/impl/shared_result_state.h

@@ -5,6 +5,7 @@
 #include "concurrencpp/results/impl/result_state.h"
 
 #include <atomic>
+#include <chrono>
 #include <semaphore>
 
 #include <cassert>

include/concurrencpp/results/result.h

@@ -11,7 +11,7 @@
 
 namespace concurrencpp {
     template<class type>
-    class result {
+    class [[nodiscard]] result {
 
         static constexpr auto valid_result_type_v = std::is_same_v<type, void> || std::is_nothrow_move_constructible_v<type>;
 

include/concurrencpp/threads/thread.h

@@ -4,6 +4,7 @@
 #include "concurrencpp/platform_defs.h"
 
 #include <functional>
+#include <string>
 #include <string_view>
 #include <thread>
 

source/threads/thread.cpp

@@ -47,11 +47,17 @@ size_t thread::hardware_concurrency() noexcept {
 #ifdef CRCPP_WIN_OS
 
 #    include <Windows.h>
+#    include <VersionHelpers.h>
 
 void thread::set_name(std::string_view name) noexcept {
-    const std::wstring utf16_name(name.begin(),
-                                  name.end());  // concurrencpp strings are always ASCII (english only)
-    ::SetThreadDescription(::GetCurrentThread(), utf16_name.data());
+    if (IsWindowsVersionOrGreater(HIBYTE(_WIN32_WINNT_WIN10), LOBYTE(_WIN32_WINNT_WIN10), 14393)) {
+        // Windows 10 1607 or greater, use SetThreadDescription
+        const std::wstring utf16_name(name.begin(),
+                                      name.end());  // concurrencpp strings are always ASCII (english only)
+        ::SetThreadDescription(::GetCurrentThread(), utf16_name.c_str());
+    } else {
+        // Windows version is less than Windows 10, do nothing
+    }
 }
 
 #elif defined(CRCPP_MINGW_OS)