Use std::copy_n instead of std::memcpy, and protect std::numeric_limits<T>::max from the max macro

Author: YexuanXiao

cppwinrt/cmd_reader.h

@@ -314,7 +314,7 @@ namespace cppwinrt
     struct option
     {
         static constexpr std::uint32_t no_min = 0;
-        static constexpr std::uint32_t no_max = std::numeric_limits<std::uint32_t>::max();
+        static constexpr std::uint32_t no_max = (std::numeric_limits<std::uint32_t>::max)();
 
         std::string_view name;
         std::uint32_t min{ no_min };

strings/base_string.h

@@ -105,7 +105,7 @@ namespace winrt::impl
         }
 
         auto header = precreate_hstring_on_heap(length);
-        std::memcpy(header->buffer, value, sizeof(wchar_t) * length);
+        std::copy_n(value, length, header->buffer);
         return header;
     }
 

strings/base_string_operators.h

@@ -104,8 +104,8 @@ namespace winrt::impl
             return{};
         }
         hstring_builder text(size);
-        std::memcpy(text.data(), left.data(), left.size() * sizeof(wchar_t));
-        std::memcpy(text.data() + left.size(), right.data(), right.size() * sizeof(wchar_t));
+        std::copy_n(left.data(), left.size(), text.data());
+        std::copy_n(right.data(), right.size(), text.data() + left.size());
         return text.to_hstring();
     }
 }