fix: avoid max/min macros from colliding with calls to std::numeric_limits<uint32_t>::max (#942)

Windows headers (e.g. minwindef.h) define min and max macros which collied with std::numeric_limits<uint32_t>::max.

The C (and therefore C++) preprocessor automatically expands function like macros. i.e. macro identifiers immediately followed by "(", e.g. max().

Author: gaurang-ingale

include/quill/std/WideString.h

@@ -48,7 +48,7 @@ struct Codec<T,
 
     // The length is stored as uint32_t on the wire. Clamp before casting so a release
     // build cannot truncate-then-overflow when given a pathologically large string.
-    constexpr size_t max_len = std::numeric_limits<uint32_t>::max();
+    constexpr size_t max_len = (std::numeric_limits<uint32_t>::max)();
     QUILL_ASSERT(
       len <= max_len,
       "Wide string length exceeds uint32_t max in Codec<std::wstring>::compute_encoded_size()");