Fixed tables getting metatables twice

ceifa / wasmoon

A real lua 5.4 VM with JS bindings made with webassembly

MIT License

496 stars 32 forks source link

Fixed tables getting metatables twice #26

Closed timstableford closed 3 years ago

timstableford commented 3 years ago

Also added a demo of wrapping native JS classes

@ceifa Sorry for the bug, though hopefully this now resolves the original that had a memory leak and my one :)

ceifa commented 3 years ago

Do this solves the problem of #25? I was trying to do a different thing on my own in #27

timstableford commented 3 years ago

Yep, solves #25 Hopefully the test shows how to accomplish with the current code what you were doing in #27

ceifa commented 3 years ago

@timstableford Tested it here and now I could understand the idea. But maybe it should be cool to work out of the box, without needing to wrap the object. I made it in #27, but I'm not sure if it's okay, can you take a look?