melonDS
melonDS copied to clipboard
Basic Lua Scripting Support
Here is a proposed starting point to adding Lua script support for MelonDS natively. Currently has bare minimal functionality, as I would like to have others review my work before making to many additions. This is my first project in c++ so please let me know if there are any errors, or parts that I need to re-do. The aim for this is to support accessibility features and trackers, not as a comprehensive debugging tool.
Still some more changes needed but majority of the refactoring is done, no more LuaThread, the Lua state is created and run by the emuThread. This should hopefully address any concerns around potential race conditions. I plan on cleaning up other issues on Sunday.
I think this should be good for another review, I believe I have addressed all the major issues that have been brought up. If there is anything else that I need to change or fix please let me know. There are still more features / functions I'd like to add, but this is the kind of project that could easily lead to feature creep, so I'm gonna cut myself off here.
there's probably still some more things, but this is what I found from a glance.
OK addressed those issues, still some bugs I found that I need to fix, I plan on working on that tomorrow.
OK ready for another review @RSDuck when you have time.
OK went over everything, double checked to make sure the spacing was good. Fixed spelling errors in the markdown. Will probably need someone else to take a look at the merge conflicts at least for the .yml and CMakeLists files.
blah something went wrong while resolving the merge conflict.
hm I don't really understand how dependencies are managed on macOS, I can't find anything on how it's installed in the code.