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

Fix potential race condition in song bar beatmap lookup flow #24691

Merged
merged 2 commits into from
Aug 31, 2023

Conversation

peppy
Copy link
Member

@peppy peppy commented Aug 31, 2023

Cancelling a web request may not necessarily cancel the callbacks. This might help with #24598.

peppy added 2 commits August 31, 2023 19:03
Cancelling a web request may not necessarily cancel the callbacks.

This might help with ppy#24598.
@peppy peppy changed the title Add more testability for song bar Fix potential race condition in song bar beatmap lookup flow Aug 31, 2023
@bdach bdach self-requested a review August 31, 2023 19:31
Copy link
Collaborator

@bdach bdach left a comment

Choose a reason for hiding this comment

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

Race manually simulated and confirmed via Scheduler.AddDelayed() calls. Looks legit.

@bdach bdach merged commit b5d8871 into ppy:master Aug 31, 2023
@peppy peppy deleted the file-ipc-race-fix branch September 1, 2023 09:07
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.

2 participants