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

tr1/option: fix crash on toggling bilinear filter #1953

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

rr-
Copy link
Collaborator

@rr- rr- commented Nov 25, 2024

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

Description

Resolves #1942.

@rr- rr- added TRX bug A bug with TRX TR1 labels Nov 25, 2024
@rr- rr- self-assigned this Nov 25, 2024
@rr- rr- requested review from a team as code owners November 25, 2024 14:18
@rr- rr- requested review from lahm86, walkawayy and aredfan and removed request for a team November 25, 2024 14:18
Copy link

github-actions bot commented Nov 25, 2024

@aredfan
Copy link
Collaborator

aredfan commented Nov 25, 2024

The game crashing is fixed. 👍

During testing, I found that toggling the texture filter in the save dialog clears the UI.

Edit: Also a regression since 4.5.

20241125_143035_Caves

@rr- rr- changed the title gtr1/option: fix crash on toggling bilinear filter tr1/option: fix crash on toggling bilinear filter Nov 25, 2024
@rr- rr- force-pushed the issue-1942-fix-bilinear-passport-crash branch from ad24519 to 46c56de Compare November 25, 2024 15:04
@rr-
Copy link
Collaborator Author

rr- commented Nov 25, 2024

During handling this bug I've noticed another one, related to changing the save slot count when the game is running. I'll create a ticket for that as it seems like a corner case scenario.
I've updated the fix for the bilinear filter toggle crash.

Copy link
Collaborator

@aredfan aredfan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 👍

@rr- rr- merged commit 5d1c3d2 into develop Nov 25, 2024
7 checks passed
@rr- rr- deleted the issue-1942-fix-bilinear-passport-crash branch November 25, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TR1 TRX bug A bug with TRX
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

TR1X bug: the game crashes when toggling texture filter while in load game dialog
2 participants