Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for RefCounted types via lightuserdata #59

Merged
merged 11 commits into from Jan 18, 2023
Merged

Added support for RefCounted types via lightuserdata #59

merged 11 commits into from Jan 18, 2023

Conversation

Trey2k
Copy link
Member

@Trey2k Trey2k commented Jan 18, 2023

Changes:

  • Changed the argument order for push_variant moving the name to be the first argument.
  • Changed the argument order for expose_constructor moving the name to be the first argument.
  • Added new metatable for mt_RefCounted.
  • Added support to LuaState::pushVariant() for mt_RefCounted.
  • Added support to LuaState::getVariant() for mt_RefCounted.
  • Changed memory management. We now ignore Lua's garbage collector. Instead, all memory owned by the lua state is free'd at its death. This was a bi product of the change, and we may want to add support back in the future for Luas garbage collector.
  • Readme and DOC info updated as well as some grammatical fixes.

@Trey2k Trey2k linked an issue Jan 18, 2023 that may be closed by this pull request
@Trey2k
Copy link
Member Author

Trey2k commented Jan 18, 2023

Support for Lua garbage collection was added back.

@Trey2k Trey2k merged commit 19d1102 into main Jan 18, 2023
@Trey2k Trey2k deleted the dev/41 branch January 18, 2023 21:28
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.

can't hold resource?!?!?!?
1 participant