fix: Remove conflicting StringMaker definitions and fix ODR violation

- Remove StringMaker<std::string_view> from text_memstream_test.cpp
- Remove StringMaker<std::byte> from data_bits_test.cpp
- Catch2 v3 provides these definitions already
- Add MH_COMPILE_LIBRARY_INLINE to thread_local static definition

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: mhaynie

cpp/include/mh/concurrency/dispatcher.inl

@@ -419,7 +419,7 @@ namespace mh
 	}
 
 	// Static member definition
-	thread_local dispatcher* dispatcher::s_current_thread_dispatcher = nullptr;
+	MH_COMPILE_LIBRARY_INLINE thread_local dispatcher* dispatcher::s_current_thread_dispatcher = nullptr;
 
 	MH_COMPILE_LIBRARY_INLINE void dispatcher::register_for_current_thread()
 	{

test/data_bits_test.cpp

@@ -2,16 +2,6 @@
 #include <catch2/catch_all.hpp>
 #include "last_include.hpp"
 
-// Provide definition for Catch2's declared StringMaker specialization
-// Only define when building mh_stuff as standalone project to avoid conflicts
-#ifdef MH_STUFF_STANDALONE_BUILD
-namespace Catch {
-    std::string StringMaker<std::byte>::convert(std::byte value) {
-        return std::to_string(static_cast<unsigned int>(value));
-    }
-}
-#endif
-
 template<unsigned bits_to_copy, unsigned src_offset, typename TSrc = void, typename TDst = void>
 static void test_bit_functions(const TSrc* src, const TDst expected)
 {

test/text_memstream_test.cpp

@@ -4,16 +4,6 @@
 #include <cstring>
 #include "last_include.hpp"
 
-// Provide definition for Catch2's declared StringMaker specialization
-// Only define when building mh_stuff as standalone project to avoid conflicts
-#ifdef MH_STUFF_STANDALONE_BUILD
-namespace Catch {
-    std::string StringMaker<std::string_view>::convert(std::string_view str) {
-        return std::string(str);
-    }
-}
-#endif
-
 TEST_CASE("memstream put", "[text][memstream]")
 {
 	char buf[128];