⚗️ add -WX flag to MSVC builds

test/src/unit-allocator.cpp

@@ -100,6 +100,9 @@ struct my_allocator : std::allocator<T>
         std::allocator<T>::deallocate(p, n);
     }
 
+    // the code below warns about p in MSVC 2015 - this could be a bug
+    DOCTEST_MSVC_SUPPRESS_WARNING_PUSH
+    DOCTEST_MSVC_SUPPRESS_WARNING(4100)
     void destroy(T* p)
     {
         if (next_destroy_fails)
@@ -110,6 +113,7 @@ struct my_allocator : std::allocator<T>
 
         p->~T();
     }
+    DOCTEST_MSVC_SUPPRESS_WARNING_POP
 
     template <class U>
     struct rebind

test/src/unit-udt.cpp

@@ -809,6 +809,10 @@ TEST_CASE("an incomplete type does not trigger a compiler error in non-evaluated
     static_assert(!is_constructible_patched<json, incomplete>::value, "");
 }
 
+// the code below warns about t in MSVC 2015 - this could be a bug
+DOCTEST_MSVC_SUPPRESS_WARNING_PUSH
+DOCTEST_MSVC_SUPPRESS_WARNING(4100)
+
 namespace
 {
 class Evil
@@ -821,6 +825,8 @@ class Evil
     int m_i = 0;
 };
 
+DOCTEST_MSVC_SUPPRESS_WARNING_POP
+
 void from_json(const json& /*unused*/, Evil& /*unused*/) {}
 } // namespace