Closed gudzpoz closed 2 months ago
Attention: Patch coverage is 75.00000%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 94.36%. Comparing base (
c5ff946
) to head (3982e58
).
Files | Patch % | Lines |
---|---|---|
.../iroiro/luajava/jsr223/LuaScriptEngineFactory.java | 0.00% | 0 Missing and 1 partial :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This PR adds LuaJ bindings to LuaJava, hopefully serving as a placeholder for platforms without binaries (or not yet supported by
jnigen
). For example, iOS users may use LuaJ bindings since they are pure Java.There are several caveats though:
LuaJ does not provide Lua C API, upon which this library builds. A compatible layer is added, but there are several hidden behavioral changes in stack operations:
When a LuaJ function is called, the library does not supply a
lua_State
-like pointer, which means we cannot possibly manipulate the stack from within the function. This is handled by tracking the thread that created the function / any value, while standard Lua operates on the current thread.These should have been documented in
docs/
?