Derek Chan

It supports helmet attachments already (https://github.com/ja2-stracciatella/ja2-stracciatella/pull/1176). It is a bug if it doesn't

~It is merged but not yet there in 0.18.0. It will be in the next release, or try nightly builds.~ No it is already in 0.18.0

I misread this. @Stephen-Cronin do you mean you have only Night Goggles but not Sun Goggles, and Ctrl-N prefers the main slot to a helmet slot? I think it should...

The code should build and work fine without the library. Maybe we can add a flag in CMake build, which would just place an empty `magic_enum.hpp` in the include path,...

I had a crash too after the first battle in Omerta. It happens right after the fight when I went back to laptop and click Email. But I can't reproduce...

## Compiler support I haven't tried anything other than Visual C++, but it seems OK by checking the version numbers. We will need to test and might need to upgrade...

Checked out the old docs, `sol2` v2 has a similar yet incompatible API. It should be relatively easy to upgrade later on. RE errors, we can first parse and process...

C++code with sol2 v2: ```c++ lua.open_libraries(sol::lib::base); // declare struct bindings lua.new_usertype("SECTORINFO", "ubNumAdmins", &SECTORINFO::ubNumAdmins, "ubNumTroops", &SECTORINFO::ubNumTroops, "ubNumElites", &SECTORINFO::ubNumElites ); // bind global variable lua["SectorInfo"] = &SectorInfo; // bind functions lua.set_function("AddItemToPool", AddItemToPool);...

Turned out even sol2(v2) requires C++ 14. Our build bots are fine with it (https://github.com/ja2-stracciatella/ja2-stracciatella/pull/1209 _- VC++ is fine, as I am using it on my dev env_), and I...

I prefer not supporting multiple mods by default. We cannot ensure mods work well with each other. If we must support that, we can check if mods override the same...