charleslemaux/Mirror
1
0
Fork 0
mirror of https://github.com/MirrorNetworking/Mirror.git synced 2024-11-18 02:50:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

OnServerAddPlayer uses NetworkMessage parameter instead of NetworkReader. Removes last dependency on NetworkReader.ReadMessage (#307)

Browse Source
This commit is contained in:
vis2k 2019-01-18 13:08:19 +01:00 committed by GitHub
parent 183afac99a
commit ce8ede3db3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Assets/Mirror/Runtime/ClientScene.cs
View File

@ -78,6 +78,7 @@ public static bool AddPlayer(NetworkConnection readyConn)
}
// use this to implicitly become ready
// -> extraMessage can contain character selection, etc.
public static bool AddPlayer(NetworkConnection readyConn, MessageBase extraMessage)
{
// ensure valid ready connection

10
Assets/Mirror/Runtime/NetworkManager.cs
View File

@ -518,8 +518,12 @@ internal void OnServerAddPlayerMessageInternal(NetworkMessage netMsg)
if (msg.value != null && msg.value.Length > 0)
{
NetworkReader reader = new NetworkReader(msg.value);
OnServerAddPlayer(netMsg.conn, reader);
// convert payload to extra message and call OnServerAddPlayer
// (usually for character selection information)
NetworkMessage extraMessage = new NetworkMessage();
extraMessage.reader = new NetworkReader(msg.value);
extraMessage.conn = netMsg.conn;
OnServerAddPlayer(netMsg.conn, extraMessage);
}
else
{
@ -629,7 +633,7 @@ public virtual void OnServerReady(NetworkConnection conn)
NetworkServer.SetClientReady(conn);
}
public virtual void OnServerAddPlayer(NetworkConnection conn, NetworkReader extraMessageReader)
public virtual void OnServerAddPlayer(NetworkConnection conn, NetworkMessage extraMessage)
{
OnServerAddPlayerInternal(conn);
}