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 311 forks source link

Refactor Connect() - give more control to developers #364

Open eusebiu opened 4 years ago

eusebiu commented 4 years ago

Clients' Connect() method should return at least bool so developers can verify if the method call was successful or not, without having to implement fail event handlers. Basically, now it forces developer to have the Connect() call as the last line in the method.

phalasz commented 4 years ago

Please fix the conflict.