BeardedManStudios / ForgeNetworkingRemastered

See various forks, also please join the Forge Community fork of Forge Alloy! -> https://github.com/ForgeAlloyCommunity/ForgeAlloy
https://twitter.com/FarrisFaulds
Apache License 2.0
1.49k stars 309 forks source link

Unified asynchronous task usage #332

Open ilookha opened 4 years ago

ilookha commented 4 years ago

When debugging issue #322, I noticed a bit of inconsistency in task/thread management, especially when it comes to joining all background threads on shutdown/session reset. This pull request is aimed to improve this situation.

Crazy8ball commented 4 years ago

Going to require further testing on this to test whether this to verify there are no issues introduced if we were to merge this in.