search

libretro-mirrors / libretro-arb

For proposed improvements to libretro API.
8 stars 2 forks source link

Clarify RETRO_ENVIRONMENT_SHUTDOWN #17

Closed mprobinson closed 9 years ago

mprobinson commented 9 years ago

Current description is ambiguous, so standardize on RetroArch's behavior. Remove game specific language and note use in error handling.

Alcaro commented 9 years ago

Keep in mind that neither libretro v1 nor RetroArch supports multi-instancing. We'll want to make it clear how this env interacts with that. My proposal:

#define RETRO_ENVIRONMENT_GAME_FINISHED 7
/* N/A (NULL) --
 * Tells the front that this game has nothing more to do (the video ended, for example),
 * and further calls to retro_run will not do anything useful.
 * The next call to the core should be retro_unload_game or retro_unserialize.
 */

If this is about crash recovery, I vote just let it explode. A crash or exception is a core bug, we might as well force ourselves to fix them properly. Alternatively, if you have a proposal for exactly how the cleanup procedure should look on the front side, we could make it take an argument.

Alcaro commented 9 years ago

I'm pretty sure this one is no longer relevant.