Interpreter PR Todo Items

[mateofio]

While we slog through the interpreter PR's here is the list of todo items which are being postponed to avoid rebase issues

• Remove Game_Map::RefreshType
• Cleanup Game_Interpreter::GetFrame()
• Streamline Game_Map::RequestMap() logic in one place. Currently in Game_Player, Scene_Map, and Scene_Debug. Should be after Implement fully async transitions #1784 is merged.
• Remove Async globals and store the async reason and parameters in the AsyncContext
• Should we verify save game command chunks against event commands in Refactor Interpreter Stack Frame Completion  #1884? How does RPG_RT do it?
• Are level up / skill learned messages word wrapped? Check EXP, Level, and Class Change.
• Can self destruct / escape be undone with show enemy command? - YES
• Verify what happens if we change event page after event erased. What happens to LSD chunks?

[Ghabry]

TBD: Fix performance issues caused by #1740 . Need to confirm my test was correct when back home.

[mateofio]

Confirmed the following in RPG_RT:

• Enemy escapes, event show enemy - reappears
• Enemy self-destructs, event show enemy - reappears
• Enemy dies, event show enemy - stays dead

Also confirmed current master emulates these

[mateofio]

Remove the MIDI bullet as thats already tracked by #1733