Thread-Safety Issues

[carlbennett]

This issue was created to track thread-safety issues in the early stages of the development of Atlas.

[carlbennett]

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 lock{} around it.

Originally posted by @carlbennett in https://github.com/BNETDocs/Atlas/pull/13#discussion_r532174135

[x3haloed]

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.

[carlbennett]

@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.