JoeStrout / miniscript

source code of both C# and C++ implementations of the MiniScript scripting language
MIT License
280 stars 64 forks source link

C++: missing `map.pull` method #141

Closed JoeStrout closed 7 months ago

JoeStrout commented 7 months ago

Calling .pull on a map should do the same thing as .pop (since order of keys in a map is undefined), and it does so in the C# version. But in the C++ implementation, it generates a runtime error.

There's code for handling maps in intrinsic_pull, so it looks like this one is simply missing from the definition of Intrinsics::MapType.