Add support for windows with MSVC toolchain (#97)

* Add nmake file for building with windows MSVC

* Fix function export for msvc build

* Add MSVC target to test matrix

* [ci] Use patched luarocks to fix msvc setup

* [ci] Add workaround for missing compat53

* [ci] Use patched luarocks to avoid compat53 issue

* [ci] Update to upstream version of luarocks

The patch for the lua compat53 issue has been merged now

Author: tobil4sk

.github/workflows/ci.yml

@@ -46,14 +46,15 @@ jobs:
       fail-fast: false
       matrix:
         lua: [lua=5.1, lua=5.2, lua=5.3, lua=5.4, luajit=2.0, luajit=2.1]
+        target: [mingw,vs]
     runs-on: windows-2022
     steps:
       # Checks-out the repository under $GITHUB_WORKSPACE.
       - uses: actions/checkout@v4
       - name: Install Lua (${{ matrix.lua }})
         run: |
           pip install hererocks
-          hererocks lua_install -r@3a142ce --${{ matrix.lua }}
+          hererocks lua_install -r@28f9d98 --${{ matrix.lua }} --target ${{ matrix.target }}
       - name: Build lua-simdjson
         run: |
           .\lua_install\bin\activate

.gitignore

@@ -3,3 +3,8 @@ test/
 *.dll
 *.o
 *.d
+
+# msvc
+*.obj
+*.lib
+*.exp

Makefile.win

@@ -0,0 +1,27 @@
+OBJ = src/luasimdjson.obj src/simdjson.obj
+CPPFLAGS = -I$(LUA_INCDIR)
+CXXFLAGS = -EHsc -std:c++17 $(CFLAGS)
+LDFLAGS = $(LIBFLAG)
+
+!ifdef LUA_LIBDIR
+LDLIBS = $(LUA_LIBDIR)/$(LUALIB)
+!endif
+
+TARGET = simdjson.dll
+
+all: $(TARGET)
+
+src/luasimdjson.obj: src/luasimdjson.h src/simdjson.h
+src/simdjson.obj: src/simdjson.h
+
+.cpp.obj::
+	$(CXX) $(CPPFLAGS) $(CXXFLAGS) -c $< -Fo:"src\\"
+
+$(TARGET): $(OBJ)
+	$(LD) $(LDFLAGS) $** -out:$@ $(LDLIBS)
+
+clean:
+	del *.dll src\*.obj *.lib *.exp 2>nul
+
+install: $(TARGET)
+	copy $(TARGET) $(INST_LIBDIR)

lua-simdjson-0.0.6-1.rockspec

@@ -36,6 +36,7 @@ build = {
          build_variables = {
             LUA_LIBDIR="$(LUA_LIBDIR)",
             LUALIB="$(LUALIB)",
+            LD="$(LD)",
          }
       }
    }

src/luasimdjson.h

@@ -1,20 +1,26 @@
 #include <lua.hpp>
+
+#ifdef _MSC_VER
+#define LUASIMDJSON_EXPORT __declspec(dllexport)
+#else
+#define LUASIMDJSON_EXPORT extern
+#endif
+
 extern "C" {
 	static int parse(lua_State*);
 	static int parse_file(lua_State*);
 	static int active_implementation(lua_State*);
 	static int ParsedObject_open(lua_State*);
 	static int ParsedObject_open_file(lua_State*);
 
-
 	static const struct luaL_Reg luasimdjson[] = {
 		{"parse", parse},
 		{"parseFile", parse_file},
 		{"activeImplementation", active_implementation},
 		{"open", ParsedObject_open},
 		{"openFile", ParsedObject_open_file},
-			
+
 		{NULL, NULL},
 	};
-	int luaopen_simdjson (lua_State*);
+	LUASIMDJSON_EXPORT int luaopen_simdjson(lua_State*);
 }