Remove remaining export_name + cdecl hacks

Author: kateinoigakukun

Examples/Embedded/Sources/EmbeddedApp/_thingsThatShouldNotBeNeeded.swift

@@ -1,12 +1,5 @@
 import JavaScriptKit
 
-// NOTE: it seems the embedded tree shaker gets rid of these exports if they are not used somewhere
-func _i_need_to_be_here_for_wasm_exports_to_work() {
-    _ = _swjs_library_features
-    _ = _swjs_call_host_function
-    _ = _swjs_free_host_function
-}
-
 // TODO: why do I need this? and surely this is not ideal... figure this out, or at least have this come from a C lib
 @_cdecl("strlen")
 func strlen(_ s: UnsafePointer<Int8>) -> Int {

Sources/JavaScriptKit/FundamentalObjects/JSClosure.swift

@@ -197,8 +197,10 @@ private func makeAsyncClosure(
 // └─────────────────────┴──────────────────────────┘
 
 /// Returns true if the host function has been already released, otherwise false.
-@_cdecl("_call_host_function_impl")
-func _call_host_function_impl(
+@_expose(wasm, "swjs_call_host_function")
+@_cdecl("_swjs_call_host_function")
+@available(*, unavailable)
+public func _swjs_call_host_function(
     _ hostFuncRef: JavaScriptHostFuncRef,
     _ argv: UnsafePointer<RawJSValue>, _ argc: Int32,
     _ callbackFuncRef: JavaScriptObjectRef
@@ -231,9 +233,10 @@ extension JSClosure {
     }
 }
 
-
-@_cdecl("_free_host_function_impl")
-func _free_host_function_impl(_ hostFuncRef: JavaScriptHostFuncRef) {}
+@_expose(wasm, "swjs_free_host_function")
+@_cdecl("_swjs_free_host_function")
+@available(*, unavailable)
+func _swjs_free_host_function(_ hostFuncRef: JavaScriptHostFuncRef) {}
 
 #else
 
@@ -244,25 +247,10 @@ extension JSClosure {
 
 }
 
-@_cdecl("_free_host_function_impl")
-func _free_host_function_impl(_ hostFuncRef: JavaScriptHostFuncRef) {
-    JSClosure.sharedClosures.wrappedValue[hostFuncRef] = nil
-}
-#endif
-
-#if compiler(>=6.0) && hasFeature(Embedded)
-// cdecls currently don't work in embedded, and expose for wasm only works >=6.0
-@_expose(wasm, "swjs_call_host_function")
-public func _swjs_call_host_function(
-        _ hostFuncRef: JavaScriptHostFuncRef,
-        _ argv: UnsafePointer<RawJSValue>, _ argc: Int32,
-        _ callbackFuncRef: JavaScriptObjectRef) -> Bool {
-
-    _call_host_function_impl(hostFuncRef, argv, argc, callbackFuncRef) 
-}
-
 @_expose(wasm, "swjs_free_host_function")
-public func _swjs_free_host_function(_ hostFuncRef: JavaScriptHostFuncRef) {
-    _free_host_function_impl(hostFuncRef)
+@_cdecl("_swjs_free_host_function")
+@available(*, unavailable)
+func _swjs_free_host_function(_ hostFuncRef: JavaScriptHostFuncRef) {
+    JSClosure.sharedClosures.wrappedValue[hostFuncRef] = nil
 }
 #endif

Sources/_CJavaScriptKit/_CJavaScriptKit.c

@@ -28,24 +28,6 @@ void swjs_cleanup_host_function_call(void *argv_buffer) {
 // cdecls don't work in Embedded, but @_expose(wasm) can be used with Swift >=6.0
 // the previously used `#if __Embedded` did not play well with SwiftPM (defines needed to be on every target up the chain)
 #ifdef __wasi__
-bool _call_host_function_impl(const JavaScriptHostFuncRef host_func_ref,
-                              const RawJSValue *argv, const int argc,
-                              const JavaScriptObjectRef callback_func);
-
-__attribute__((export_name("swjs_call_host_function")))
-bool swjs_call_host_function(const JavaScriptHostFuncRef host_func_ref,
-                             const RawJSValue *argv, const int argc,
-                             const JavaScriptObjectRef callback_func) {
-    return _call_host_function_impl(host_func_ref, argv, argc, callback_func);
-}
-
-void _free_host_function_impl(const JavaScriptHostFuncRef host_func_ref);
-
-__attribute__((export_name("swjs_free_host_function")))
-void swjs_free_host_function(const JavaScriptHostFuncRef host_func_ref) {
-    _free_host_function_impl(host_func_ref);
-}
-
 int swjs_get_worker_thread_id_cached(void) {
     _Thread_local static int tid = 0;
     if (tid == 0) {