In NandPaths.cpp, the `std::initializer_list<char>` of illegal characters has been turned into a `char[]` (similar to the one in GameList.cpp).

The reverse iteration in ResourcePack.cpp seemed to provide no benefits, and doing without it it seemed to have no ill effects.

The inequality operator is automatically generated by the compiler if `operator==` is defined.

Move some #includes around to match the Contributing guidelines.

Ensures that these go into the ro section.

We can use the string_view arguments to directly construct strings
inside of the global_result vector.

We can just use holds_alternative here instead.

Instead, we make the event take a reference to the system and then pass
it in when the event is triggered.

This does introduce two other accessors, but these are much easier to
refactor out over time, and without modification to the existing event
interface.