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 CLuaObject + Updated CLuaInterface #109

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

RaphaelIT7
Copy link
Contributor

Added a few members to CLuaInterface and Added all members to CLuaObject with a few useful functions.
While not all fields are directly useful, they could be in some cases, and having a more accurate class is also not a bad thing.
I tested it on Windows on x86 and x86-64, but I didn't test if it works on Apple and if the offset for it is still needed.

@danielga danielga self-assigned this Apr 29, 2024
@danielga danielga self-requested a review April 29, 2024 22:57
@RaphaelIT7
Copy link
Contributor Author

Added a few more changes today but I think it should be done now. (If I cant find out what the remaining unknown members are used for)
New list of changes to get a better Overview:
[+] Added ILuaThreadedCall
[+] Added CLuaObject
[#] Added a NOTE to the ILuaObject
[#] Updated ILuaInterface

  • [#] AddThreadedCall returns an int (was void*)
  • [#] GetNewTable returns an ILuaObject* (was void)

[#] Updated CLuaInterface

  • [+] Added a bunch of private members
  • [+] Added GetStringPool()

Forgot to change it and to include <list>
Hopefully the last commit xd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants