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

Try to fix game hang caused by shelf linking. #53

Merged
merged 2 commits into from
Jun 16, 2024
Merged

Try to fix game hang caused by shelf linking. #53

merged 2 commits into from
Jun 16, 2024

Conversation

Foxtr0t1337
Copy link
Contributor

Fixing an infinite loop caused by unchecked cache invalidation.

Sometime when you link shelves, a storageGroup in cache will not null, but with no SpawnedMember.

If unchecked, it will cause groupsOfPriorityCount to decrease by -1, which is a infinite loop.

This fix will FixCache when a group have no member.

#47 #44

Fixing an infinite loop caused by unchecked cache invalidation.

Sometime when you link shelves, a storageGroup in cache will not null, but with no SpawnedMember.

If unchecked, it will cause groupsOfPriorityCount to decrease by -1, which is a infinite loop.

This fix will FixCache when a group have no member.
@bbradson
Copy link
Owner

Looks good. Thanks!
SpawnedMemberCount is tracked by Fish there, and off because of groups having members assigned from more than one method, even outside of their class, with Fish simply not having every place patched. The FixCache path like used here triggers a recalculation when required

@bbradson bbradson merged commit 627d928 into bbradson:main Jun 16, 2024
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