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

Check if INetworkFacade is server or client #356

Closed mcguinnessdr closed 4 years ago

mcguinnessdr commented 4 years ago

Noticed a todo in the code, does this seem like a cleaner way of checking whether to interpret a message? I made IsServer and IsClient properties on INetworkFacade.

BrentFarris commented 4 years ago

👍 Yeah, this is nice and abstracts away client/server determination so that it can be changed later. Also thanks for fixing the TODO, I drop those when I'm busy with other things and sometimes forget to come back to them