melonDS icon indicating copy to clipboard operation
melonDS copied to clipboard

Basic Lua Scripting Support

Open NPO-197 opened this issue 1 year ago • 13 comments

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.

NPO-197 avatar Apr 20 '23 23:04 NPO-197

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.

NPO-197 avatar Apr 28 '23 16:04 NPO-197

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.

NPO-197 avatar May 06 '23 19:05 NPO-197

there's probably still some more things, but this is what I found from a glance.

RSDuck avatar May 06 '23 19:05 RSDuck

OK addressed those issues, still some bugs I found that I need to fix, I plan on working on that tomorrow.

NPO-197 avatar May 07 '23 19:05 NPO-197

OK ready for another review @RSDuck when you have time.

NPO-197 avatar May 23 '23 21:05 NPO-197

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.

NPO-197 avatar Jun 08 '23 21:06 NPO-197

blah something went wrong while resolving the merge conflict.

RSDuck avatar Jun 12 '23 12:06 RSDuck

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.

RSDuck avatar Jun 12 '23 12:06 RSDuck