-
Notifications
You must be signed in to change notification settings - Fork 4
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
Thread-Safety Issues #14
Comments
lock{}
around it.
Looping over the gameAds list isn't thread-safe. This may cause problems if a game is added or removed while this loop is processing. This safety issue is a systemic problem within Atlas however, and I understand if this is out of scope; I would just add a Originally posted by @carlbennett in #13 (comment) |
Maybe I can help out. Are you worried about adding locks? If not, I can go through and analyze the multithreaded stuff so it's all thread-safe with locks. |
@x3haloed If you want to fork the project and assist by creating a pull request with your work, it would be reviewed and likely welcomed. |
This issue was created to track thread-safety issues in the early stages of the development of Atlas.
The text was updated successfully, but these errors were encountered: