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

Allow overriding state lock in glitchrunner mode 2.2 or 2.0 #1133

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

InfoTeddy
Copy link
Collaborator

This fixes a regression where attempting to warp to ship with a trinket text box open in glitchrunner 2.0, and then incrementing the gamestate afterwards, would result in a softlock. This is a speedrunning strat that speedrunners use.

The state lock wasn't ever intended to remove any strats or anything, just fix warping to ship under normal circumstances. So it's okay to re-enable interrupting the state by having glitchrunner enabled.

This bug was reported by mohoc in the VVVVVV Speedrunning Discord server.

Legal Stuff:

By submitting this pull request, I confirm that...

  • My changes may be used in a future commercial release of VVVVVV
  • I will be credited in a CONTRIBUTORS file and the "GitHub Friends"
    section of the credits for all of said releases, but will NOT be compensated
    for these changes unless there is a prior written agreement

This fixes a regression where attempting to warp to ship with a trinket
text box open in glitchrunner 2.0, and then incrementing the gamestate
afterwards, would result in a softlock. This is a speedrunning strat
that speedrunners use.

The state lock wasn't ever intended to remove any strats or anything,
just fix warping to ship under normal circumstances. So it's okay to
re-enable interrupting the state by having glitchrunner enabled.

This bug was reported by mohoc in the VVVVVV Speedrunning Discord
server.
@InfoTeddy InfoTeddy merged commit 8d2a0c2 into TerryCavanagh:master Jan 23, 2024
3 checks passed
@InfoTeddy InfoTeddy deleted the general-bug-fixes branch January 23, 2024 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant