Code review comment for lp:~widelands-dev/widelands/net-relay

Revision history for this message
GunChleoc (gunchleoc) wrote :

The protocol looks good to me.

There is no description of how the relay server get shut down yet, either in game end or when host and clients disappear without telling the relay about it.

I'd add a note to RLYCMD_TO_HOST that it's identical to RLYCMD_TO_CLIENT etc., to make the documentation easier to read.

There are also a bunch of commented out braces.

« Back to merge proposal