vlad.os/LuaCsForBarotraumaEP
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed clients incrementing their lastReceivedID every time they receive an empty entity event (even if the event has already been received)

Browse Source
This commit is contained in:
Regalis
2017-05-02 22:15:18 +03:00
parent 6c770afd6a
commit 06db2fed86
2 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
Subsurface/Source/Networking/NetEntityEvent/ClientEntityEventManager.cs
View File

@@ -130,9 +130,11 @@ namespace Barotrauma.Networking
{
UInt16 thisEventID = (UInt16)(firstEventID + (UInt16)i);
UInt16 entityID = msg.ReadUInt16();
if (entityID == 0)
if (entityID == 0 && thisEventID == (UInt16)(lastReceivedID + 1))
{
DebugConsole.NewMessage("received empty event " + thisEventID, Microsoft.Xna.Framework.Color.Orange);
msg.ReadPadBits();
lastReceivedID++;
continue;

4
Subsurface/Source/Networking/NetEntityEvent/NetEntityEventManager.cs
View File

@@ -45,9 +45,11 @@ namespace Barotrauma.Networking
tempBuffer.LengthBytes < 128,
"Maximum EntityEvent size exceeded when serializing \""+e.Entity+"\"!");
//the ID has been taken by another entity (the original entity has been removed) -> write an empty event
if (Entity.FindEntityByID(e.Entity.ID) != e.Entity)
{
//DebugConsole.ThrowError("Error in NetEntityEventManager.Write (FindEntityByID(e.Entity.ID) != e.Entity)");
//technically the clients don't have any use for these, but removing events and shifting the IDs of all
//consecutive ones is so error-prone that I think this is a safer option
msg.Write((UInt16)0);
msg.WritePadBits();
}