Merge pull request #853 from furkanusta/windows-parsingchecks

Enable numberparsingcheck and stringparsingcheck on MSVC

tests/CMakeLists.txt

@@ -42,12 +42,12 @@ include(${PROJECT_SOURCE_DIR}/tests/add_cpp_test.cmake)
 #
 # These tests explicitly do #include "simdjson.cpp" so they can override stuff
 #
-if (NOT MSVC) # Can't get simdjson-source to compile on Windows for some reason.
-  add_cpp_test(numberparsingcheck LABELS acceptance per_implementation)
-  target_link_libraries(numberparsingcheck simdjson-include-source)
-  add_cpp_test(stringparsingcheck LABELS acceptance per_implementation)
-  target_link_libraries(stringparsingcheck simdjson-include-source)
-endif()
+add_cpp_test(numberparsingcheck LABELS acceptance per_implementation)
+target_link_libraries(numberparsingcheck simdjson-include-source simdjson-windows-headers)
+target_compile_definitions(numberparsingcheck PRIVATE NOMINMAX)
+add_cpp_test(stringparsingcheck LABELS acceptance per_implementation)
+target_link_libraries(stringparsingcheck simdjson-include-source simdjson-windows-headers)
+target_compile_definitions(stringparsingcheck PRIVATE NOMINMAX)
 
 # All remaining tests link with simdjson proper
 link_libraries(simdjson)

tests/numberparsingcheck.cpp

@@ -1,5 +1,4 @@
 #include <cstring>
-#include <dirent.h>
 #include <inttypes.h>
 #include <math.h>
 #include <stdbool.h>
@@ -10,6 +9,11 @@
 #define JSON_TEST_NUMBERS
 #endif
 
+#ifndef _MSC_VER
+#include <dirent.h>
+#else
+#include <dirent_portable.h>
+#endif
 #include "simdjson.h"
 
 // ulp distance

tests/stringparsingcheck.cpp

@@ -1,7 +1,6 @@
 #include <cassert>
 #include <climits>
 #include <cstring>
-#include <dirent.h>
 #include <inttypes.h>
 #include <iostream>
 #include <math.h>
@@ -13,6 +12,11 @@
 #define JSON_TEST_STRINGS
 #endif
 
+#ifndef _MSC_VER
+#include <dirent.h>
+#else
+#include <dirent_portable.h>
+#endif
 #include "simdjson.h"
 
 char *fullpath;

windows/CMakeLists.txt

@@ -1,4 +1,4 @@
 add_library(simdjson-windows-headers INTERFACE)
 if(MSVC)
   target_include_directories(simdjson-windows-headers INTERFACE .)
-endif()
+endif()