Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exit room timer save #245

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SergioMartin86
Copy link
Contributor

The overflow glitch that allows for skips on level 10 and 12 requires that the exit room timer is conserved through savestates.

Currently in SDLpop, if you quicksave just before grabbing the ledge and quickload, then the overflow glitch won't happen. This pull request solves the problem and allows our bot to find the trick.

@mooskagh
Copy link

mooskagh commented May 6, 2021

Note that it will likely break forward and backward compatibility between savegame versions.

If so, I'd suggest to wait a bit more to gather other possible changes to break it once instead of several times.
(and in any case it would be great that even after the change, new SDLPoP could open old savegames)

@SergioMartin86
Copy link
Contributor Author

(and in any case it would be great that even after the change, new SDLPoP could open old savegames)

I purposefully appended this value at the end, so that if you load an old savestate (2908 bytes instead of 2910) you can set these missing 2 bytes to zero.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants