Skip to content

fix: protect Lua event registration calls - #71

Merged
eduardodoria merged 2 commits into
doriaxengine:mainfrom
nkbeast:fix/protect-lua-event-registration
Sep 3, 2026
Merged

fix: protect Lua event registration calls#71
eduardodoria merged 2 commits into
doriaxengine:mainfrom
nkbeast:fix/protect-lua-event-registration

Conversation

@nkbeast

@nkbeast nkbeast commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • route the Lua event:add call through the existing traceback-aware protected call helper
  • propagate registration errors to the calling Lua script instead of allowing an unprotected panic

Validation

  • source check: no raw lua_call remains in LuaBinding.cpp
  • git diff --check — passed
  • CMake configure was attempted but this environment lacks the system CURL development library (CURL_LIBRARY/CURL_INCLUDE_DIR)

@eduardodoria
eduardodoria merged commit 080532b into doriaxengine:main Sep 3, 2026
3 checks passed
@eduardodoria

Copy link
Copy Markdown
Contributor

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants