Skip to content

Commit b550976

Browse files
Revert "Use RETRO_SAVESTATE_CONTEXT_RUNAHEAD_SAME_INSTANCE for rewind savesta…" (libretro#18383)
This reverts commit aa89097.
1 parent 38dc5d8 commit b550976

File tree

3 files changed

+14
-27
lines changed

3 files changed

+14
-27
lines changed

runloop.c

Lines changed: 13 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3153,37 +3153,29 @@ bool runloop_environment_cb(unsigned cmd, void *data)
31533153
{
31543154
int result = RETRO_SAVESTATE_CONTEXT_NORMAL;
31553155

3156-
#ifdef HAVE_REWIND
3157-
if (runloop_st->rewind_st.flags &
3158-
STATE_MGR_REWIND_ST_FLAG_IS_REWIND_SERIALIZE)
3159-
result = RETRO_SAVESTATE_CONTEXT_RUNAHEAD_SAME_INSTANCE;
3160-
else
3161-
#endif
3162-
{
31633156
#if defined(HAVE_RUNAHEAD) || defined(HAVE_NETWORKING)
3164-
if (runloop_st->flags & RUNLOOP_FLAG_REQUEST_SPECIAL_SAVESTATE)
3165-
{
3157+
if (runloop_st->flags & RUNLOOP_FLAG_REQUEST_SPECIAL_SAVESTATE)
3158+
{
31663159
#ifdef HAVE_NETWORKING
3167-
if (netplay_driver_ctl(RARCH_NETPLAY_CTL_IS_ENABLED, NULL))
3168-
result = RETRO_SAVESTATE_CONTEXT_ROLLBACK_NETPLAY;
3169-
else
3160+
if (netplay_driver_ctl(RARCH_NETPLAY_CTL_IS_ENABLED, NULL))
3161+
result = RETRO_SAVESTATE_CONTEXT_ROLLBACK_NETPLAY;
3162+
else
31703163
#endif
3171-
{
3164+
{
31723165
#ifdef HAVE_RUNAHEAD
31733166
#if defined(HAVE_DYNAMIC) || defined(HAVE_DYLIB)
3174-
settings_t *settings = config_get_ptr();
3175-
if ( settings->bools.run_ahead_secondary_instance
3176-
&& (runloop_st->flags & RUNLOOP_FLAG_RUNAHEAD_SECONDARY_CORE_AVAILABLE)
3177-
&& secondary_core_ensure_exists(runloop_st, settings))
3178-
result = RETRO_SAVESTATE_CONTEXT_RUNAHEAD_SAME_BINARY;
3179-
else
3167+
settings_t *settings = config_get_ptr();
3168+
if ( settings->bools.run_ahead_secondary_instance
3169+
&& (runloop_st->flags & RUNLOOP_FLAG_RUNAHEAD_SECONDARY_CORE_AVAILABLE)
3170+
&& secondary_core_ensure_exists(runloop_st, settings))
3171+
result = RETRO_SAVESTATE_CONTEXT_RUNAHEAD_SAME_BINARY;
3172+
else
31803173
#endif
31813174
result = RETRO_SAVESTATE_CONTEXT_RUNAHEAD_SAME_INSTANCE;
31823175
#endif
3183-
}
31843176
}
3185-
#endif
31863177
}
3178+
#endif
31873179

31883180
if (data)
31893181
*(int*)data = result;

state_manager.c

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -626,9 +626,7 @@ void state_manager_event_init(
626626

627627
state_manager_push_where(rewind_st->state, &state);
628628

629-
rewind_st->flags |= STATE_MGR_REWIND_ST_FLAG_IS_REWIND_SERIALIZE;
630629
content_serialize_state_rewind(state, rewind_st->size);
631-
rewind_st->flags &= ~STATE_MGR_REWIND_ST_FLAG_IS_REWIND_SERIALIZE;
632630

633631
state_manager_push_do(rewind_st->state);
634632
}
@@ -805,9 +803,7 @@ bool state_manager_check_rewind(
805803
void *state = NULL;
806804
state_manager_push_where(rewind_st->state, &state);
807805

808-
rewind_st->flags |= STATE_MGR_REWIND_ST_FLAG_IS_REWIND_SERIALIZE;
809806
content_serialize_state_rewind(state, rewind_st->size);
810-
rewind_st->flags &= ~STATE_MGR_REWIND_ST_FLAG_IS_REWIND_SERIALIZE;
811807

812808
state_manager_push_do(rewind_st->state);
813809
}

state_manager.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,7 @@ enum state_manager_rewind_st_flags
3333
STATE_MGR_REWIND_ST_FLAG_FRAME_IS_REVERSED = (1 << 0),
3434
STATE_MGR_REWIND_ST_FLAG_INIT_ATTEMPTED = (1 << 1),
3535
STATE_MGR_REWIND_ST_FLAG_HOTKEY_WAS_CHECKED = (1 << 2),
36-
STATE_MGR_REWIND_ST_FLAG_HOTKEY_WAS_PRESSED = (1 << 3),
37-
STATE_MGR_REWIND_ST_FLAG_IS_REWIND_SERIALIZE = (1 << 4)
36+
STATE_MGR_REWIND_ST_FLAG_HOTKEY_WAS_PRESSED = (1 << 3)
3837
};
3938

4039
struct state_manager

0 commit comments

Comments
 (0)