Pass pointers as arguments instead of FFI symbols

This way we don't need to worry about exporting said symbols.
This change also deprecates the need of current library path.

Author: gilzoide

Makefile

@@ -183,7 +183,6 @@ build/%/liblua_pluginscript.so: $(BUILT_OBJS) build/%/luajit/src/libluajit.a
 
 build/%/lua_pluginscript.dll: TARGET_SYS = Windows
 build/%/lua_pluginscript.dll: EXE = .exe
-build/%/lua_pluginscript.dll: CFLAGS += -DLUAJIT_DYNAMICALLY_LINKED
 build/%/lua_pluginscript.dll: $(BUILT_OBJS) build/%/lua51.dll
 	$(_CC) -o $@ $^ -shared $(CFLAGS) $(LDFLAGS)
 	$(call STRIP_CMD,$@)

src/cache_lua_libs.lua

@@ -54,6 +54,7 @@ local setthreadfunc, touserdata
 -- FFI
 local ffi_cast, ffi_cdef, ffi_copy, ffi_gc, ffi_istype, ffi_metatype, ffi_new, ffi_sizeof, ffi_string, ffi_typeof
     = ffi.cast, ffi.cdef, ffi.copy, ffi.gc, ffi.istype, ffi.metatype, ffi.new, ffi.sizeof, ffi.string, ffi.typeof
+local clib = ffi.C
 
 -- Weak tables
 local weak_kv = { __mode = 'kv' }

src/godot_ffi.lua

@@ -1411,20 +1411,12 @@ typedef struct godot_pluginscript_language_desc {
 	void (*profiling_frame)(godot_pluginscript_language_data *p_data);
 	godot_pluginscript_script_desc script_desc;
 } godot_pluginscript_language_desc;
-
-// Global API pointers
-const godot_gdnative_core_api_struct *hgdn_core_api;
-const godot_gdnative_core_1_1_api_struct *hgdn_core_1_1_api;
-const godot_gdnative_core_1_2_api_struct *hgdn_core_1_2_api;
-const godot_gdnative_core_1_3_api_struct *hgdn_core_1_3_api;
-godot_object *hgdn_library;
 ]]
 
-local pluginscript_callbacks, in_editor, active_library_path = ...
-local clib = active_library_path and ffi.load(active_library_path, true) or ffi.C
-
-local api = clib.hgdn_core_api
-local api_1_1 = clib.hgdn_core_1_1_api
-local api_1_2 = clib.hgdn_core_1_2_api
-local api_1_3 = clib.hgdn_core_1_3_api
+local pluginscript_callbacks, in_editor, api, api_1_1, api_1_2, api_1_3, gdnativelibrary = ...
 
+api = ffi.cast('godot_gdnative_core_api_struct *', api)
+api_1_1 = ffi.cast('godot_gdnative_core_1_1_api_struct *', api_1_1)
+api_1_2 = ffi.cast('godot_gdnative_core_1_2_api_struct *', api_1_2)
+api_1_3 = ffi.cast('godot_gdnative_core_1_3_api_struct *', api_1_3)
+gdnativelibrary = ffi.cast('godot_object *', gdnativelibrary)

src/language_gdnative.c

@@ -48,10 +48,6 @@ extern const size_t LUA_INIT_SCRIPT_SIZE;
 	HGDN_PRINT_ERROR(prefix ": %s", lua_tostring(L, -1)); \
 	lua_pop(L, 1)
 
-#ifdef LUAJIT_DYNAMICALLY_LINKED
-// Active shared library path, for loading symbols in FFI
-static hgdn_string lps_active_library_path;
-#endif
 static bool lps_in_editor;
 static int lps_pcall_error_handler_index;
 static lua_State *lps_L;
@@ -136,12 +132,12 @@ static godot_pluginscript_language_data *lps_language_init() {
 	lua_setfield(L, LUA_REGISTRYINDEX, PLUGINSCRIPT_CALLBACKS_KEY);
 
 	lua_pushboolean(L, lps_in_editor);
-#ifdef LUAJIT_DYNAMICALLY_LINKED
-	lua_pushlstring(L, lps_active_library_path.ptr, lps_active_library_path.length);
-#else
-	lua_pushnil(L);
-#endif
-	if (lua_pcall(L, 3, 0, lps_pcall_error_handler_index) != LUA_OK) {
+	lua_pushlightuserdata(L, (void *) hgdn_core_api);
+	lua_pushlightuserdata(L, (void *) hgdn_core_1_1_api);
+	lua_pushlightuserdata(L, (void *) hgdn_core_1_2_api);
+	lua_pushlightuserdata(L, (void *) hgdn_core_1_3_api);
+	lua_pushlightuserdata(L, (void *) hgdn_library);
+	if (lua_pcall(L, 7, 0, lps_pcall_error_handler_index) != LUA_OK) {
 		LPS_PCALL_CONSUME_ERROR(L, "Error in Lua language initialization script");
 	}
 	return L;
@@ -384,31 +380,10 @@ GDN_EXPORT void PREFIX_SYMBOL(gdnative_init)(godot_gdnative_init_options *option
 		lps_register_in_editor_callbacks(&lps_language_desc);
 	}
 
-#ifdef LUAJIT_DYNAMICALLY_LINKED
-	godot_object *OS = hgdn_core_api->godot_global_get_singleton("OS");
-	if (hgdn_variant_get_bool_own(hgdn_object_call(OS, "has_feature", "standalone"))) {
-		godot_variant exepath_var = hgdn_object_callv(OS, "get_executable_path", NULL);
-		godot_string exepath = hgdn_core_api->godot_variant_as_string(&exepath_var);
-		godot_string exedir = hgdn_core_api->godot_string_get_base_dir(&exepath);
-		godot_string library_filepath = hgdn_core_api->godot_string_get_file(options->active_library_path);
-		lps_active_library_path = hgdn_string_get_own(hgdn_core_api->godot_string_plus_file(&exedir, &library_filepath));
-		hgdn_core_api->godot_string_destroy(&library_filepath);
-		hgdn_core_api->godot_string_destroy(&exedir);
-		hgdn_core_api->godot_string_destroy(&exepath);
-		hgdn_core_api->godot_variant_destroy(&exepath_var);
-	}
-	else {
-		lps_active_library_path = hgdn_string_get(options->active_library_path);
-	}
-#endif
-
 	hgdn_pluginscript_api->godot_pluginscript_register_language(&lps_language_desc);
 }
 
 GDN_EXPORT void PREFIX_SYMBOL(gdnative_terminate)(godot_gdnative_terminate_options *options) {
-#ifdef LUAJIT_DYNAMICALLY_LINKED
-	hgdn_string_destroy(&lps_active_library_path);
-#endif
 	hgdn_gdnative_terminate(options);
 }
 

src/late_globals.lua

@@ -35,7 +35,7 @@ GD = {
 	--- (`const godot_gdnative_core_1_2_api_struct *`) GDNative core API 1.2
 	api_1_2 = api_1_2,
 	--- (`Object`) [GDNativeLibrary](https://docs.godotengine.org/en/stable/classes/class_gdnativelibrary.html) instance
-	gdnativelibrary = clib.hgdn_library,
+	gdnativelibrary = gdnativelibrary,
 	--- `Enumerations.Error`
 	Error = Error,
 	--- `Enumerations.VariantType`

src/lua_object_wrapper.lua

@@ -26,7 +26,7 @@
 -- @module lua_object_wrapper
 -- @local
 
-local library_resource_dir = clib.hgdn_library.resource_path:get_base_dir()
+local library_resource_dir = gdnativelibrary.resource_path:get_base_dir()
 
 --- Global cache of Lua object wrappers
 local lps_lua_object_references = setmetatable({}, weak_kv)