search

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

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

Closed eusebiu closed 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 submit PRs against the develop branch

eusebiu commented 4 years ago

Please submit PRs against the develop branch

Ok. Sure, will create a separate PR. You can close this one.