epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

When serializing terrain positions for an order, serialize a 0-length array in a way that roundtrips.

Browse Source 
Previously, a 0 length array would not roundtrip and would deserialize as a center position instead.
This commit is contained in:
RoosterDragon
2023-08-26 15:55:22 +01:00
committed by Gustas
parent e41279fe6b
commit a67320e431
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
OpenRA.Game/Network/Order.cs
View File

@@ -157,8 +157,8 @@ namespace OpenRA
{
var pos = new WPos(r.ReadInt32(), r.ReadInt32(), r.ReadInt32());
var numberOfTerrainPositions = r.ReadByte();
if (numberOfTerrainPositions == 0)
var numberOfTerrainPositions = r.ReadInt16();
if (numberOfTerrainPositions == -1)
target = Target.FromPos(pos);
else
{
@@ -403,10 +403,10 @@ namespace OpenRA
// Don't send extra data over the network that will be restored by the Target ctor
var terrainPositions = targetState.TerrainPositions.Length;
if (terrainPositions == 1 && targetState.TerrainPositions[0] == targetState.Pos)
w.Write((byte)0);
w.Write((short)-1);
else
{
w.Write((byte)terrainPositions);
w.Write((short)terrainPositions);
foreach (var position in targetState.TerrainPositions)
{
w.Write(position.X);