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

tr2/shell: fix exiting the game #2427

Merged
merged 1 commit into from
Feb 1, 2025
Merged

tr2/shell: fix exiting the game #2427

merged 1 commit into from
Feb 1, 2025

Conversation

rr-
Copy link
Collaborator

@rr- rr- commented Feb 1, 2025

Checklist

  • I have read the coding conventions
  • I have added a changelog entry about what my pull request accomplishes, or it is an internal change
  • I have added a readme entry about my new feature or OG bug fix, or it is a different change

Description

Resolves #2425. Regression from 1240537, which attempted to optimize away a memory leak by destroying the main window. However, this action was performed too soon, causing certain subsystems to malfunction during the shutdown phase. As a result, since GL wasn't in a stable state, not only the textures would fail to be destroyed, but also the error handling routine activated during shutdown would get stuck in an infinite loop.

Resolves #2425. Regression from 1240537.
@rr- rr- added TRX bug A bug with TRX Unreleased Applies to things that aren't in a stable release. TR2 labels Feb 1, 2025
@rr- rr- self-assigned this Feb 1, 2025
@rr- rr- requested review from a team as code owners February 1, 2025 19:04
@rr- rr- requested review from lahm86, walkawayy and aredfan and removed request for a team February 1, 2025 19:04
@rr- rr- merged commit a6bc138 into develop Feb 1, 2025
9 checks passed
@rr- rr- deleted the issue-2425-fix-tr2-exit branch February 1, 2025 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TR2 TRX bug A bug with TRX Unreleased Applies to things that aren't in a stable release.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

TR2X bug: the game persists in Task Manager's background processes
3 participants