search

arcticfox1919 / LuaDardo

A Lua virtual machine written in Dart
Apache License 2.0
172 stars 32 forks source link

Refactored LuaTable implementation #16

Closed Mytrin closed 1 year ago

Mytrin commented 1 year ago

Greetings,

I've decided to fix https://github.com/arcticfox1919/LuaDardo/issues/10. First commit of this MR contains a simple test, which proves that "next keys" map is not reinitialized after insertion of new key.

I ended up extracting parts of LuaTable to their own classes, so lazy initialization and null checks are encapsulated and easier to maintain.

Mytrin commented 1 year ago

I moved changes into a new branch, so it messed up this PR a bit. Moved on into a new: https://github.com/arcticfox1919/LuaDardo/pull/19